Add NetBSD's compat syscalls

[ais2397]

Syscalls added:

• compat_90_getvfsstat

• _getvfsstat90

• compat_30_getfh

• __getfh30

• compat_30_fhopen  

• __fhopen40

• compat_30_fhstat

• compat_30___fhstat30

• __fhstat50

• compat_30_fhstatvfs1  

• compat_90_fhstatvfs1

• __fhstatvfs190

• ktrace

• fktrace

• compat_43_mmap

• compat_30___stat13

• compat_30___ltat13

• compat_30___fstat13

• compat_50___stat30

• compat_50___lstat30

• compat_50___fstat30

• compat_90_statvfs1

• compat_90_fstatvfs1

• compat_43_gethostname

• compat_43_sethostname

• compat_43_ogethostid

• compat_43_osethostid

• compat_09_getdomainname

• compat_09_setdomainname

• pathconf

• fpathconf

• compat_43_ogetdirentries

• compat_12_getdirentries

• undelete

• __posix_rename

• __posix_chown

• __posix_fchown

• __posix_lchown

• __posix_fadvise50

• compat_43_osend

• compat_43_orecv

• compat_43_orecvmsg

• compat_43_osendmsg

[dvyukov]

Hi Ayushi,

Please rebase this change, it pushes 148 commits.

[google-cla]

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

[google-cla]

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

[codecov]

Codecov Report

Merging #2640 (a87a54c) into master (282f03f) will decrease coverage by 0.0%.
The diff coverage is n/a.

❗ Current head a87a54c differs from pull request most recent head d226ad9. Consider uploading reports for the commit d226ad9 to get more accurate results
| Impacted Files | Coverage Δ | |
|---|---|---|
| pkg/csource/csource.go | 74.7% <0.0%> (-4.5%) | ⬇️ |
| prog/mutation.go | 89.0% <0.0%> (-0.2%) | ⬇️ |
| prog/any.go | 83.5% <0.0%> (ø) | |
| pkg/osutil/osutil_linux.go | 0.0% <0.0%> (ø) | |
| pkg/osutil/osutil.go | 17.3% <0.0%> (+0.4%) | ⬆️ |

[google-cla]

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

[krytarowski]

I haven't checked the values, but this patch looks rational.

[R3x]

Were these generated using sys2syz or handwritten?

[R3x]

@ais2397 any reason why the bufsize fields are marked optional here?

[ais2397]

Was referring to the man page for getvfsstat, which mentions NULL could be passed to bufsize.

[R3x]

From the man page, it only mentions NULL for the buf not for the bufsize, I think removing the [opt] would be more appropriate

     If buf is given as NULL, getvfsstat() returns just the number of mounted
     file systems.

[ais2397]

Sorry, would get it done right.

[R3x]

maybe we could use the len[] field here for the size? You seem to have used it in the other files for buffer size associations, any reason for not using them in this file?

[ais2397]

Yep, would include len[] in the description.

[dvyukov]

Were these generated using sys2syz or handwritten?

Interesting, can you share what/where is sys2syz?

[R3x]

Interesting, can you share what/where is sys2syz?

The syz-headerparser was not properly working for NetBSD, so during GSoC 2020, we worked on automating grammar generation up to a certain point, including extraction of ioctl calls.

Here is the link to the repo : https://github.com/ais2397/sys2syz

[dvyukov]

Thanks, added the link to #590

[ais2397]

Were these generated using sys2syz or handwritten?

Sys2syz, mostly, however flags were added by hand.

[R3x]

@ais2397 is there anything else to be done here? Shall I merge this?

[ais2397]

@ais2397 is there anything else to be done here? Shall I merge this?

Yes, we can merge this.

[ais2397]

@R3x This one is ready to be merged.