mbox series

[v2,0/2] pci: assert that irqnum is between 0 and bus->nirqs in pci_change_irq_level()

Message ID 20201024203900.3619498-1-f4bug@amsat.org
Headers show
Series pci: assert that irqnum is between 0 and bus->nirqs in pci_change_irq_level() | expand

Message

Philippe Mathieu-Daudé Oct. 24, 2020, 8:38 p.m. UTC 
Respin of Mark's patch:
https://www.mail-archive.com/qemu-devel@nongnu.org/msg749459.html

Supersedes: <20201011082022.3016-1-mark.cave-ayland@ilande.co.uk>

Mark Cave-Ayland (1):
  pci: Assert irqnum is between 0 and bus->nirqs in
    pci_bus_change_irq_level

Philippe Mathieu-Daudé (1):
  hw/pci: Extract pci_bus_change_irq_level() from pci_change_irq_level()

 hw/pci/pci.c | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

-- 
2.26.2

Comments

Mark Cave-Ayland Oct. 25, 2020, 11:09 a.m. UTC | #1 
On 24/10/2020 21:38, Philippe Mathieu-Daudé wrote:

> Respin of Mark's patch:

> https://www.mail-archive.com/qemu-devel@nongnu.org/msg749459.html

> 

> Supersedes: <20201011082022.3016-1-mark.cave-ayland@ilande.co.uk>

> 

> Mark Cave-Ayland (1):

>    pci: Assert irqnum is between 0 and bus->nirqs in

>      pci_bus_change_irq_level

> 

> Philippe Mathieu-Daudé (1):

>    hw/pci: Extract pci_bus_change_irq_level() from pci_change_irq_level()

> 

>   hw/pci/pci.c | 11 +++++++++--

>   1 file changed, 9 insertions(+), 2 deletions(-)


This version LGTM - I think it's just waiting on review from the PCI folks to see 
whether they agree with the change.

Reviewed-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>



ATB,

Mark.