qemu-devel
[Top][All Lists]
Advanced

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

Re: [PULL 00/19] virtio, vhost, acpi: features, fixes, tests


From: Michael S. Tsirkin
Subject: Re: [PULL 00/19] virtio, vhost, acpi: features, fixes, tests
Date: Tue, 15 Oct 2019 17:03:25 -0400

On Mon, Oct 07, 2019 at 05:31:32PM +0100, Peter Maydell wrote:
> On Sat, 5 Oct 2019 at 22:58, Michael S. Tsirkin <address@hidden> wrote:
> >
> > A couple of tweaks to the bios table test weren't
> > reviewed yet but as they are really helpful to
> > the arm tests I'm pushing, and affecting only the
> > test so fairly benign (dropped assert + a comment),
> > I cut a corner and pushed them straight away.
> > Will be easy to tweak with a patch on top or revert.
> >
> > The following changes since commit 4f59102571fce49af180cfc6d4cdd2b5df7bdb14:
> >
> >   Merge remote-tracking branch 
> > 'remotes/amarkovic/tags/mips-queue-oct-01-2019' into staging (2019-10-01 
> > 16:21:42 +0100)
> >
> > are available in the Git repository at:
> >
> >   git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_upstream
> >
> > for you to fetch changes up to 9d59bed1da5e5815987879346cf708344060ea63:
> >
> >   virtio: add vhost-user-fs-pci device (2019-10-05 17:43:55 -0400)
> >
> > ----------------------------------------------------------------
> > virtio, vhost, acpi: features, fixes, tests
> >
> > Tests for arm/virt ACPI tables.
> > Virtio fs support (no migration).
> > A vhost-user reconnect bugfix.
> 
> Hi -- this summary of the changes just says it has
> updates to the test code, but the patches include:
> 
> > Shameer Kolothum (8):
> >       hw/arm/virt: Enable device memory cold/hot plug with ACPI boot
> 
> which is to say actually enabling the hotplug memory feature,
> not just tweaking tests. Did you mean to put the whole feature
> in this pullreq? (If you did, that's fine, it just doesn't
> match up with the cover letter so maybe it was an accident?)
> 
> thanks
> -- PMM

Yes this was intentional. I meant I was asked to merge it because of
the tests. But tests wouldn't pass without the feature :)
I've updated the description. Pls pull.

-- 
MST




reply via email to

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