Re: [Qemu-devel] [RFC/PATCH] virtio-9p: Add Read only support for 9p export.

[Stefan Hajnoczi]

On Mon, May 23, 2011 at 12:49 PM, M. Mohan Kumar <address@hidden> wrote:
> On Mon, May 23, 2011 at 11:08:34AM +0100, Stefan Hajnoczi wrote:
>> On Mon, May 23, 2011 at 8:58 AM, M. Mohan Kumar <address@hidden> wrote:
>> > A new fsdev parameter "access" is introduced to control accessing 9p 
>> > export.
>> > access=ro|rw can be used to specify the access type. By default rw access
>> > is given to 9p export.
>>
>> It would be consistent with -drive to use readonly=on|off (default:
>> off).  Do you have any future access modes in mind that require more
>> than readonly on|off?
>
> You mean to use 'readonly=on|off' instead of 'access=ro|rw' ?

Yes.

>
>>
>> It seems the client does not know ahead of time that the mount is
>> read-only.  I wonder if this exposes any weirdness compared to
>> actually mounting ro on the client side (i.e. operations that appear
>> to start okay but then fail with -EROFS at a different point than if
>> you had mounted ro)?  It could cause apps to error in new ways.
>>
>
> When a NFS share is exported as read-only, client will come to know about
> that when it does some write operation. IMHO its not an issue.

If it works the same way on NFS then great.

Stefan