From: Tiwei Bie <tiwei.bie@intel.com>
To: Cornelia Huck <cohuck@redhat.com>
Cc: mst@redhat.com, stefanha@redhat.com, pbonzini@redhat.com,
virtio-dev@lists.oasis-open.org, dan.daly@intel.com,
cunming.liang@intel.com, zhihong.wang@intel.com
Subject: [virtio-dev] Re: [RFC] content: use a consistent wording for device status
Date: Wed, 13 Jun 2018 17:13:08 +0800 [thread overview]
Message-ID: <20180613091308.GA2692@debian> (raw)
In-Reply-To: <20180613110552.7a097717.cohuck@redhat.com>
On Wed, Jun 13, 2018 at 11:05:52AM +0200, Cornelia Huck wrote:
> On Tue, 12 Jun 2018 13:32:21 +0800
> Tiwei Bie <tiwei.bie@intel.com> wrote:
>
> > There is a part called "device status field" in a
> > Virtio device. Currently, both of \field{status}
> > and \field{device status} are used to refer to it,
> > which is a bit confusing. This patch changes spec
> > to use \field{device status} consistently.
> >
> > Suggested-by: Michael S. Tsirkin <mst@redhat.com>
> > Signed-off-by: Tiwei Bie <tiwei.bie@intel.com>
> > ---
> > If we want this change, I'll open an issue and
> > send a formal patch.
>
> Sounds good, given that there are some device-type specific 'status'
> fields, which makes things confusing.
>
> >
> > content.tex | 21 ++++++++++++---------
> > 1 file changed, 12 insertions(+), 9 deletions(-)
>
> Patch looks good.
>
> Reviewed-by: Cornelia Huck <cohuck@redhat.com>
Thank you very much for the review! :)
I'll open an issue and send a new patch (just
to drop the "RFC" from the subject) with your
Reviewed-by.
Best regards,
Tiwei Bie
---------------------------------------------------------------------
To unsubscribe, e-mail: virtio-dev-unsubscribe@lists.oasis-open.org
For additional commands, e-mail: virtio-dev-help@lists.oasis-open.org
prev parent reply other threads:[~2018-06-13 9:13 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-06-12 5:32 [virtio-dev] [RFC] content: use a consistent wording for device status Tiwei Bie
2018-06-13 9:05 ` [virtio-dev] " Cornelia Huck
2018-06-13 9:13 ` Tiwei Bie [this message]
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=20180613091308.GA2692@debian \
--to=tiwei.bie@intel.com \
--cc=cohuck@redhat.com \
--cc=cunming.liang@intel.com \
--cc=dan.daly@intel.com \
--cc=mst@redhat.com \
--cc=pbonzini@redhat.com \
--cc=stefanha@redhat.com \
--cc=virtio-dev@lists.oasis-open.org \
--cc=zhihong.wang@intel.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.