Port Native libraries to SunOS#34867
Port Native libraries to SunOS#34867jkotas merged 1 commit intodotnet:masterfrom am11:feature/solaris/libraries-port
Conversation
src/libraries/Common/src/Interop/Unix/System.Native/Interop.SysLog.cs
Outdated
Show resolved
Hide resolved
src/libraries/Common/src/Interop/Unix/System.Native/Interop.SysLog.cs
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
On Linux, cfmakeraw shows without the IMAXBEL. It was really just convenience and I'm wondering if we can get away without the fork and always list all options. (that can be done later)
There was a problem hiding this comment.
Yes. I think three things can be simplified:
- cfmakeraw -> list of options
- cfsetspeed -> i/o speed separately
- statfs -> statvfs
- former is kernel call, latter is libc standard function, and Solaris/Illumos is known to err to the libc functions generally, without
sysctl(2)like Linux orsysctl(3)like BSD.
- former is kernel call, latter is libc standard function, and Solaris/Illumos is known to err to the libc functions generally, without
I wanted to keep changes simple for the first pass, however, can make these changes and simplify the PR. Wdyt, should we make these three now or postpone it until later?
There was a problem hiding this comment.
same as above. cfsetspeed() is a 4.4BSD extension. We may use cfsetispeed and cfsetospeed on all platforms.
|
I think this work item can have os-sunos and area-Infrastructure-libraries labels. |
|
Tagging @safern, @ViktorHofer as an area owner. If you would like to be tagged for a label, please notify danmosemsft. |
Summary of changes:
MountPoints.FormatInfo.__c_static_assert__.cfsetspeedis not supported.cfmakerawis not supported.netpacket/packet.handnet/if_arp.hheaders.fopenfallback, asgetmntinfois not supported.statfs(2)prototype.statfs(2)with different prototype is supported, preferred way is to use libc'sstatvfs(3).RLIMIT_*resources are different/non-supported, mapped them accordingly.getgrouplist(3).Contributes to: #4173.