On Thu, Oct 15, 2015 at 10:07:45AM +0100, Richard W.M. Jones wrote:
On Wed, Oct 14, 2015 at 07:55:41PM +0300, Roman Kagan wrote:
Libguestfs supports passing an ISO image as a source of virtio windows drivers to v2v.
This series attempts to make it simpler and better scoped.
Looks good (apart from patch 3 - the Gc.compact is necessary!)
Why? As a matter of fact that was the primary reason I even started with this patchset: that Gc.compact was sitting in the middle of the main v2v function and was yet another obstacle in the refactoring I was after. Now that the extra guestfs handle doesn't survive beyond copy_virtio_drivers() what is the need for that explicit GC (which was introduced at the time exactly to clear up that handle in commit 47b5f245bec908f803f0a89c3b1e3166cfe33aad)? Roman.