On Fri, Apr 4, 2025 at 10:50 AM Shawn Webb <shawn.w...@hardenedbsd.org> wrote: > > On Thu, Apr 03, 2025 at 06:12:59PM -0700, Rick Macklem wrote: > > On Thu, Apr 3, 2025 at 4:52 PM Shawn Webb <shawn.w...@hardenedbsd.org> > > wrote: > > > > > > On Wed, Apr 02, 2025 at 01:51:26PM -0700, Rick Macklem wrote: > > > > The commit 2ec2ba7e232d just hit main. I do not think it will > > > > cause problems, but it is fairly large. > > > > > > > > Man page updates will be done as separate commits. > > > > > > > > Hopefully this will not cause grief, rick > > > > > > Hey Rick, > > > > > > The patch review test plan mentions a patch to ZFS itself to support > > > named attributes. Is that patch available somewhere? > > The ZFS patch is currently in phabricator as D49654. > > Feel free to review it. > > > > It can also be found at: > > https://people.freebsd.org/~rmacklem/zfs-xattr.patch > > (this is a smaller diff which can be applied to an up-to-date main src > > tree easily) > > Hey Rick, > > I applied that zfs patch, but trying pathconf(2) on a file on a ZFS > dataset with xattr=on (which seems to be the default) returns 0. Am I > doing something wrong? > > ==== BEGIN LOG ==== > hbsd-current-01[shawn]:/home/shawn/tmp $ ./xattrtest xattrtest > xattrtest: Named attributes not enabled: No error: 0 > hbsd-current-01[shawn]:/home/shawn/tmp (1) $ zfs list /usr/home/shawn > NAME USED AVAIL REFER MOUNTPOINT > rpool/usr/home 10.4G 71.4G 9.85G /usr/home > hbsd-current-01[shawn]:/home/shawn/tmp $ zfs get xattr rpool/usr/home > NAME PROPERTY VALUE SOURCE > rpool/usr/home xattr on default > ==== END LOG ==== > > That xattrtest application is yours from: > https://people.freebsd.org/~rmacklem/xattrtest.c No idea. It works for me. You used up-to-date kernel sources? (Check that VIRF_NAMEDATTR is defined in sys/sys/vnode.h.) Oh, and one more thing to check. zfs_xattr_compat needs to be non-zero. (It's found in sys/contrib/openzfs/module/os/freebsd/zfs/zfs_vnops_os.c. It's initialized to 1 and I don't see anything that sets it to 0?)
The only thing I can think if is, if you changed xattr to on, you need to reboot (or at least remount) to get it to take effect. (Maybe try setting it to "dir" and then reboot/remount. Maybe there is a difference between "on" and "dir"?) Or, did you build zfs.ko some other way than as part of a kernel build? (It needs the patched .h files in the kernel sources, not something in /usr/include/sys that has not yet been updated.) All the ZFS changes are #ifdef'd, since OpenZFS requires the sources build for older kernels. (Basically #ifdef'd on that VIRF_NAMEDATTR mentioned above.) It does remind me that I need to try a build of zfs.ko by doing a "make" in the module directory. You can try the attached trivial patch and see if it spits out "pathconf ret=1" on the console. rick > > Thanks, > > -- > Shawn Webb > Cofounder / Security Engineer > HardenedBSD > > Signal Username: shawn_webb.74 > Tor-ified Signal: +1 303-901-1600 / shawn_webb_opsec.50 > https://git.hardenedbsd.org/hardenedbsd/pubkeys/-/raw/master/Shawn_Webb/03A4CBEBB82EA5A67D9F3853FF2E67A277F8E1FA.pub.asc
pathconf-zfs.patch
Description: Binary data
