* [virtio-comment] [PATCH] content: Minor change to clarify free_page_hint_cmd_id
@ 2020-11-12 22:57 Alexander Duyck
2020-11-13 7:24 ` [virtio-comment] " Michael S. Tsirkin
2020-11-13 10:16 ` Cornelia Huck
0 siblings, 2 replies; 3+ messages in thread
From: Alexander Duyck @ 2020-11-12 22:57 UTC (permalink / raw)
To: mst, david, cohuck; +Cc: virtio-comment, virtio-dev, wei.w.wang
From: Alexander Duyck <alexander.h.duyck@linux.intel.com>
The original wording was a bit unclear and could have been misinterpreted
as indicating that VIRTIO_BALLOON_FREE_PAGE_HINT was read-only instead of
the field free_page_hint_cmd_id. To clarify that break it up into two
sentences making it clear that the field is only available if the feature
is negotiated, and that the field is read-only.
Reviewed-by: Cornelia Huck <cohuck@redhat.com>
Signed-off-by: Alexander Duyck <alexander.h.duyck@linux.intel.com>
---
content.tex | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/content.tex b/content.tex
index 21e0cee903b0..9c39831acc15 100644
--- a/content.tex
+++ b/content.tex
@@ -5206,8 +5206,8 @@ \subsection{Device configuration layout}\label{sec:Device Types / Memory Balloon
\field{num_pages} and \field{actual} are always available.
\field{free_page_hint_cmd_id} is available if
- VIRTIO_BALLOON_F_FREE_PAGE_HINT has been negotiated and is read-only by
- the driver.
+ VIRTIO_BALLOON_F_FREE_PAGE_HINT has been negotiated. The field is
+ read-only by the driver.
\field{poison_val} is available if VIRTIO_BALLOON_F_PAGE_POISON has been
negotiated.
This publicly archived list offers a means to provide input to the
OASIS Virtual I/O Device (VIRTIO) TC.
In order to verify user consent to the Feedback License terms and
to minimize spam in the list archive, subscription is required
before posting.
Subscribe: virtio-comment-subscribe@lists.oasis-open.org
Unsubscribe: virtio-comment-unsubscribe@lists.oasis-open.org
List help: virtio-comment-help@lists.oasis-open.org
List archive: https://lists.oasis-open.org/archives/virtio-comment/
Feedback License: https://www.oasis-open.org/who/ipr/feedback_license.pdf
List Guidelines: https://www.oasis-open.org/policies-guidelines/mailing-lists
Committee: https://www.oasis-open.org/committees/virtio/
Join OASIS: https://www.oasis-open.org/join/
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [virtio-comment] Re: [PATCH] content: Minor change to clarify free_page_hint_cmd_id
2020-11-12 22:57 [virtio-comment] [PATCH] content: Minor change to clarify free_page_hint_cmd_id Alexander Duyck
@ 2020-11-13 7:24 ` Michael S. Tsirkin
2020-11-13 10:16 ` Cornelia Huck
1 sibling, 0 replies; 3+ messages in thread
From: Michael S. Tsirkin @ 2020-11-13 7:24 UTC (permalink / raw)
To: Alexander Duyck; +Cc: david, cohuck, virtio-comment, virtio-dev, wei.w.wang
On Thu, Nov 12, 2020 at 02:57:50PM -0800, Alexander Duyck wrote:
> From: Alexander Duyck <alexander.h.duyck@linux.intel.com>
>
> The original wording was a bit unclear and could have been misinterpreted
> as indicating that VIRTIO_BALLOON_FREE_PAGE_HINT was read-only instead of
> the field free_page_hint_cmd_id. To clarify that break it up into two
> sentences making it clear that the field is only available if the feature
> is negotiated, and that the field is read-only.
>
> Reviewed-by: Cornelia Huck <cohuck@redhat.com>
> Signed-off-by: Alexander Duyck <alexander.h.duyck@linux.intel.com>
Acked-by: Michael S. Tsirkin <mst@redhat.com>
> ---
> content.tex | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/content.tex b/content.tex
> index 21e0cee903b0..9c39831acc15 100644
> --- a/content.tex
> +++ b/content.tex
> @@ -5206,8 +5206,8 @@ \subsection{Device configuration layout}\label{sec:Device Types / Memory Balloon
> \field{num_pages} and \field{actual} are always available.
>
> \field{free_page_hint_cmd_id} is available if
> - VIRTIO_BALLOON_F_FREE_PAGE_HINT has been negotiated and is read-only by
> - the driver.
> + VIRTIO_BALLOON_F_FREE_PAGE_HINT has been negotiated. The field is
> + read-only by the driver.
> \field{poison_val} is available if VIRTIO_BALLOON_F_PAGE_POISON has been
> negotiated.
>
>
This publicly archived list offers a means to provide input to the
OASIS Virtual I/O Device (VIRTIO) TC.
In order to verify user consent to the Feedback License terms and
to minimize spam in the list archive, subscription is required
before posting.
Subscribe: virtio-comment-subscribe@lists.oasis-open.org
Unsubscribe: virtio-comment-unsubscribe@lists.oasis-open.org
List help: virtio-comment-help@lists.oasis-open.org
List archive: https://lists.oasis-open.org/archives/virtio-comment/
Feedback License: https://www.oasis-open.org/who/ipr/feedback_license.pdf
List Guidelines: https://www.oasis-open.org/policies-guidelines/mailing-lists
Committee: https://www.oasis-open.org/committees/virtio/
Join OASIS: https://www.oasis-open.org/join/
^ permalink raw reply [flat|nested] 3+ messages in thread
* [virtio-comment] Re: [PATCH] content: Minor change to clarify free_page_hint_cmd_id
2020-11-12 22:57 [virtio-comment] [PATCH] content: Minor change to clarify free_page_hint_cmd_id Alexander Duyck
2020-11-13 7:24 ` [virtio-comment] " Michael S. Tsirkin
@ 2020-11-13 10:16 ` Cornelia Huck
1 sibling, 0 replies; 3+ messages in thread
From: Cornelia Huck @ 2020-11-13 10:16 UTC (permalink / raw)
To: Alexander Duyck; +Cc: mst, david, virtio-comment, virtio-dev, wei.w.wang
On Thu, 12 Nov 2020 14:57:50 -0800
Alexander Duyck <alexander.duyck@gmail.com> wrote:
> From: Alexander Duyck <alexander.h.duyck@linux.intel.com>
>
> The original wording was a bit unclear and could have been misinterpreted
> as indicating that VIRTIO_BALLOON_FREE_PAGE_HINT was read-only instead of
> the field free_page_hint_cmd_id. To clarify that break it up into two
> sentences making it clear that the field is only available if the feature
> is negotiated, and that the field is read-only.
>
> Reviewed-by: Cornelia Huck <cohuck@redhat.com>
> Signed-off-by: Alexander Duyck <alexander.h.duyck@linux.intel.com>
> ---
> content.tex | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
Thanks, pushed under the minor cleanups rule.
This publicly archived list offers a means to provide input to the
OASIS Virtual I/O Device (VIRTIO) TC.
In order to verify user consent to the Feedback License terms and
to minimize spam in the list archive, subscription is required
before posting.
Subscribe: virtio-comment-subscribe@lists.oasis-open.org
Unsubscribe: virtio-comment-unsubscribe@lists.oasis-open.org
List help: virtio-comment-help@lists.oasis-open.org
List archive: https://lists.oasis-open.org/archives/virtio-comment/
Feedback License: https://www.oasis-open.org/who/ipr/feedback_license.pdf
List Guidelines: https://www.oasis-open.org/policies-guidelines/mailing-lists
Committee: https://www.oasis-open.org/committees/virtio/
Join OASIS: https://www.oasis-open.org/join/
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-11-13 10:16 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-12 22:57 [virtio-comment] [PATCH] content: Minor change to clarify free_page_hint_cmd_id Alexander Duyck
2020-11-13 7:24 ` [virtio-comment] " Michael S. Tsirkin
2020-11-13 10:16 ` Cornelia Huck
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.