[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL] pci,msi,virtio
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PULL] pci,msi,virtio |
Date: |
Sun, 29 Jul 2012 16:58:33 +0300 |
On Mon, Jul 23, 2012 at 11:05:25AM -0600, Alex Williamson wrote:
> On Mon, 2012-07-23 at 11:39 -0500, Anthony Liguori wrote:
> > Andreas Färber <address@hidden> writes:
> >
> > > Am 19.07.2012 17:15, schrieb Michael S. Tsirkin:
> > >> The following changes since commit
> > >> 80aa796bf38b7ef21daa42673b4711510c450d8a:
> > >>
> > >> pci_bridge_dev: fix error path in pci_bridge_dev_initfn() (2012-06-11
> > >> 22:55:13 +0300)
> > >>
> > >> 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 932d4a42afa28829fadf3cbfbb0507cc09aafd8b:
> > >>
> > >> msi/msix: added API to set MSI message address and data (2012-07-19
> > >> 17:56:42 +0300)
> > >>
> > >> ----------------------------------------------------------------
> > >> pci,msi,virtio
> > >>
> > >> This pull includes preparation patches mostly by Jan and Alex
> > >> that should help merge device assignment down the road.
> > >> And there's a new API needed for emulating POWER firmware.
> > >>
> > >> So no new functionality and some unused APIs but it looks like
> > >> merging will help people make progress.
> > >>
> > >> Signed-off-by: Michael S. Tsirkin <address@hidden>
> > >
> > > Usually, PULLs are expected to carry the individual patches as replies.
> > >
> > > But more important, did something go wrong with rebasing before sending
> > > out the PULL? June 11 is more than a month ago. And if I try to rebase
> > > my pci_host branch on your "pci" branch it tries to replay loads of
> > > really old post-1.1 commits (e.g., my "Pass PowerPCCPU to...") and
> > > fails... am I doing something wrong? "for_anthony" tag and "pci" branch
> > > seem to match in date at least.
> >
> >
> > It's a tag, not a branch.
> >
> > I had to add an explicit remote to pull tags in. I'm not sure if
> > there's a better way to do it in git.
>
> Looks like it merged ok for me, but you'll need this to complete the
> PCIUnregisterFunc conversion. Not sure how this built in MST's tree.
I didn't rebase my tree, this one is newer.
> Anthony, it'd be great if we could merge and fix this in your tree since
> MST is out of the office. Thanks,
>
> Alex
> commit 34e37573f1a983dab43673d664eb500bc87a46d4
> Author: Alex Williamson <address@hidden>
> Date: Mon Jul 23 10:59:43 2012 -0600
>
> esp: Update PCIUnregisterFunc function
>
> Function prototype changed in f90c2bcd
>
> Signed-off-by: Alex Williamson <address@hidden>
>
> diff --git a/hw/esp.c b/hw/esp.c
> index c6422ad..a011347 100644
> --- a/hw/esp.c
> +++ b/hw/esp.c
> @@ -1153,13 +1153,11 @@ static int esp_pci_scsi_init(PCIDevice *dev)
> return 0;
> }
>
> -static int esp_pci_scsi_uninit(PCIDevice *d)
> +static void esp_pci_scsi_uninit(PCIDevice *d)
> {
> PCIESPState *pci = DO_UPCAST(PCIESPState, dev, d);
>
> memory_region_destroy(&pci->io);
> -
> - return 0;
> }
>
> static void esp_pci_class_init(ObjectClass *klass, void *data)
>
- [Qemu-devel] [PULL] pci,msi,virtio, Michael S. Tsirkin, 2012/07/19
- Re: [Qemu-devel] [PULL] pci,msi,virtio, Andreas Färber, 2012/07/23
- Re: [Qemu-devel] [PULL] pci,msi,virtio, Anthony Liguori, 2012/07/23
- Re: [Qemu-devel] [PULL] pci,msi,virtio, Andreas Färber, 2012/07/23
- Re: [Qemu-devel] [PULL] pci,msi,virtio, Alex Williamson, 2012/07/23
- Re: [Qemu-devel] [PULL] pci,msi,virtio, Anthony Liguori, 2012/07/23
- Re: [Qemu-devel] [PULL] pci,msi,virtio, Alex Williamson, 2012/07/23
- Re: [Qemu-devel] [PULL] pci,msi,virtio, Andreas Färber, 2012/07/24
- Re: [Qemu-devel] [PULL] pci,msi,virtio, Alexey Kardashevskiy, 2012/07/24
- Re: [Qemu-devel] [PULL] pci,msi,virtio, Andreas Färber, 2012/07/25
- Re: [Qemu-devel] [PULL] pci,msi,virtio,
Michael S. Tsirkin <=
- Re: [Qemu-devel] [PULL] pci,msi,virtio, Michael S. Tsirkin, 2012/07/29
Re: [Qemu-devel] [PULL] pci,msi,virtio, Michael S. Tsirkin, 2012/07/29