[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 09/13] 9p: darwin: Provide a compatibility defin
From: |
Keno Fischer |
Subject: |
Re: [Qemu-devel] [PATCH 09/13] 9p: darwin: Provide a compatibility definition for XATTR_SIZE_MAX |
Date: |
Sat, 26 May 2018 12:00:30 -0400 |
> > +#if defined(CONFIG_DARWIN) && !defined(XATTR_SIZE_MAX)
> > +/* Darwin doesn't seem to define a maximum xattr size in its user
> > + user space header, but looking at the kernel source, HFS supports
> > + up to INT32_MAX, so use that as the maximum.
> > +*/
> > +#define XATTR_SIZE_MAX INT32_MAX
> > +#endif
>
> Do we really need the CONFIG_DARWIN part of this check?
Right now this code only runs on Linux (and Darwin after this series).
On Linux it's always defined,
but I'd rather this code give an error when somebody tries to port it
to a new OS than have it
silently use an incorrect value. The ` !defined(XATTR_SIZE_MAX)` is
just there in case Apple ever
decides to define it in their headers. I can remove that part if you
would prefer.
- Re: [Qemu-devel] [PATCH 06/13] 9p: darwin: Address minor differences, (continued)
- [Qemu-devel] [PATCH 08/13] 9p: darwin: Ignore O_{NOATIME, DIRECT}, keno, 2018/05/26
- [Qemu-devel] [PATCH 09/13] 9p: darwin: Provide a compatibility definition for XATTR_SIZE_MAX, keno, 2018/05/26
- [Qemu-devel] [PATCH 10/13] 9p: darwin: *xattr_nofollow implementations, keno, 2018/05/26
- [Qemu-devel] [PATCH 11/13] 9p: darwin: Mark mknod as unsupported, keno, 2018/05/26
- Re: [Qemu-devel] [PATCH 11/13] 9p: darwin: Mark mknod as unsupported, Greg Kurz, 2018/05/30
- Re: [Qemu-devel] [PATCH 11/13] 9p: darwin: Mark mknod as unsupported, Keno Fischer, 2018/05/31
- Re: [Qemu-devel] [PATCH 11/13] 9p: darwin: Mark mknod as unsupported, Greg Kurz, 2018/05/31
- Re: [Qemu-devel] [PATCH 11/13] 9p: darwin: Mark mknod as unsupported, Keno Fischer, 2018/05/31
- Re: [Qemu-devel] [PATCH 11/13] 9p: darwin: Mark mknod as unsupported, Keno Fischer, 2018/05/31
- Re: [Qemu-devel] [PATCH 11/13] 9p: darwin: Mark mknod as unsupported, Keno Fischer, 2018/05/31
[Qemu-devel] [PATCH 12/13] 9p: darwin: Provide a fallback implementation for utimensat, keno, 2018/05/26