On Wed, Nov 04, 2020 at 07:58:07PM +0000, Ben Hutchings wrote: > From: Juergen Gross <jgross@xxxxxxxx> > > commit 0891fb39ba67bd7ae023ea0d367297ffff010781 upstream. > > Since commit c330fb1ddc0a ("XEN uses irqdesc::irq_data_common::handler_data to store a per interrupt XEN data pointer which contains XEN specific information.") > Xen is using the chip_data pointer for storing IRQ specific data. When > running as a HVM domain this can result in problems for legacy IRQs, as > those might use chip_data for their own purposes. > > Use a local array for this purpose in case of legacy IRQs, avoiding the > double use. > > Cc: stable@xxxxxxxxxxxxxxx > Fixes: c330fb1ddc0a ("XEN uses irqdesc::irq_data_common::handler_data to store a per interrupt XEN data pointer which contains XEN specific information.") > Signed-off-by: Juergen Gross <jgross@xxxxxxxx> > Tested-by: Stefan Bader <stefan.bader@xxxxxxxxxxxxx> > Reviewed-by: Boris Ostrovsky <boris.ostrovsky@xxxxxxxxxx> > Link: https://lore.kernel.org/r/20200930091614.13660-1-jgross@xxxxxxxx > Signed-off-by: Juergen Gross <jgross@xxxxxxxx> > [bwh: Backported to 4.9: adjust context] > Signed-off-by: Ben Hutchings <benh@xxxxxxxxxx> > --- > drivers/xen/events/events_base.c | 29 +++++++++++++++++++++-------- > 1 file changed, 21 insertions(+), 8 deletions(-) This was in the 4.19.155 release, but I have added this to the other kernels now too. thanks, greg k-h