Bug 1218449

Summary: Ship qemupciserial.cat on the iso
Product: Red Hat Enterprise Linux 7 Reporter: Cole Robinson <crobinso>
Component: virtio-winAssignee: Danilo de Paula <ddepaula>
virtio-win sub component: distribution QA Contact: Virtualization Bugs <virt-bugs>
Status: CLOSED ERRATA Docs Contact:
Severity: unspecified    
Priority: unspecified CC: ailan, crobinso, ddepaula, jen, jherrman, juzhang, knoel, lijin, lmiksik, lprosek, michen, rbalakri, tlavigne, virt-maint, vrozenfe, wyu, ymankad
Version: 7.3   
Target Milestone: rc   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
The qemupciserial driver is now properly signed and works on 64-bit Windows guests.
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-08-01 12:53:08 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Cole Robinson 2015-05-04 23:48:11 UTC
We currently ship qemupciserial.inf on the virtio-win .iso, however there's also a qemupciserial.cat file which we _don't_ ship. It's unclear if that omission was intentional or just an oversight of the previous build system. I asked Vadim in private mail:

On 05/02/2015 04:10 AM, Vadim Rozenfeld wrote:>>>> > >> The qemupciserial thing seems strange... we _do_ ship the .inf but not the
>>>> > >> .cat file? What's the reason for that?
>>> > > 
>>> > > It's a very special case. The short story is like this - MS provides in-box
>>> > > serial.sys driver, which can be used for an OEM provided serial (UART) devices.
>>> > > So, it's basically what qemupciserial.inf does - binding the standard MS-provided
>>> > > in-box driver to custom PCI based device, which is fully compatible with th
>>> > > on-board serial devices.
>>> > > 
>> > 
>> > Gotchya. What's the .cat file for then? Just curious
> Oops, we must be referencing two different qemupciserial.inf files. I
> am, by mistake,  was looking into the old one, which is "Class=Ports",
> and for this one no cat file was needed, while you are asking about the
> new one "Class=MultiFunction". So please ignore everything I told you 
> before regarding to qemupciserial.inf and qemupciserial.cat files.
> For this approach we still use in in-box serial.sys file, but in this
> case we use it in conjunction with with another one in-box multifunction
> driver. For the case .cat file is absolutely necessary and should be
> distributed together with qemupciserial.inf file.


So we should distribute the .cat file as well

Comment 2 Cole Robinson 2015-08-10 23:44:21 UTC
This will be picked up automatically by the next virtio-win RPM/ISO build, the upstream scripts already have the change.

Comment 3 Jeff Nelson 2015-10-01 20:32:56 UTC
Fixed in virtio-win-1.8.0-1.el7

Comment 5 Yu Wang 2015-10-14 08:33:28 UTC
Hi

There is still no qemupciserial.cat file in /iso/qemupciserial .

Thanks
wyu

Comment 10 Cole Robinson 2016-07-18 21:31:49 UTC
This isn't blocker worthy, doesn't seem to be impacting anyone really, so dropping that flag and moving to proposing for rhel 7.4

Comment 14 Cole Robinson 2017-06-06 18:10:01 UTC
yash, I know there's been some discussion about getting this driver signed so maybe you fixed this... is qemupciserial.cat on the rhel iso these days?

Comment 15 Yash Mankad 2017-06-06 19:17:46 UTC
Yes, the qemupciserial.cat file will be on the rhel iso for the 7.4 release.

Comment 19 Danilo de Paula 2017-06-07 19:36:15 UTC
Back to MODIFIED

Comment 21 Yu Wang 2017-06-13 01:59:30 UTC
Hi,

After installing the virtio-win rpm package, qemupciserial.cat file are in iso.
version: virtio-win-1.9.1-0.el7.noarch.rpm

change status to verified.

Thanks
Yu Wang

Comment 23 errata-xmlrpc 2017-08-01 12:53:08 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2017:2341