Bug 1249462 - beaker-system-scan (lshw branch) does not report PCIID for virtio memory balloon device
Summary: beaker-system-scan (lshw branch) does not report PCIID for virtio memory ball...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Beaker
Classification: Retired
Component: inventory
Version: 21
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: 21.1
Assignee: Dan Callaghan
QA Contact: tools-bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-08-03 04:36 UTC by Dan Callaghan
Modified: 2015-10-21 03:25 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-10-21 03:25:47 UTC
Embargoed:


Attachments (Terms of Use)

Description Dan Callaghan 2015-08-03 04:36:55 UTC
{ "vendorID": "1af4", "description": "Virtio memory balloon", "subsysVendorID": "1af4", "bus": "pci", "driver": "virtio-pci", "deviceID": "1002", "subsysDeviceID": "0005", "type": "OTHER" }

Missing from new beaker-system-scan's output.

The problem is that the lshw device has class="memory" which beaker-system-scan explicitly skips in order to avoid reporting DIMMs/banks as devices.

Comment 1 Dan Callaghan 2015-08-19 07:03:29 UTC
The class="memory" is coming from the PCI class of the mem balloon device. It seems to depend on the version of the host hypervisor. Testing with a recent qemu version on the host, the mem balloon class is 0x00ff00 which is "other". This is actually more accurate (the balloon is not a piece of memory, it's a thing that *consumes* memory according to instructions from the host) so I guess it has been fixed in qemu at some point.

... And indeed, here is the commit:

http://git.qemu.org/?p=qemu.git;a=commitdiff;h=2ba1d381c2f5f5868fe071b45977c2ed459d78f0

I would be tempted to close this bug as WONTFIX on that basis. However I am guessing RHEL6 qemu does not have this fix and so guests hosted on RHEL6 hosts will continue to have this problem. So we probably need a workaround in beaker-system-scan.

Comment 2 Dan Callaghan 2015-08-19 07:16:37 UTC
http://gerrit.beaker-project.org/4355

Comment 5 Dan Callaghan 2015-10-21 03:25:47 UTC
Beaker 21.1 has been released.


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