mbox

[PULL] rtl8139,pci,msi

Message ID 20120607163830.GA13873@redhat.com
State New
Headers show

Pull-request

git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_anthony

Message

Michael S. Tsirkin June 7, 2012, 4:38 p.m. UTC
The following changes since commit 8cc9b43f7c5f826b39af4b012ad89bb55faac29c:

  target-microblaze: lwx/swx: first implementation (2012-06-04 10:19:46 +0200)

are available in the git repository at:

  git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_anthony

for you to fetch changes up to 11e02c393109508fde4d99650f003dd2d6029bc5:

  pci_bridge_dev: fix error path in pci_bridge_dev_initfn() (2012-06-07 17:19:01 +0300)

----------------------------------------------------------------
rtl8139,pci,msi

This pull includes a bugfix for rtl8139 (this is independent of
Jason Wang's fix which Anthony has merged) and pci bridge hotplug bugfixes.
Further there are MSI/MSIX fixes and changes by Jan which should
also help merging device assignment down the road.

Signed-off-by: Michael S. Tsirkin <mst@redhat.com>

----------------------------------------------------------------
Fernando Luis Vazquez Cao (1):
      rtl8139: honor RxOverflow flag in can_receive method

Jan Kiszka (9):
      ahci: Fix reset of MSI function
      intel-hda: Fix reset of MSI function
      ahci: Clean up reset functions
      msi: Guard msi_reset with msi_present
      msi: Invoke msi/msix_reset from PCI core
      msi: Guard msi/msix_write_config with msi_present
      msi: Invoke msi/msix_write_config from PCI core
      msi: Use msi/msix_present more consistently
      msix: Drop unused msix_bar_size

Jason Baron (2):
      qdev: release parent properties on dc->init failure
      pci_bridge_dev: fix error path in pci_bridge_dev_initfn()

Michael S. Tsirkin (1):
      shpc: unparent device before free

 hw/ide/ahci.c           | 25 +++++++++++++++----------
 hw/ide/ahci.h           |  2 +-
 hw/ide/ich.c            | 19 ++++++++-----------
 hw/intel-hda.c          | 12 ------------
 hw/ioh3420.c            |  3 +--
 hw/ivshmem.c            |  1 -
 hw/msi.c                | 11 ++++++++---
 hw/msix.c               | 23 +++++++++--------------
 hw/msix.h               |  2 --
 hw/pci.c                |  8 ++++++++
 hw/pci.h                |  2 --
 hw/pci_bridge.c         | 10 ++--------
 hw/pci_bridge_dev.c     |  8 ++++----
 hw/qdev.c               |  1 +
 hw/rtl8139.c            |  2 +-
 hw/shpc.c               |  1 +
 hw/virtio-pci.c         |  3 ---
 hw/xio3130_downstream.c |  3 +--
 hw/xio3130_upstream.c   |  3 +--
 19 files changed, 61 insertions(+), 78 deletions(-)

Comments

Anthony Liguori June 11, 2012, 5:16 p.m. UTC | #1
On 06/07/2012 11:38 AM, Michael S. Tsirkin wrote:
> The following changes since commit 8cc9b43f7c5f826b39af4b012ad89bb55faac29c:
>
>    target-microblaze: lwx/swx: first implementation (2012-06-04 10:19:46 +0200)
>
> are available in the git repository at:
>
>    git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_anthony
>
> for you to fetch changes up to 11e02c393109508fde4d99650f003dd2d6029bc5:
>
>    pci_bridge_dev: fix error path in pci_bridge_dev_initfn() (2012-06-07 17:19:01 +0300)
>
> ----------------------------------------------------------------
> rtl8139,pci,msi
>
> This pull includes a bugfix for rtl8139 (this is independent of
> Jason Wang's fix which Anthony has merged) and pci bridge hotplug bugfixes.
> Further there are MSI/MSIX fixes and changes by Jan which should
> also help merging device assignment down the road.
>
> Signed-off-by: Michael S. Tsirkin<mst@redhat.com>
>
> ----------------------------------------------------------------
> Fernando Luis Vazquez Cao (1):
>        rtl8139: honor RxOverflow flag in can_receive method
>
> Jan Kiszka (9):
>        ahci: Fix reset of MSI function
>        intel-hda: Fix reset of MSI function
>        ahci: Clean up reset functions
>        msi: Guard msi_reset with msi_present
>        msi: Invoke msi/msix_reset from PCI core
>        msi: Guard msi/msix_write_config with msi_present
>        msi: Invoke msi/msix_write_config from PCI core
>        msi: Use msi/msix_present more consistently
>        msix: Drop unused msix_bar_size

This last commit made GCC unhappy.

   CC    libhw64/hw/msix.o
/home/anthony/git/qemu/hw/msix.c: In function ‘msix_add_config’:
/home/anthony/git/qemu/hw/msix.c:58:14: error: variable ‘new_size’ set but not 
used [-Werror=unused-but-set-variable]
cc1: all warnings being treated as errors

Regards,

Anthony Liguori

>
> Jason Baron (2):
>        qdev: release parent properties on dc->init failure
>        pci_bridge_dev: fix error path in pci_bridge_dev_initfn()
>
> Michael S. Tsirkin (1):
>        shpc: unparent device before free
>
>   hw/ide/ahci.c           | 25 +++++++++++++++----------
>   hw/ide/ahci.h           |  2 +-
>   hw/ide/ich.c            | 19 ++++++++-----------
>   hw/intel-hda.c          | 12 ------------
>   hw/ioh3420.c            |  3 +--
>   hw/ivshmem.c            |  1 -
>   hw/msi.c                | 11 ++++++++---
>   hw/msix.c               | 23 +++++++++--------------
>   hw/msix.h               |  2 --
>   hw/pci.c                |  8 ++++++++
>   hw/pci.h                |  2 --
>   hw/pci_bridge.c         | 10 ++--------
>   hw/pci_bridge_dev.c     |  8 ++++----
>   hw/qdev.c               |  1 +
>   hw/rtl8139.c            |  2 +-
>   hw/shpc.c               |  1 +
>   hw/virtio-pci.c         |  3 ---
>   hw/xio3130_downstream.c |  3 +--
>   hw/xio3130_upstream.c   |  3 +--
>   19 files changed, 61 insertions(+), 78 deletions(-)
>
>
Jan Kiszka June 11, 2012, 5:22 p.m. UTC | #2
On 2012-06-11 19:16, Anthony Liguori wrote:
> On 06/07/2012 11:38 AM, Michael S. Tsirkin wrote:
>> The following changes since commit 8cc9b43f7c5f826b39af4b012ad89bb55faac29c:
>>
>>    target-microblaze: lwx/swx: first implementation (2012-06-04 10:19:46 +0200)
>>
>> are available in the git repository at:
>>
>>    git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_anthony
>>
>> for you to fetch changes up to 11e02c393109508fde4d99650f003dd2d6029bc5:
>>
>>    pci_bridge_dev: fix error path in pci_bridge_dev_initfn() (2012-06-07 17:19:01 +0300)
>>
>> ----------------------------------------------------------------
>> rtl8139,pci,msi
>>
>> This pull includes a bugfix for rtl8139 (this is independent of
>> Jason Wang's fix which Anthony has merged) and pci bridge hotplug bugfixes.
>> Further there are MSI/MSIX fixes and changes by Jan which should
>> also help merging device assignment down the road.
>>
>> Signed-off-by: Michael S. Tsirkin<mst@redhat.com>
>>
>> ----------------------------------------------------------------
>> Fernando Luis Vazquez Cao (1):
>>        rtl8139: honor RxOverflow flag in can_receive method
>>
>> Jan Kiszka (9):
>>        ahci: Fix reset of MSI function
>>        intel-hda: Fix reset of MSI function
>>        ahci: Clean up reset functions
>>        msi: Guard msi_reset with msi_present
>>        msi: Invoke msi/msix_reset from PCI core
>>        msi: Guard msi/msix_write_config with msi_present
>>        msi: Invoke msi/msix_write_config from PCI core
>>        msi: Use msi/msix_present more consistently
>>        msix: Drop unused msix_bar_size
> 
> This last commit made GCC unhappy.
> 
>    CC    libhw64/hw/msix.o
> /home/anthony/git/qemu/hw/msix.c: In function ‘msix_add_config’:
> /home/anthony/git/qemu/hw/msix.c:58:14: error: variable ‘new_size’ set but not 
> used [-Werror=unused-but-set-variable]
> cc1: all warnings being treated as errors

Grr, compiler too old to have this.

Please just drop, I'll post a better cleanup in another round.

Jan
Michael S. Tsirkin June 11, 2012, 8:01 p.m. UTC | #3
On Mon, Jun 11, 2012 at 07:22:53PM +0200, Jan Kiszka wrote:
> On 2012-06-11 19:16, Anthony Liguori wrote:
> > On 06/07/2012 11:38 AM, Michael S. Tsirkin wrote:
> >> The following changes since commit 8cc9b43f7c5f826b39af4b012ad89bb55faac29c:
> >>
> >>    target-microblaze: lwx/swx: first implementation (2012-06-04 10:19:46 +0200)
> >>
> >> are available in the git repository at:
> >>
> >>    git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_anthony
> >>
> >> for you to fetch changes up to 11e02c393109508fde4d99650f003dd2d6029bc5:
> >>
> >>    pci_bridge_dev: fix error path in pci_bridge_dev_initfn() (2012-06-07 17:19:01 +0300)
> >>
> >> ----------------------------------------------------------------
> >> rtl8139,pci,msi
> >>
> >> This pull includes a bugfix for rtl8139 (this is independent of
> >> Jason Wang's fix which Anthony has merged) and pci bridge hotplug bugfixes.
> >> Further there are MSI/MSIX fixes and changes by Jan which should
> >> also help merging device assignment down the road.
> >>
> >> Signed-off-by: Michael S. Tsirkin<mst@redhat.com>
> >>
> >> ----------------------------------------------------------------
> >> Fernando Luis Vazquez Cao (1):
> >>        rtl8139: honor RxOverflow flag in can_receive method
> >>
> >> Jan Kiszka (9):
> >>        ahci: Fix reset of MSI function
> >>        intel-hda: Fix reset of MSI function
> >>        ahci: Clean up reset functions
> >>        msi: Guard msi_reset with msi_present
> >>        msi: Invoke msi/msix_reset from PCI core
> >>        msi: Guard msi/msix_write_config with msi_present
> >>        msi: Invoke msi/msix_write_config from PCI core
> >>        msi: Use msi/msix_present more consistently
> >>        msix: Drop unused msix_bar_size
> > 
> > This last commit made GCC unhappy.
> > 
> >    CC    libhw64/hw/msix.o
> > /home/anthony/git/qemu/hw/msix.c: In function ‘msix_add_config’:
> > /home/anthony/git/qemu/hw/msix.c:58:14: error: variable ‘new_size’ set but not 
> > used [-Werror=unused-but-set-variable]
> > cc1: all warnings being treated as errors
> 
> Grr, compiler too old to have this.
> 
> Please just drop, I'll post a better cleanup in another round.
> 
> Jan

I pushed a fixed for_anthony tag but didn't have time to test yet.

> -- 
> Siemens AG, Corporate Technology, CT T DE IT 1
> Corporate Competence Center Embedded Linux