Richard W.M. Jones
2022-Sep-09 11:51 UTC
[Libguestfs] [p2v PATCH v2 0/6] restrict vCPU topology to (a) fully populated physical, or (b) 1 * N * 1
On Fri, Sep 09, 2022 at 11:10:47AM +0200, Laszlo Ersek wrote:> On 09/08/22 18:27, Laszlo Ersek wrote: > > Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1590721 > > v1: https://listman.redhat.com/archives/libguestfs/2022-September/029806.html > > > > Please see the Notes section on each patch for the updates in this > > version (addressing v1 feedback). Patch #5 is a candidate for dropping > > in particular. > > > > I'm also including a range-diff between v1 and v2, below. > > > > [...] > > I meant to append: v2 builds fine at every stage again, and I retested > full conversions too, with both states of the new boolean knob.Updated series is fine, thanks. Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Read my programming and virtualization blog: http://rwmj.wordpress.com virt-p2v converts physical machines to virtual machines. Boot with a live CD or over the network (PXE) and turn machines into KVM guests. http://libguestfs.org/virt-v2v
Laszlo Ersek
2022-Sep-09 12:53 UTC
[Libguestfs] [p2v PATCH v2 0/6] restrict vCPU topology to (a) fully populated physical, or (b) 1 * N * 1
On 09/09/22 13:51, Richard W.M. Jones wrote:> On Fri, Sep 09, 2022 at 11:10:47AM +0200, Laszlo Ersek wrote: >> On 09/08/22 18:27, Laszlo Ersek wrote: >>> Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1590721 >>> v1: https://listman.redhat.com/archives/libguestfs/2022-September/029806.html >>> >>> Please see the Notes section on each patch for the updates in this >>> version (addressing v1 feedback). Patch #5 is a candidate for dropping >>> in particular. >>> >>> I'm also including a range-diff between v1 and v2, below. >>> >>> [...] >> >> I meant to append: v2 builds fine at every stage again, and I retested >> full conversions too, with both states of the new boolean knob. > > Updated series is fine, thanks.Should I drop patch#5? Dropping #5 seems to follow directly from Daniel's suggestion, and I wanted to clear it with you before merging. Thanks Laszlo