Some small cleanups and corrections to PCI-E handling

[Qemu-devel] [PATCH 0/2] Some small cleanups and corrections to PCI-E handling

David Gibson posted 2 patches 6 years, 8 months ago

Download series mbox

Test docker-clang@ubuntu failed

Test asan failed

Test docker-mingw@fedora passed

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190214050808.16653-1-david@gibson.dropbear.id.au

Maintainers: "Michael S. Tsirkin" <mst@redhat.com>, Marcel Apfelbaum <marcel.apfelbaum@gmail.com>

hw/pci-bridge/pci_expander_bridge.c |  6 -----
hw/pci/pci.c                        | 40 ++++++++++++++++++++++-------
hw/virtio/virtio-pci.c              |  1 +
include/hw/pci/pci.h                |  5 ++--
include/hw/pci/pci_bus.h            | 13 ++++++++++
5 files changed, 48 insertions(+), 17 deletions(-)

Expand all Fold all

[Qemu-devel] [PATCH 0/2] Some small cleanups and corrections to PCI-E handling

Posted by David Gibson 6 years, 8 months ago

Here are a couple of cleanups for PCI-E handling which arose out of
some other work I have in progress.  At least the first of these
patches has been posted before as part of a different series, but
wasn't ready to go at the time.

David Gibson (2):
  pci: Simplify pci_bus_is_root()
  pcie: Don't allow extended config space access via conventional PCI
    bridges

 hw/pci-bridge/pci_expander_bridge.c |  6 -----
 hw/pci/pci.c                        | 40 ++++++++++++++++++++++-------
 hw/virtio/virtio-pci.c              |  1 +
 include/hw/pci/pci.h                |  5 ++--
 include/hw/pci/pci_bus.h            | 13 ++++++++++
 5 files changed, 48 insertions(+), 17 deletions(-)

-- 
2.20.1

Re: [Qemu-devel] [PATCH 0/2] Some small cleanups and corrections to PCI-E handling

Posted by David Gibson 6 years, 8 months ago

On Thu, Feb 14, 2019 at 04:08:06PM +1100, David Gibson wrote:
> Here are a couple of cleanups for PCI-E handling which arose out of
> some other work I have in progress.  At least the first of these
> patches has been posted before as part of a different series, but
> wasn't ready to go at the time.

Self NACK.

Sorry, made a sloppy error in my testing and this breaks make check.
I'll repost once I've fixed it.

> 
> David Gibson (2):
>   pci: Simplify pci_bus_is_root()
>   pcie: Don't allow extended config space access via conventional PCI
>     bridges
> 
>  hw/pci-bridge/pci_expander_bridge.c |  6 -----
>  hw/pci/pci.c                        | 40 ++++++++++++++++++++++-------
>  hw/virtio/virtio-pci.c              |  1 +
>  include/hw/pci/pci.h                |  5 ++--
>  include/hw/pci/pci_bus.h            | 13 ++++++++++
>  5 files changed, 48 insertions(+), 17 deletions(-)
> 

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson