The summary says it all. NO_SYMLINK_XATTRS is supposed to be defined on systems that don't allow xattrs on symlinks, but if this is forgotten (bug 7109), data may be lost.
Specifically, I propose that the code in xattrs.c that swallows EPERM for symlinks be removed as explained in bug 7109 comment 4.