Hi,

> On Oct 19, 2023, at 02:00, Julien Grall <jul...@xen.org> wrote:
> 
> Hi Jan,
> 
> On 18/10/2023 15:58, Jan Beulich wrote:
>> From: Manuel Bouyer <bou...@antioche.eu.org>
>> The use of rcu_lock_domain_by_id() right in switch_serial_input() makes
>> assumptions about domain IDs which don't hold when in shim mode: The
>> sole (initial) domain there has a non-zero ID. Obtain the real domain ID
>> in that case (generalized as get_initial_domain_id() returns zero when
>> not in shim mode).
>> Note that console_input_domain() isn't altered, for not being used when
>> in shim mode (or more generally on x86).
> 
> I think it would be worth to either add a comment in console_input_domain() 
> and/or #ifdef the code. In any case...
> 
>> Fixes: c2581c58bec9 ("xen/console: skip switching serial input to non 
>> existing domains")
>> Signed-off-by: Manuel Bouyer <bou...@antioche.eu.org>
>> Signed-off-by: Jan Beulich <jbeul...@suse.com>
> 
> ...
> 
> Reviewed-by: Julien Grall <jgr...@amazon.com>
> 
> 
> Also, should we consider it for xen 4.18? (I notice there is no for-4.18 tag).

Yes, so:

Release-acked-by: Henry Wang <henry.w...@arm.com>

Kind regards,
Henry


> 
> Cheers,
> 
> -- 
> Julien Grall


Reply via email to