Re: [patch 10/26] mount options: fix devpts

2008-01-25 Thread Miklos Szeredi
> > Also add minor fix: when parsing the "mode" option, mask with > > S_IALLUGO instead of ~S_IFMT, which could leave unsed bits in the > > mask. > > umode_t is 16 bits, so it doesn't. The change is still good, of course. We still use 16 bit types? Strange ;) > > > + if (config.mode != DEVP

Re: [patch 10/26] mount options: fix devpts

2008-01-24 Thread H. Peter Anvin
Miklos Szeredi wrote: Also add minor fix: when parsing the "mode" option, mask with S_IALLUGO instead of ~S_IFMT, which could leave unsed bits in the mask. umode_t is 16 bits, so it doesn't. The change is still good, of course. + if (config.mode != DEVPTS_DEFAULT_MODE) +

[patch 10/26] mount options: fix devpts

2008-01-24 Thread Miklos Szeredi
From: Miklos Szeredi <[EMAIL PROTECTED]> Add a .show_options super operation to devpts. Also add minor fix: when parsing the "mode" option, mask with S_IALLUGO instead of ~S_IFMT, which could leave unsed bits in the mask. Signed-off-by: Miklos Szeredi <[EMAIL PROTECTED]> --- Index: linux/fs/dev