From: Kevin Wolf <kwolf@redhat.com>
To: Stefan Hajnoczi <stefanha@redhat.com>
Cc: qemu-block@nongnu.org, armbru@redhat.com, pkrempa@redhat.com,
qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH for-4.0 0/2] file-posix: query-qemu-features for auto-read-only
Date: Fri, 5 Apr 2019 14:46:08 +0200 [thread overview]
Message-ID: <20190405124608.GA4597@linux.fritz.box> (raw)
In-Reply-To: <20190405103247.GG25152@stefanha-x1.localdomain>
[-- Attachment #1: Type: text/plain, Size: 1166 bytes --]
Am 05.04.2019 um 12:32 hat Stefan Hajnoczi geschrieben:
> On Thu, Mar 28, 2019 at 07:28:08PM +0100, Kevin Wolf wrote:
> > auto-read-only=on changed its behaviour in file-posix for the 4.0
> > release. This change cannot be detected through the usual mechanisms
> > like schema introspection.
> >
> > I took Stefan's patch 'qmp: add query-qemu-capabilities', removed the
> > feature that he originally wanted this for and addressed the review
> > comments. The introduction of the file-posix-dynamic-auto-read-only
> > feature is a separate patch.
> >
> > Stefan, I kept you as the author of the first patch and also your
> > Signed-off-by. Is this okay with you or would you prefer a change?
>
> Sorry I was unresponsive. I'm fine with the authorship and
> Signed-off-by: but won't be offended if either or both are removed when
> the patch is reworked (my code is trivial).
The plan is now that we integrate features in the QAPI schema (attached
to struct definitions), so they will be available with the normal schema
introspection. Obviously, this will be a completely new patch series
that isn't based on this patch any more.
Kevin
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Kevin Wolf <kwolf@redhat.com>
To: Stefan Hajnoczi <stefanha@redhat.com>
Cc: pkrempa@redhat.com, armbru@redhat.com, qemu-block@nongnu.org,
qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH for-4.0 0/2] file-posix: query-qemu-features for auto-read-only
Date: Fri, 5 Apr 2019 14:46:08 +0200 [thread overview]
Message-ID: <20190405124608.GA4597@linux.fritz.box> (raw)
Message-ID: <20190405124608.n-SHnhQfdezt5RWEbRTqzQ8yTnjwvS7NnwbCOVg79E0@z> (raw)
In-Reply-To: <20190405103247.GG25152@stefanha-x1.localdomain>
[-- Attachment #1: Type: text/plain, Size: 1166 bytes --]
Am 05.04.2019 um 12:32 hat Stefan Hajnoczi geschrieben:
> On Thu, Mar 28, 2019 at 07:28:08PM +0100, Kevin Wolf wrote:
> > auto-read-only=on changed its behaviour in file-posix for the 4.0
> > release. This change cannot be detected through the usual mechanisms
> > like schema introspection.
> >
> > I took Stefan's patch 'qmp: add query-qemu-capabilities', removed the
> > feature that he originally wanted this for and addressed the review
> > comments. The introduction of the file-posix-dynamic-auto-read-only
> > feature is a separate patch.
> >
> > Stefan, I kept you as the author of the first patch and also your
> > Signed-off-by. Is this okay with you or would you prefer a change?
>
> Sorry I was unresponsive. I'm fine with the authorship and
> Signed-off-by: but won't be offended if either or both are removed when
> the patch is reworked (my code is trivial).
The plan is now that we integrate features in the QAPI schema (attached
to struct definitions), so they will be available with the normal schema
introspection. Obviously, this will be a completely new patch series
that isn't based on this patch any more.
Kevin
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]
next prev parent reply other threads:[~2019-04-05 12:46 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20190328182810.21497-1-kwolf@redhat.com>
[not found] ` <20190328182810.21497-2-kwolf@redhat.com>
[not found] ` <2fc10494-0f79-8d18-610d-be21cfc6c30b@redhat.com>
[not found] ` <87ftr5iy8i.fsf@dusky.pond.sub.org>
[not found] ` <c233c2b4-5257-2df7-3c1a-06bcf269d657@redhat.com>
[not found] ` <20190329170439.GK5081@localhost.localdomain>
[not found] ` <87o95sa40r.fsf@dusky.pond.sub.org>
[not found] ` <87y34v42z0.fsf_-_@dusky.pond.sub.org>
2019-04-01 11:54 ` [Qemu-devel] Options for 4.0 (was: [PATCH 1/2] qmp: Add query-qemu-features) Kevin Wolf
2019-04-01 13:11 ` [Qemu-devel] Options for 4.0 Markus Armbruster
2019-04-01 14:08 ` Kevin Wolf
2019-04-01 15:31 ` Markus Armbruster
2019-04-01 19:54 ` Markus Armbruster
2019-04-01 12:20 ` [Qemu-devel] Options for 4.0 (was: [PATCH 1/2] qmp: Add query-qemu-features) Peter Krempa
2019-04-01 14:55 ` [Qemu-devel] Options for 4.0 Markus Armbruster
2019-04-01 16:07 ` Peter Krempa
2019-04-01 16:22 ` Markus Armbruster
2019-04-02 8:50 ` Kevin Wolf
2019-04-01 20:02 ` Markus Armbruster
2019-04-02 8:19 ` Kevin Wolf
2019-04-02 8:42 ` Peter Krempa
2019-04-02 9:09 ` Markus Armbruster
2019-04-05 10:32 ` [Qemu-devel] [PATCH for-4.0 0/2] file-posix: query-qemu-features for auto-read-only Stefan Hajnoczi
2019-04-05 10:32 ` Stefan Hajnoczi
2019-04-05 12:46 ` Kevin Wolf [this message]
2019-04-05 12:46 ` Kevin Wolf
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190405124608.GA4597@linux.fritz.box \
--to=kwolf@redhat.com \
--cc=armbru@redhat.com \
--cc=pkrempa@redhat.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.