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

Re: [Libguestfs] [PATCH] v2v: virtio-win: include *.dll too



On Tue, Oct 27, 2015 at 09:12:41AM +0000, Richard W.M. Jones wrote:
> On Mon, Oct 26, 2015 at 09:00:03PM +0300, Roman Kagan wrote:
> > Windows QXL drivers include also qxldd.dll which used to get filtered
> > out and not copied over into the guest.  As a result QXL driver failed
> > to install due to a missing file.
> >      (* Skip files without specific extensions. *)
> > -    let extensions = ["cat"; "inf"; "pdb"; "sys"] in
> > +    let extensions = ["cat"; "dll"; "inf"; "pdb"; "sys"] in
> 
> Actually no, this part is (may be) wrong.
> 
> This causes WdfCoInstaller*.dll from the ISO to be copied in.  It's
> not clear to me whether these files are needed - I suspect not.

Yes it is, see the corresponding *.inf where it's mentioned in the
corresponding CopyFiles directive (and yes, I just verified that with
that file missing the balloon driver installation fails with 'file not
found' error).

As a matter of fact I tend to think that filtering of files by suffix is
a bad idea in general: the packager of the drivers should be free to
ship any files he wants, including docs and text notes.  It's only the
path that should match against the guest OS flavor and that really
matters.  E.g. the balloon driver directory also contains
blnsvr.exe (and its .pdb) which is a usermode service responsible for
feeding the balloon device with the memory stats; it's not a part of the
driver per se but it's currently bundled with it.  (I'm yet to figure
out who's responsible for installing it).

> It also causes the tests to fail, although only when you have
> installed the optional ocaml oUnit2 package:

That's a problem with the test then (and no I didn't have that package
installed so my make -C v2v check passed).

Roman.


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