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 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
signature.asc
Description: PGP signature
