qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCHv3 0/2] pci: allow PCI bus slots to be marked as


From: Marcel Apfelbaum
Subject: Re: [Qemu-devel] [PATCHv3 0/2] pci: allow PCI bus slots to be marked as reserved
Date: Mon, 4 Sep 2017 12:50:31 +0300
User-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:52.0) Gecko/20100101 Thunderbird/52.1.1

On 31/08/2017 9:54, Mark Cave-Ayland wrote:
On 16/07/17 21:27, Mark Cave-Ayland wrote:

For some machines it is impossible to plug devices into a particular PCI bus
slot, e.g. for a real Ultra 5 there are 2 PCI bridges attached to the root
bus behind which all devices must be plugged. Ignoring this rule will cause
problems with interrupt routing since the interrupt numbers are calculated
based upon PCI bridge id and secondary PCI bus slot id.

This patchset adds a new slot_reserved_mask property to PCIBus which is a
bitmask used to indicate whether PCI bus slots are reserved, i.e. they cannot
be used for hot or cold plugging on a particular PCI bus.

Signed-off-by: Mark Cave-Ayland <address@hidden>

v3:
- Rebase onto master
- Simplify pci_bus_devfn_available() as suggested by Marcel
- Also simplify pci_bus_devfn_reserved() in a similar manner

v2:
- Rename dev_reserved_mask to slot_reserved_mask as suggested by Marcel
- Squash patches 2 and 3 together


Mark Cave-Ayland (2):
   pci: move check for existing devfn into new pci_bus_devfn_available()
     helper
   pci: add reserved slot check to do_pci_register_device()

  hw/pci/pci.c             |   26 ++++++++++++++++++++++----
  include/hw/pci/pci_bus.h |    1 +
  2 files changed, 23 insertions(+), 4 deletions(-)

Ping? Is there any chance to get this in soon, as I ended up having to
drop a couple of sun4u patch series for 2.10 that were dependent upon
this :(

I have R-B tags from Marcel, however my understanding is that this patch
still needs review from Michael?


Hi Mark,
I think Michael did not start yet to pick patches for 2.11 .

Thanks,
Marcel

Here is the link to the patch that depends upon this:
https://lists.nongnu.org/archive/html/qemu-devel/2017-07/msg03041.html


Many thanks,

Mark.





reply via email to

[Prev in Thread] Current Thread [Next in Thread]