Bug 1219042 - For public/fedora drivers, provide direct viostor download for libosinfo
Summary: For public/fedora drivers, provide direct viostor download for libosinfo
Keywords:
Status: CLOSED DEFERRED
Alias: None
Product: Virtualization Tools
Classification: Community
Component: virtio-win
Version: unspecified
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Vadim Rozenfeld
QA Contact: Virtualization Bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-05-06 13:25 UTC by Zeeshan Ali
Modified: 2019-05-25 13:18 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-05-25 13:18:23 UTC
Embargoed:


Attachments (Terms of Use)

Description Zeeshan Ali 2015-05-06 13:25:28 UTC
This would be useful for libosinfo that needs to install certain drivers (virtio-block to be exact) as part of pre-installation drivers on windows.

Comment 1 Zeeshan Ali 2015-05-06 13:26:09 UTC
Just to be clear, by 'unpacked' i mean as files outside an ISO image.

Comment 2 Gal Hammer 2015-05-06 13:28:11 UTC
Is there a problem to mount the iso and copy the required file for libosinfo?

Comment 3 Cole Robinson 2015-05-06 14:40:30 UTC
I think zeeshan wants the files unpacked to a public URL as part of the fedora/public build process. This is from the discussion here:

https://www.redhat.com/archives/libosinfo/2015-May/msg00001.html

Comment 4 Zeeshan Ali 2015-06-09 16:21:56 UTC
(In reply to Gal Hammer from comment #2)
> Is there a problem to mount the iso and copy the required file for libosinfo?

I think the question should be why NOT provide the driver files unpacked too? i-e Why require apps (e.g Boxes) to have to unpack the drivers. In case of winxp the driver files need to be provide on the floppy disk.

(In reply to Cole Robinson from comment #3)
> I think zeeshan wants the files unpacked to a public URL as part of the
> fedora/public build process. This is from the discussion here:
> 
> https://www.redhat.com/archives/libosinfo/2015-May/msg00001.html

Well that is part of it but not the main motivation of this bug. It was more about having both the public unsigned drivers and our signed drivers be available in the same format and with same hierarchy (so libosinfo only needs to be patched to change location of drivers).

Comment 5 Vadim Rozenfeld 2015-06-25 03:18:57 UTC
(In reply to Zeeshan Ali from comment #4)
> (In reply to Gal Hammer from comment #2)
> > Is there a problem to mount the iso and copy the required file for libosinfo?
> 
> I think the question should be why NOT provide the driver files unpacked
> too? i-e Why require apps (e.g Boxes) to have to unpack the drivers. In case
> of winxp the driver files need to be provide on the floppy disk.
> 
> (In reply to Cole Robinson from comment #3)
> > I think zeeshan wants the files unpacked to a public URL as part of the
> > fedora/public build process. This is from the discussion here:
> > 
> > https://www.redhat.com/archives/libosinfo/2015-May/msg00001.html
> 
> Well that is part of it but not the main motivation of this bug. It was more
> about having both the public unsigned drivers and our signed drivers be
> available in the same format and with same hierarchy (so libosinfo only
> needs to be patched to change location of drivers).

If I understand it correctly, it is more about arranging a publicly accessible location for keeping unpacked virtio-win drivers, rather than virtio-win issue. Technically we don't pack/unpack or in any other way modify virtio-win drivers
layout. We just build the drivers and provide them as a zip file which after extracting has the following (sorry for the long post) layout :

/home/share/builds/b105/
├── qemupciserial.cat
├── qemupciserial.inf
├── Vista
│   ├── amd64
│   │   ├── netkvm.cat
│   │   ├── netkvmco.dll
│   │   ├── netkvm.inf
│   │   ├── netkvm.pdb
│   │   ├── netkvm.sys
│   │   ├── NetKVMTemporaryCert.cer
│   │   └── readme.doc
│   └── x86
│       ├── netkvm.cat
│       ├── netkvmco.dll
│       ├── netkvm.inf
│       ├── netkvm.pdb
│       ├── netkvm.sys
│       ├── NetKVMTemporaryCert.cer
│       └── readme.doc
├── win7
│   ├── amd64
│   │   ├── balloon.cat
│   │   ├── balloon.inf
│   │   ├── balloon.pdb
│   │   ├── balloon.sys
│   │   ├── blnsvr.exe
│   │   ├── blnsvr.pdb
│   │   ├── netkvm.cat
│   │   ├── netkvmco.dll
│   │   ├── netkvm.inf
│   │   ├── netkvm.pdb
│   │   ├── netkvm.sys
│   │   ├── NetKVMTemporaryCert.cer
│   │   ├── pvpanic.cat
│   │   ├── pvpanic.inf
│   │   ├── pvpanic.pdb
│   │   ├── pvpanic.sys
│   │   ├── readme.doc
│   │   ├── viorng.cat
│   │   ├── viorngci.dll
│   │   ├── viorng.inf
│   │   ├── viorng.pdb
│   │   ├── viorng.sys
│   │   ├── viorngum.dll
│   │   ├── vioscsi.cat
│   │   ├── vioscsi.inf
│   │   ├── vioscsi.pdb
│   │   ├── vioscsi.sys
│   │   ├── vioser.cat
│   │   ├── vioser.inf
│   │   ├── vioser.pdb
│   │   ├── vioser.sys
│   │   ├── vioser-test.exe
│   │   ├── vioser-test.pdb
│   │   ├── viostor.cat
│   │   ├── viostor.inf
│   │   ├── viostor.pdb
│   │   ├── viostor.sys
│   │   └── WdfCoInstaller01009.dll
│   └── x86
│       ├── balloon.cat
│       ├── balloon.inf
│       ├── balloon.pdb
│       ├── balloon.sys
│       ├── blnsvr.exe
│       ├── blnsvr.pdb
│       ├── netkvm.cat
│       ├── netkvmco.dll
│       ├── netkvm.inf
│       ├── netkvm.pdb
│       ├── netkvm.sys
│       ├── NetKVMTemporaryCert.cer
│       ├── pvpanic.cat
│       ├── pvpanic.inf
│       ├── pvpanic.pdb
│       ├── pvpanic.sys
│       ├── readme.doc
│       ├── viorng.cat
│       ├── viorngci.dll
│       ├── viorng.inf
│       ├── viorng.pdb
│       ├── viorng.sys
│       ├── viorngum.dll
│       ├── vioscsi.cat
│       ├── vioscsi.inf
│       ├── vioscsi.pdb
│       ├── vioscsi.sys
│       ├── vioser.cat
│       ├── vioser.inf
│       ├── vioser.pdb
│       ├── vioser.sys
│       ├── vioser-test.exe
│       ├── vioser-test.pdb
│       ├── viostor.cat
│       ├── viostor.inf
│       ├── viostor.pdb
│       ├── viostor.sys
│       └── WdfCoInstaller01009.dll
├── win8
│   ├── amd64
│   │   ├── balloon.cat
│   │   ├── balloon.inf
│   │   ├── balloon.pdb
│   │   ├── balloon.sys
│   │   ├── blnsvr.exe
│   │   ├── blnsvr.pdb
│   │   ├── netkvm.cat
│   │   ├── netkvmco.dll
│   │   ├── netkvm.DVL.XML
│   │   ├── netkvm.inf
│   │   ├── netkvm.pdb
│   │   ├── netkvm.sys
│   │   ├── NetKVMTemporaryCert.cer
│   │   ├── pvpanic.cat
│   │   ├── pvpanic.inf
│   │   ├── pvpanic.pdb
│   │   ├── pvpanic.sys
│   │   ├── readme.doc
│   │   ├── viorng.cat
│   │   ├── viorngci.dll
│   │   ├── viorng.inf
│   │   ├── viorng.pdb
│   │   ├── viorng.sys
│   │   ├── viorngum.dll
│   │   ├── vioscsi.cat
│   │   ├── vioscsi.DVL.XML
│   │   ├── vioscsi.inf
│   │   ├── vioscsi.pdb
│   │   ├── vioscsi.sys
│   │   ├── vioser.cat
│   │   ├── vioser.inf
│   │   ├── vioser.pdb
│   │   ├── vioser.sys
│   │   ├── viostor.cat
│   │   ├── viostor.DVL.XML
│   │   ├── viostor.inf
│   │   ├── viostor.pdb
│   │   ├── viostor.sys
│   │   └── WdfCoInstaller01011.dll
│   └── x86
│       ├── balloon.cat
│       ├── balloon.inf
│       ├── balloon.pdb
│       ├── balloon.sys
│       ├── blnsvr.exe
│       ├── blnsvr.pdb
│       ├── netkvm.cat
│       ├── netkvmco.dll
│       ├── netkvm.inf
│       ├── netkvm.pdb
│       ├── netkvm.sys
│       ├── NetKVMTemporaryCert.cer
│       ├── pvpanic.cat
│       ├── pvpanic.inf
│       ├── pvpanic.pdb
│       ├── pvpanic.sys
│       ├── readme.doc
│       ├── viorng.cat
│       ├── viorngci.dll
│       ├── viorng.inf
│       ├── viorng.pdb
│       ├── viorng.sys
│       ├── viorngum.dll
│       ├── vioscsi.cat
│       ├── vioscsi.inf
│       ├── vioscsi.pdb
│       ├── vioscsi.sys
│       ├── vioser.cat
│       ├── vioser.inf
│       ├── vioser.pdb
│       ├── vioser.sys
│       ├── viostor.cat
│       ├── viostor.inf
│       ├── viostor.pdb
│       ├── viostor.sys
│       └── WdfCoInstaller01011.dll
├── Wlh
│   ├── amd64
│   │   ├── balloon.cat
│   │   ├── balloon.inf
│   │   ├── balloon.pdb
│   │   ├── balloon.sys
│   │   ├── blnsvr.exe
│   │   ├── blnsvr.pdb
│   │   ├── pvpanic.cat
│   │   ├── pvpanic.inf
│   │   ├── pvpanic.pdb
│   │   ├── pvpanic.sys
│   │   ├── viorng.cat
│   │   ├── viorngci.dll
│   │   ├── viorng.inf
│   │   ├── viorng.pdb
│   │   ├── viorng.sys
│   │   ├── viorngum.dll
│   │   ├── vioscsi.cat
│   │   ├── vioscsi.inf
│   │   ├── vioscsi.pdb
│   │   ├── vioscsi.sys
│   │   ├── vioser.cat
│   │   ├── vioser.inf
│   │   ├── vioser.pdb
│   │   ├── vioser.sys
│   │   ├── vioser-test.exe
│   │   ├── vioser-test.pdb
│   │   ├── viostor.cat
│   │   ├── viostor.inf
│   │   ├── viostor.pdb
│   │   ├── viostor.sys
│   │   └── WdfCoInstaller01009.dll
│   └── x86
│       ├── balloon.cat
│       ├── balloon.inf
│       ├── balloon.pdb
│       ├── balloon.sys
│       ├── blnsvr.exe
│       ├── blnsvr.pdb
│       ├── pvpanic.cat
│       ├── pvpanic.inf
│       ├── pvpanic.pdb
│       ├── pvpanic.sys
│       ├── viorng.cat
│       ├── viorngci.dll
│       ├── viorng.inf
│       ├── viorng.pdb
│       ├── viorng.sys
│       ├── viorngum.dll
│       ├── vioscsi.cat
│       ├── vioscsi.inf
│       ├── vioscsi.pdb
│       ├── vioscsi.sys
│       ├── vioser.cat
│       ├── vioser.inf
│       ├── vioser.pdb
│       ├── vioser.sys
│       ├── vioser-test.exe
│       ├── vioser-test.pdb
│       ├── viostor.cat
│       ├── viostor.inf
│       ├── viostor.pdb
│       ├── viostor.sys
│       └── WdfCoInstaller01009.dll
├── Wnet
│   ├── amd64
│   │   ├── balloon.cat
│   │   ├── balloon.inf
│   │   ├── balloon.pdb
│   │   ├── balloon.sys
│   │   ├── blnsvr.exe
│   │   ├── blnsvr.pdb
│   │   ├── vioser.cat
│   │   ├── vioser.inf
│   │   ├── vioser.pdb
│   │   ├── vioser.sys
│   │   ├── vioser-test.exe
│   │   ├── vioser-test.pdb
│   │   ├── viostor.cat
│   │   ├── viostor.inf
│   │   ├── viostor.pdb
│   │   ├── viostor.sys
│   │   └── WdfCoInstaller01009.dll
│   └── x86
│       ├── balloon.cat
│       ├── balloon.inf
│       ├── balloon.pdb
│       ├── balloon.sys
│       ├── blnsvr.exe
│       ├── blnsvr.pdb
│       ├── vioser.cat
│       ├── vioser.inf
│       ├── vioser.pdb
│       ├── vioser.sys
│       ├── vioser-test.exe
│       ├── vioser-test.pdb
│       ├── viostor.cat
│       ├── viostor.inf
│       ├── viostor.pdb
│       ├── viostor.sys
│       └── WdfCoInstaller01009.dll
├── WXp
│   └── x86
│       ├── balloon.cat
│       ├── balloon.inf
│       ├── balloon.pdb
│       ├── balloon.sys
│       ├── blnsvr.exe
│       ├── blnsvr.pdb
│       ├── vioser.cat
│       ├── vioser.inf
│       ├── vioser.pdb
│       ├── vioser.sys
│       ├── vioser-test.exe
│       ├── vioser-test.pdb
│       ├── viostor.cat
│       ├── viostor.inf
│       ├── viostor.pdb
│       ├── viostor.sys
│       └── WdfCoInstaller01009.dll
└── XP
    ├── amd64
    │   ├── netkvm.cat
    │   ├── netkvm.inf
    │   ├── netkvm.pdb
    │   └── netkvm.sys
    └── x86
        ├── netkvm.cat
        ├── netkvm.inf
        ├── netkvm.pdb
        └── netkvm.sys

Please let me know if you think that any future layout modification is needed,
but at virtio-win team we usually don't deal with that stuff, leaving it to others to rearrange and adjust the layout to suit their needs.

Best regards,
Vadim.

Comment 7 Cole Robinson 2019-05-25 13:18:23 UTC
I'm closing this. I'm more involved in libosinfo side of things these days. We have had some high level discussions about how it consumes virtio-win bits. It may involve some changes/additions to what we publish publicly but until then this bug doesn't add much


Note You need to log in before you can comment on or make changes to this bug.