Re: [PATCH] cramfs is ro only, so honour this in inode->mode

2001-01-09 Thread Doug McNaught
"Albert D. Cahalan" <[EMAIL PROTECTED]> writes: > Doug writes: > > bash-2.03$ cd /tmp > > bash-2.03$ cat >foo > > This is a test. > > bash-2.03$ chmod u-r foo > > No, you zeroed the owner's read bit. When the bit isn't > implemented it must be always set. > > By "(owner may read own files)" I

Re: [PATCH] cramfs is ro only, so honour this in inode->mode

2001-01-09 Thread Albert D. Cahalan
Doug writes: > "Albert D. Cahalan" <[EMAIL PROTECTED]> writes: >> If you need to steal a bit, grab one that won't hurt. >> Take the owner's read bit. (owner may read own files) > > Er, > > bash-2.03$ cd /tmp > bash-2.03$ cat >foo > This is a test. > bash-2.03$ chmod u-r foo No, you zeroed the

Re: [PATCH] cramfs is ro only, so honour this in inode->mode

2001-01-09 Thread Doug McNaught
"Albert D. Cahalan" <[EMAIL PROTECTED]> writes: > Shane Nay writes: > > > but the bits are useless in the "normal interpretation" of it, > ... > > But then you pull out the write bits, > > If you need to steal a bit, grab one that won't hurt. > Take the owner's read bit. (owner may read own

Re: [PATCH] cramfs is ro only, so honour this in inode->mode

2001-01-09 Thread Albert D. Cahalan
Shane Nay writes: > but the bits are useless in the "normal interpretation" of it, ... > But then you pull out the write bits, If you need to steal a bit, grab one that won't hurt. Take the owner's read bit. (owner may read own files) - To unsubscribe from this list: send the line "unsubscribe

Re: [PATCH] cramfs is ro only, so honour this in inode-mode

2001-01-09 Thread Albert D. Cahalan
Shane Nay writes: but the bits are useless in the "normal interpretation" of it, ... But then you pull out the write bits, If you need to steal a bit, grab one that won't hurt. Take the owner's read bit. (owner may read own files) - To unsubscribe from this list: send the line "unsubscribe

Re: [PATCH] cramfs is ro only, so honour this in inode-mode

2001-01-09 Thread Doug McNaught
"Albert D. Cahalan" [EMAIL PROTECTED] writes: Shane Nay writes: but the bits are useless in the "normal interpretation" of it, ... But then you pull out the write bits, If you need to steal a bit, grab one that won't hurt. Take the owner's read bit. (owner may read own files) Er,

Re: [PATCH] cramfs is ro only, so honour this in inode-mode

2001-01-09 Thread Albert D. Cahalan
Doug writes: "Albert D. Cahalan" [EMAIL PROTECTED] writes: If you need to steal a bit, grab one that won't hurt. Take the owner's read bit. (owner may read own files) Er, bash-2.03$ cd /tmp bash-2.03$ cat foo This is a test. bash-2.03$ chmod u-r foo No, you zeroed the owner's read

Re: [PATCH] cramfs is ro only, so honour this in inode-mode

2001-01-09 Thread Doug McNaught
"Albert D. Cahalan" [EMAIL PROTECTED] writes: Doug writes: bash-2.03$ cd /tmp bash-2.03$ cat foo This is a test. bash-2.03$ chmod u-r foo No, you zeroed the owner's read bit. When the bit isn't implemented it must be always set. By "(owner may read own files)" I refer to what

Re: [PATCH] cramfs is ro only, so honour this in inode->mode

2001-01-08 Thread Linus Torvalds
On Tue, 9 Jan 2001, Rusty Russell wrote: > In message <[EMAIL PROTECTED]> you > write: > > I've been thinking of doing a cramfs2, and the only thing I'd change is > > (a) slightly bigger blocksize (maybe 8k or 16k) and (b) re-order the > > meta-data and the real data so that I could easily

Re: [PATCH] cramfs is ro only, so honour this in inode->mode

2001-01-08 Thread Rusty Russell
In message <[EMAIL PROTECTED]> you write: > I've been thinking of doing a cramfs2, and the only thing I'd change is > (a) slightly bigger blocksize (maybe 8k or 16k) and (b) re-order the > meta-data and the real data so that I could easily compress the metadata > too. cramfs doesn't have any

Re: [PATCH] cramfs is ro only, so honour this in inode->mode

2001-01-08 Thread Linus Torvalds
On Mon, 8 Jan 2001, Ingo Oeser wrote: > On Mon, Jan 08, 2001 at 12:13:39PM +, Shane Nay wrote: > > This may not initially seem like such a great thing..., but imagine a base > > distro being distributed as a cramfs file. Copy the thing over to your HD > > and you're done, otherwise the

Re: [PATCH] cramfs is ro only, so honour this in inode->mode

2001-01-08 Thread Linus Torvalds
On Mon, 8 Jan 2001, Ingo Oeser wrote: > > cramfs is a read-only fs. So we should honour that in inode->mode to > avoid confusion of programs. No no no. This breaks device nodes etc quite badly. A change to mkcramfs might be fine - but it has to conditionalize on the file being a regular

Re: [PATCH] cramfs is ro only, so honour this in inode->mode

2001-01-08 Thread Ingo Oeser
On Mon, Jan 08, 2001 at 08:42:35AM -0500, Alexander Viro wrote: > If program considers these bits of st_mode as indication of ability > to write into file - program is buggy and should be fixed. Regardless > of cramfs. Ok, point taken. I fixed the generation of the tree to be crammed into the

Re: [PATCH] cramfs is ro only, so honour this in inode->mode

2001-01-08 Thread Shane Nay
Ingo, > You can use (GNU-)tar for this. It even keeps track of other bits like > ext2fs attributes, AFAIK. True..., but cramfs is acting like a mountable (tar czvf) because of the compressed pages. Seems redundant to have a tar on top of what is basically a segmented tar with frontal

Re: [PATCH] cramfs is ro only, so honour this in inode->mode

2001-01-08 Thread Alexander Viro
On Mon, 8 Jan 2001, Ingo Oeser wrote: > Then we might need W bits, but currently they disturb things like > "test" and the perl equivalent, which is quite annoying and > complexifies code. (Yes, I'm selfish too ;-)) Huh??? Consider write-protected floppy. What, you mean that it also should

Re: [PATCH] cramfs is ro only, so honour this in inode->mode

2001-01-08 Thread Ingo Oeser
On Mon, Jan 08, 2001 at 12:13:39PM +, Shane Nay wrote: > This may not initially seem like such a great thing..., but imagine a base > distro being distributed as a cramfs file. Copy the thing over to your HD > and you're done, otherwise the distro packaging has to keep track of >

Re: [PATCH] cramfs is ro only, so honour this in inode->mode

2001-01-08 Thread Shane Nay
Ingo, > cramfs is a read-only fs. So we should honour that in inode->mode > to avoid confusion of programs. > > My isofs shows this too, so I think I'm right deleting the write > permissions in the inode. May be we should change it in > mkcramfs (too). > > I don't know what POSIX says about RO

[PATCH] cramfs is ro only, so honour this in inode->mode

2001-01-08 Thread Ingo Oeser
Hi Linus, hi all, cramfs is a read-only fs. So we should honour that in inode->mode to avoid confusion of programs. My isofs shows this too, so I think I'm right deleting the write permissions in the inode. May be we should change it in mkcramfs (too). I don't know what POSIX says about RO fs,

[PATCH] cramfs is ro only, so honour this in inode-mode

2001-01-08 Thread Ingo Oeser
Hi Linus, hi all, cramfs is a read-only fs. So we should honour that in inode-mode to avoid confusion of programs. My isofs shows this too, so I think I'm right deleting the write permissions in the inode. May be we should change it in mkcramfs (too). I don't know what POSIX says about RO fs,

Re: [PATCH] cramfs is ro only, so honour this in inode-mode

2001-01-08 Thread Shane Nay
Ingo, cramfs is a read-only fs. So we should honour that in inode-mode to avoid confusion of programs. My isofs shows this too, so I think I'm right deleting the write permissions in the inode. May be we should change it in mkcramfs (too). I don't know what POSIX says about RO fs, but I

Re: [PATCH] cramfs is ro only, so honour this in inode-mode

2001-01-08 Thread Ingo Oeser
On Mon, Jan 08, 2001 at 12:13:39PM +, Shane Nay wrote: This may not initially seem like such a great thing..., but imagine a base distro being distributed as a cramfs file. Copy the thing over to your HD and you're done, otherwise the distro packaging has to keep track of permisions

Re: [PATCH] cramfs is ro only, so honour this in inode-mode

2001-01-08 Thread Alexander Viro
On Mon, 8 Jan 2001, Ingo Oeser wrote: Then we might need W bits, but currently they disturb things like "test" and the perl equivalent, which is quite annoying and complexifies code. (Yes, I'm selfish too ;-)) Huh??? Consider write-protected floppy. What, you mean that it also should

Re: [PATCH] cramfs is ro only, so honour this in inode-mode

2001-01-08 Thread Shane Nay
Ingo, You can use (GNU-)tar for this. It even keeps track of other bits like ext2fs attributes, AFAIK. True..., but cramfs is acting like a mountable (tar czvf) because of the compressed pages. Seems redundant to have a tar on top of what is basically a segmented tar with frontal indexing

Re: [PATCH] cramfs is ro only, so honour this in inode-mode

2001-01-08 Thread Ingo Oeser
On Mon, Jan 08, 2001 at 08:42:35AM -0500, Alexander Viro wrote: If program considers these bits of st_mode as indication of ability to write into file - program is buggy and should be fixed. Regardless of cramfs. Ok, point taken. I fixed the generation of the tree to be crammed into the

Re: [PATCH] cramfs is ro only, so honour this in inode-mode

2001-01-08 Thread Linus Torvalds
On Mon, 8 Jan 2001, Ingo Oeser wrote: cramfs is a read-only fs. So we should honour that in inode-mode to avoid confusion of programs. No no no. This breaks device nodes etc quite badly. A change to mkcramfs might be fine - but it has to conditionalize on the file being a regular file.

Re: [PATCH] cramfs is ro only, so honour this in inode-mode

2001-01-08 Thread Linus Torvalds
On Mon, 8 Jan 2001, Ingo Oeser wrote: On Mon, Jan 08, 2001 at 12:13:39PM +, Shane Nay wrote: This may not initially seem like such a great thing..., but imagine a base distro being distributed as a cramfs file. Copy the thing over to your HD and you're done, otherwise the distro

Re: [PATCH] cramfs is ro only, so honour this in inode-mode

2001-01-08 Thread Rusty Russell
In message [EMAIL PROTECTED] you write: I've been thinking of doing a cramfs2, and the only thing I'd change is (a) slightly bigger blocksize (maybe 8k or 16k) and (b) re-order the meta-data and the real data so that I could easily compress the metadata too. cramfs doesn't have any

Re: [PATCH] cramfs is ro only, so honour this in inode-mode

2001-01-08 Thread Linus Torvalds
On Tue, 9 Jan 2001, Rusty Russell wrote: In message [EMAIL PROTECTED] you write: I've been thinking of doing a cramfs2, and the only thing I'd change is (a) slightly bigger blocksize (maybe 8k or 16k) and (b) re-order the meta-data and the real data so that I could easily compress the