From: Jan Beulich <jbeulich@suse.com>
To: "Oleksii K." <oleksii.kurochko@gmail.com>
Cc: George Dunlap <George.Dunlap@citrix.com>,
Stefano Stabellini <sstabellini@kernel.org>,
Daniel Smith <dpsmith@apertussolutions.com>,
Xen-devel <xen-devel@lists.xenproject.org>,
"committers@xenproject.org" <committers@xenproject.org>,
Julien Grall <julien@xen.org>,
Andrew Cooper <andrew.cooper3@citrix.com>
Subject: Re: [PATCH] Revert "evtchn: refuse EVTCHNOP_status for Xen-bound event channels"
Date: Fri, 17 May 2024 09:01:33 +0200 [thread overview]
Message-ID: <afac644e-240f-47b6-be01-699b05e853dc@suse.com> (raw)
In-Reply-To: <66ac3d6aaf0e393ce76580f274c222d26c0aa0a1.camel@gmail.com>
On 16.05.2024 21:15, Oleksii K. wrote:
> I am not deeply familiar with the technical details surrounding XSM,
> but if I understand Daniel's point correctly, the original change
> violates the access control framework. This suggests to me that the
> revert should be merged.
>
> However, I have a question: if we merge this revert, does it mean that
> using channels a user ( domain ) will be able to get information about
> certain events such as EVTCHNSTAT_unbound, EVTCHNSTAT_interdomain,
> EVTCHNSTAT_pirq, EVTCHNSTAT_virq, and EVTCHNSTAT_IPI (based on the code
> of lseventch.c)? Is this information really so critical that it cannot
> be exposed for some time until a patch that changes the XSM policy
> consistently is provided and merged?
>
> If this information is indeed critical and should not be exposed, I
> think we can consider Daniel's suggestion to add a check to the dummy
> XSM policy as a solution.
The question isn't whether it's critical. As pointed out elsewhere, my
view is that any exposure of information needs to come with a proof that
nothing undue can be derived from that information. I see Daniel has
responded there, so we'll continue the discussion there.
Jan
next prev parent reply other threads:[~2024-05-17 7:01 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-04-02 17:06 [PATCH] Revert "evtchn: refuse EVTCHNOP_status for Xen-bound event channels" Andrew Cooper
2024-04-03 6:16 ` Jan Beulich
2024-04-03 6:52 ` Jan Beulich
2024-04-03 11:50 ` Daniel P. Smith
2024-04-03 11:54 ` Jan Beulich
2024-04-03 13:31 ` Daniel P. Smith
2024-04-04 8:11 ` Jan Beulich
2024-04-03 11:10 ` Daniel P. Smith
2024-04-03 12:05 ` Jan Beulich
2024-04-03 13:27 ` Daniel P. Smith
2024-04-04 7:57 ` Jan Beulich
2024-04-05 5:59 ` Jan Beulich
2024-05-14 9:25 ` Jan Beulich
2024-05-14 9:51 ` Andrew Cooper
2024-05-14 10:03 ` Jan Beulich
2024-05-14 11:13 ` Julien Grall
2024-05-14 21:35 ` Stefano Stabellini
2024-05-15 7:33 ` Jan Beulich
2024-05-16 19:15 ` Oleksii K.
2024-05-17 7:01 ` Jan Beulich [this message]
2024-05-15 10:49 ` Kelly Choi
2024-05-15 12:59 ` George Dunlap
2024-05-16 6:41 ` Jan Beulich
2024-05-17 1:21 ` Stefano Stabellini
2024-05-17 7:04 ` Jan Beulich
2024-05-17 20:28 ` Stefano Stabellini
2024-05-21 6:17 ` Jan Beulich
2024-05-22 1:33 ` Stefano Stabellini
2024-05-17 1:22 ` Daniel P. Smith
2024-05-17 7:24 ` Jan Beulich
2024-04-03 13:35 ` Daniel P. Smith
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=afac644e-240f-47b6-be01-699b05e853dc@suse.com \
--to=jbeulich@suse.com \
--cc=George.Dunlap@citrix.com \
--cc=andrew.cooper3@citrix.com \
--cc=committers@xenproject.org \
--cc=dpsmith@apertussolutions.com \
--cc=julien@xen.org \
--cc=oleksii.kurochko@gmail.com \
--cc=sstabellini@kernel.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.