diff mbox series

[08/12] evtchn: ECS_CLOSED => ECS_FREE

Message ID eeb0f674-1eb5-85a2-70b2-42041588e3d9@suse.com (mailing list archive)
State Superseded
Headers show
Series evtchn: recent XSAs follow-on | expand

Commit Message

Jan Beulich Sept. 28, 2020, 11 a.m. UTC 
There's no ECS_CLOSED; correct a comment naming it.

Signed-off-by: Jan Beulich <jbeulich@suse.com>

Comments

Julien Grall Sept. 29, 2020, 12:19 p.m. UTC | #1 
Hi,

On 28/09/2020 12:00, Jan Beulich wrote:
> There's no ECS_CLOSED; correct a comment naming it.
> Signed-off-by: Jan Beulich <jbeulich@suse.com>

Acked-by: Julien Grall <jgrall@amazon.com>

Cheers,
Paul Durrant Sept. 29, 2020, 4:56 p.m. UTC | #2 
> -----Original Message-----
> From: Xen-devel <xen-devel-bounces@lists.xenproject.org> On Behalf Of Jan Beulich
> Sent: 28 September 2020 12:00
> To: xen-devel@lists.xenproject.org
> Cc: Andrew Cooper <andrew.cooper3@citrix.com>; George Dunlap <George.Dunlap@eu.citrix.com>; Ian
> Jackson <iwj@xenproject.org>; Julien Grall <julien@xen.org>; Wei Liu <wl@xen.org>; Stefano Stabellini
> <sstabellini@kernel.org>
> Subject: [PATCH 08/12] evtchn: ECS_CLOSED => ECS_FREE
> 
> There's no ECS_CLOSED; correct a comment naming it.
> 
> Signed-off-by: Jan Beulich <jbeulich@suse.com>
> 

Reviewed-by: Paul Durrant <paul@xen.org>

> --- a/xen/common/event_channel.c
> +++ b/xen/common/event_channel.c
> @@ -673,7 +673,7 @@ int evtchn_close(struct domain *d1, int
>               * We can only get here if the port was closed and re-bound after
>               * unlocking d1 but before locking d2 above. We could retry but
>               * it is easier to return the same error as if we had seen the
> -             * port in ECS_CLOSED. It must have passed through that state for
> +             * port in ECS_FREE. It must have passed through that state for
>               * us to end up here, so it's a valid error to return.
>               */
>              rc = -EINVAL;
>
diff mbox series

Patch

--- a/xen/common/event_channel.c
+++ b/xen/common/event_channel.c
@@ -673,7 +673,7 @@  int evtchn_close(struct domain *d1, int
              * We can only get here if the port was closed and re-bound after
              * unlocking d1 but before locking d2 above. We could retry but
              * it is easier to return the same error as if we had seen the
-             * port in ECS_CLOSED. It must have passed through that state for
+             * port in ECS_FREE. It must have passed through that state for
              * us to end up here, so it's a valid error to return.
              */
             rc = -EINVAL;