Re: [PATCH] get/set FAT filesystem attribute bits

[Nicholas Miell <nmiell@comcast.net>]

On Mon, 2005-01-03 at 15:22 -0800, H. Peter Anvin wrote:
> [Pruning the Cc: list.]
> 
> Nicholas Miell wrote:
> > On Mon, 2005-01-03 at 14:25 -0800, H. Peter Anvin wrote:
> > 
> >>I'm honestly not sure that using an ASCII string in an xattr is the sane 
> >>way of doing this.  Even a binary byte in an xattr would make more sense 
> >>in some ways.
> > 
> > ASCII strings require no special tools to manipulate from shell scripts.
> > 
> 
> You need some kind of special tool anyway, i.e. getfattr/setfattr.  What 
> tool you use isn't really important.

I was talking about getdosattr and setdosattr (and the corresponding
pair for every other filesystem with it's own set of special
attributes).

getfattr and setfattr are standard tools already provided with the
distro.

> The fact that getfattr/setfattr can't deal with attributes that aren't 
> ASCII strings seem like flaws in these tools.

They can. Non-ASCII xattrs are either base64 encoded or octal escaped.

Try

getfattr -n system.posix_acl_access some_file_with_an_acl
getfattr -e text -n system.posix_acl_access some_file_with_an_acl

for a quick example.

> > The design does allow users to store whatever they want as an xattr
> > without having to worry about how the kernel chooses to interpret it,
> > though. (i.e. the user namespace is just a byte array that the kernel
> > stores for you, while the system/security namespaces are probably
> > generated and interpreted on demand.)
>  >
> 
> Exactly, and that's a total screwup.  It makes something that would 
> otherwise be possible -- for some filesystems to have an attribute (call 
> it "system.dosattrib") which is used, and for others which is stored. 
> The problem is that with the current design, that won't happen.
> 

I responded to this in the other thread already.

> Encoding this in the namespace, therefore preventing this kind of 
> compatiblity, is daft.  From the looks of it, the CIFS people were 
> planning to do the "put everything in user.*" workaround for this design 
> error.
> 



> 	-hpa
-- 
Nicholas Miell <nmiell@comcast.net>