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@.

In D53368#1218929, @markj wrote:

There is a similar pattern in nfsrpc_writeds(). Does that need to be fixed too?

Add the same fix to nfsrpc_writeds().

Oh, and it needs the date to be updated.