[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v2 7/8] gluster: Support auto-read-only option
From: |
Niels de Vos |
Subject: |
Re: [Qemu-block] [PATCH v2 7/8] gluster: Support auto-read-only option |
Date: |
Sun, 14 Oct 2018 13:04:03 +0200 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
On Fri, Oct 12, 2018 at 12:31:21PM -0500, Eric Blake wrote:
> On 10/12/18 6:55 AM, Kevin Wolf wrote:
> > If read-only=off, but auto-read-only=on is given, open the file
> > read-write if we have the permissions, but instead of erroring out for
> > read-only files, just degrade to read-only.
> >
> > Signed-off-by: Kevin Wolf <address@hidden>
> > ---
> > block/gluster.c | 9 +++++++++
> > 1 file changed, 9 insertions(+)
> >
> > diff --git a/block/gluster.c b/block/gluster.c
> > index 4fd55a9cc5..68d20c8830 100644
> > --- a/block/gluster.c
> > +++ b/block/gluster.c
> > @@ -849,6 +849,15 @@ static int qemu_gluster_open(BlockDriverState *bs,
> > QDict *options,
> > qemu_gluster_parse_flags(bdrv_flags, &open_flags);
> > s->fd = glfs_open(s->glfs, gconf->path, open_flags);
> > + if (!s->fd && errno == EACCES) {
>
> EROFS is not possible as it was for posix file?
EROFS can happen, depending on the configuration of the Gluster volume.
In that case, opening read-only should work fine.
Niels
- Re: [Qemu-block] [PATCH v2 2/8] block: Add auto-read-only option, (continued)
[Qemu-block] [PATCH v2 3/8] block: Require auto-read-only for existing fallbacks, Kevin Wolf, 2018/10/12
[Qemu-block] [PATCH v2 4/8] nbd: Support auto-read-only option, Kevin Wolf, 2018/10/12
[Qemu-block] [PATCH v2 7/8] gluster: Support auto-read-only option, Kevin Wolf, 2018/10/12
[Qemu-block] [PATCH v2 5/8] file-posix: Support auto-read-only option, Kevin Wolf, 2018/10/12
[Qemu-block] [PATCH v2 6/8] curl: Support auto-read-only option, Kevin Wolf, 2018/10/12
[Qemu-block] [PATCH v2 8/8] iscsi: Support auto-read-only option, Kevin Wolf, 2018/10/12