Re: [PATCH] x86/ioperm: add new paravirt function update_io_bitmap

[JÃrgen GroÃ]

On 18.02.20 22:03, Thomas Gleixner wrote:
> Juergen Gross <jgross@xxxxxxxx> writes:
> > Commit 111e7b15cf10f6 ("x86/ioperm: Extend IOPL config to control
> > ioperm() as well") reworked the iopl syscall to use I/O bitmaps.
> >
> > Unfortunately this broke Xen PV domains using that syscall as there
> > is currently no I/O bitmap support in PV domains.
> >
> > Add I/O bitmap support via a new paravirt function update_io_bitmap
> > which Xen PV domains can use to update their I/O bitmaps via a
> > hypercall.
> >
> > Fixes: 111e7b15cf10f6 ("x86/ioperm: Extend IOPL config to control ioperm() as well")
> > Reported-by: Jan Beulich <jbeulich@xxxxxxxx>
> > Cc: <stable@xxxxxxxxxxxxxxx> # 5.5
> > Signed-off-by: Juergen Gross <jgross@xxxxxxxx>
> > Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>
> > Tested-by: Jan Beulich <jbeulich@xxxxxxxx>
>
> Duh, sorry about that and thanks for fixing it.
>
> BTW, why isn't stuff like this not catched during next or at least
> before the final release? Is nothing running CI on upstream with all
> that XEN muck active?

This problem showed up by not being able to start the X server (probably
not the freshest one) in dom0 on a moderate aged AMD system.

Our CI tests tend do be more text console based for dom0.


Juergen