On 01.04.2021 11:15, Roger Pau Monné wrote: > On Wed, Mar 31, 2021 at 06:04:43PM +0200, Jan Beulich wrote: >> On 31.03.2021 12:32, Roger Pau Monne wrote: >>> EOIs are always executed in guest vCPU context, so there's no reason to >>> pass a domain parameter around as can be fetched from current->domain. >>> >>> No functional change intended. >>> >>> Signed-off-by: Roger Pau Monné <roger....@citrix.com> >>> Reviewed-by: Paul Durrant <p...@xen.org> >>> --- >>> Changes since v1: >>> - New in this version. >> >> Just to make it explicit - possibly same thing as with patch 1, >> depending on how exactly the issue there gets taken care of. > > I don't think we have any scenario ATM where we allow EOI'ing of > interrupts from a different vCPU context, neither I can see us > allowing such in the future, but I don't want this discussion to block > the series, hence I'm going to drop this patch together with patch 1.
Well, having seen also your reply to Andrew wrt patch 1, I just wanted to clarify that dropping the two patches isn't the only option. Making Viridian code resilient in this regard might be another one. Jan
