Thanks for the atomic_load_short explanation.

It all looks good to me, although I won't claim to
understand all the required semantics in
vn_lock_delayed_setsize().

Changed the variable name has_export to only_export,
as suggested by kib@. I also changed the comment
related to this vaiable.

Thanks for doing this. I think it does clarify
the relationship between EXTATTR_NAMESPACE_EMPTY
and EXTATTR_NAMESPACE_EMPTY_STRING for your
other patch.

It looks ok to me, although I do not know
anything about the atf testing part.

I probably should resign, since I don't understand
when/if ACCEPT4_INHERIT might be specified
as an argument for solisten_dequeue()?

This looks fine to me.
I will note that the NFS server never uses svc_vc_accept().
(The accept() is done in userspace.)

Added NET_EPOCH_ENTER()/NET_EPOCH_EXIT() and
cleaned up the code a bit.

As suggested by jilles@ and to be more consistent
with rmextattr(8), change "-S/--symlink" to "-h/--nofollow".

In D55023#1257720, @rmacklem wrote:

In D55023#1257715, @kib wrote:

BTW, chmod(1), chflags(1), ls(1) and find(1) consistently use -P for the same functionality. ls(1) does not fail if the argument is not a symbolic link.

I can switch it to "-P". It was Lionel Cons that suggested "-S".

In D55023#1257715, @kib wrote:

BTW, chmod(1), chflags(1), ls(1) and find(1) consistently use -P for the same functionality. ls(1) does not fail if the argument is not a symbolic link.

I'm going to redo this using getopt().
I didn't bother for one option, but I
now think that's silly.

Looks fine to me. I'll admit I didn't bother to look
to see if/where they get called.

I know nothing about gcc flags, but if it gets
rid of the error, that seems fine to me.

Do roundup2() after the min(args->count, MAXBSIZE)
as suggsted by markj@.

Did formatting changes suggested by Ziaee.

Made formatting changes suggested by kib@.

Updated as suggested by kib@.

Add words similar to what was in the email,
as requested by kib@.

Add ziaee@'s suggested change.

Added a short description of the rc.conf
variables, as suggested by kib@.

Commit to OpenZFS was done in August 2024.

Looks fine to me.

Reword the line to say "performs case insensitive lookups,"
as suggested by kib@.