On Wed, Dec 6, 2017 at 12:10 AM, Theodore Ts'o <tytso@xxxxxxx> wrote: > > On Mon, Dec 04, 2017 at 10:56:35PM +0900, Wang Shilong wrote: > > From: Wang Shilong <wshilong@xxxxxxx> > > > > Steps to reproduce: > > > > $ touch a > > $ lsattr -dp a > > 2000 ----------------P a > > $ ln -s a b > > $ lsattr -dp b > > lsattr: Operation not supported While reading flags on b > > > > Link files should be supported, fix it. > > > > Signed-off-by: Wang Shilong <wshilong@xxxxxxx> > > You can't actually set or get the extended attributes for a symlink; > the *kernel* doesn't support what you seem to be hoping to achieve. > This patch result in fgetflags() following the symlink and opening > whatever file it points at. Yup, you are right. But this is one of our customers feedback, this is not good that lsattr/chattr did not support symlink, we might need make it clear, for example, we support symlink, but it always follow original files, that is even better than output errors. In this way, we need fix chattr too. What do you think? Thanks, Shilong > > - Ted
