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

From: Jon Masters
Date: Sat Nov 05 2005 - 13:40:50 EST


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

> Jon Masters <jonathan@xxxxxxxxxxxxxx> 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.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/