Re: PATCH: fix-readonly-policy-use-and-floppy-ro-rw-status

[Jon Masters <jonathan@jonmasters.org>]

On Sat, Nov 05, 2005 at 10:33:58AM -0800, Andrew Morton wrote:

> Jon Masters <jonathan@jonmasters.org> wrote:
> >
> > [PATCH]: This modifies the gendisk and hd_struct structs to replace "policy"
> >  with "readonly" (as that's the only use for this field). It also introduces a
> >  new function disk_read_only, which behaves like the corresponding device
> >  functions do. I've also replaced direct usage of the old policy fields with
> >  calls to the appropriate function.
> 
> These are separate things and should be done in separate patches, please.

I'm following up with another patch that just has the first bit (floppy)
removed.

> Because, for exmaple, we may decide to revert the floppy change only. 
> Because, as I said off-list, being able to do `remount,rw' of a floppy after
> having flipped its switch is useful.

And as I said, the situation as it stands leads to potential data
corruption but I agree with you - we need a VFS callback to handle
readwrite/readonly change on remount I think. Comments?

Jon.