All of lore.kernel.org
 help / color / mirror / Atom feed
From: Julien Grall <julien@xen.org>
To: Jan Beulich <jbeulich@suse.com>,
	"xen-devel@lists.xenproject.org" <xen-devel@lists.xenproject.org>
Cc: Andrew Cooper <andrew.cooper3@citrix.com>,
	George Dunlap <George.Dunlap@eu.citrix.com>,
	Ian Jackson <iwj@xenproject.org>, Wei Liu <wl@xen.org>,
	Stefano Stabellini <sstabellini@kernel.org>
Subject: Re: [PATCH 01/12] evtchn: refuse EVTCHNOP_status for Xen-bound event channels
Date: Tue, 29 Sep 2020 09:49:19 +0100	[thread overview]
Message-ID: <035484d6-4472-c8ec-bc65-756b94ff98b4@xen.org> (raw)
In-Reply-To: <e7331fa6-e557-4319-6137-2c2525f78822@suse.com>

Hi Jan,

On 28/09/2020 11:56, Jan Beulich wrote:
> Callers have no business knowing the state of the Xen end of an event
> channel.
> 
> Signed-off-by: Jan Beulich <jbeulich@suse.com>

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

Cheers,

> 
> --- a/xen/common/event_channel.c
> +++ b/xen/common/event_channel.c
> @@ -933,6 +933,11 @@ int evtchn_status(evtchn_status_t *statu
>       }
>   
>       chn = evtchn_from_port(d, port);
> +    if ( consumer_is_xen(chn) )
> +    {
> +        rc = -EACCES;
> +        goto out;
> +    }
>   
>       rc = xsm_evtchn_status(XSM_TARGET, d, chn);
>       if ( rc )
> 

-- 
Julien Grall


  reply	other threads:[~2020-09-29  8:49 UTC|newest]

Thread overview: 60+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-09-28 10:54 [PATCH 00/12] evtchn: recent XSAs follow-on Jan Beulich
2020-09-28 10:56 ` [PATCH 01/12] evtchn: refuse EVTCHNOP_status for Xen-bound event channels Jan Beulich
2020-09-29  8:49   ` Julien Grall [this message]
2020-09-29 15:39   ` Paul Durrant
2020-09-28 10:56 ` [PATCH 02/12] evtchn: avoid race in get_xen_consumer() Jan Beulich
2020-09-29  9:35   ` Julien Grall
2020-09-29 15:44   ` Paul Durrant
2020-09-29 15:58     ` Jan Beulich
2020-09-28 10:57 ` [PATCH 03/12] evtchn: don't call Xen consumer callback with per-channel lock held Jan Beulich
2020-09-29 10:16   ` Julien Grall
2020-09-29 10:54     ` Jan Beulich
2020-09-28 10:57 ` [PATCH 04/12] evtchn: evtchn_set_priority() needs to acquire the per-channel lock Jan Beulich
2020-09-29 10:21   ` Julien Grall
2020-09-29 11:49     ` Jan Beulich
2020-09-29 16:31   ` Paul Durrant
2020-09-30  6:29     ` Jan Beulich
2020-09-30  6:41     ` Jan Beulich
2020-09-30  7:31   ` Paul Durrant
2020-09-30  8:31     ` Jan Beulich
2020-09-30  8:36       ` Paul Durrant
2020-09-30  8:41         ` Jan Beulich
2020-09-28 10:58 ` [PATCH 05/12] evtchn/sched: reject poll requests for unusable ports Jan Beulich
2020-09-29 12:17   ` Julien Grall
2020-09-29 13:00   ` Dario Faggioli
2020-09-29 16:51   ` Paul Durrant
2020-09-28 10:59 ` [PATCH 06/12] evtchn: don't bypass unlinking pIRQ when closing port Jan Beulich
2020-09-29 17:07   ` Paul Durrant
2020-09-28 11:00 ` [PATCH 07/12] evtchn: cut short evtchn_reset()'s loop in the common case Jan Beulich
2020-09-29 17:16   ` Paul Durrant
2020-10-01 15:54   ` Julien Grall
2020-09-28 11:00 ` [PATCH 08/12] evtchn: ECS_CLOSED => ECS_FREE Jan Beulich
2020-09-29 12:19   ` Julien Grall
2020-09-29 16:56   ` Paul Durrant
2020-09-28 11:00 ` [PATCH 09/12] evtchn: move FIFO-private struct declarations Jan Beulich
2020-09-29 12:26   ` Julien Grall
2020-09-29 12:49     ` Jan Beulich
2020-09-30  8:43       ` Julien Grall
2020-09-30  7:37   ` Paul Durrant
2020-09-30  8:32     ` Jan Beulich
2020-09-28 11:01 ` [PATCH 10/12] evtchn/fifo: use stable fields when recording "last queue" information Jan Beulich
2020-09-29 12:29   ` Julien Grall
2020-09-30  7:35   ` Paul Durrant
2020-09-30  8:35     ` Jan Beulich
2020-09-30  8:38       ` Paul Durrant
2020-09-28 11:02 ` [PATCH 11/12] evtchn: convert vIRQ lock to an r/w one Jan Beulich
2020-09-29 13:03   ` Julien Grall
2020-09-29 13:37     ` Jan Beulich
2020-09-29 17:18       ` Julien Grall
2020-09-30  6:26         ` Jan Beulich
2020-09-30  9:09           ` Julien Grall
2020-09-30  7:58   ` Paul Durrant
2020-09-30  8:37     ` Jan Beulich
2020-09-30  8:52       ` Paul Durrant
2020-09-30 10:16         ` Jan Beulich
2020-10-01 16:21           ` Julien Grall
2020-10-02  6:12             ` Jan Beulich
2020-10-02  8:43               ` Julien Grall
2020-09-28 11:02 ` [PATCH 12/12] evtchn: convert domain event " Jan Beulich
2020-09-28 16:44   ` Roger Pau Monné
2020-09-29 14:32     ` Jan Beulich

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=035484d6-4472-c8ec-bc65-756b94ff98b4@xen.org \
    --to=julien@xen.org \
    --cc=George.Dunlap@eu.citrix.com \
    --cc=andrew.cooper3@citrix.com \
    --cc=iwj@xenproject.org \
    --cc=jbeulich@suse.com \
    --cc=sstabellini@kernel.org \
    --cc=wl@xen.org \
    --cc=xen-devel@lists.xenproject.org \
    /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.