Bug 671326

Summary: Add new smbios information can't work
Product: Red Hat Enterprise Linux 6 Reporter: yanbing du <ydu>
Component: libvirtAssignee: Daniel Veillard <veillard>
Status: CLOSED NOTABUG QA Contact: Virtualization Bugs <virt-bugs>
Severity: low Docs Contact:
Priority: low    
Version: 6.1CC: dyuan, eblake, xen-maint, yoyzhang
Target Milestone: rc   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-01-21 10:27:15 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description yanbing du 2011-01-21 08:59:46 UTC
Description of problem:
Through edit a guest's XML file, adding new smbios information for the guest, but it doesn't work.

Version-Release number of selected component (if applicable):
#uname -a
Linux dhcp-65-128.nay.redhat.com 2.6.32-100.el6.x86_64 #1 SMP Wed Jan 19 12:11:17 EST 2011 x86_64 x86_64 x86_64 GNU/Linux
# rpm -q libvirt
libvirt-0.8.7-3.el6.x86_64
# rpm -q qemu-kvm
qemu-kvm-0.12.1.2-2.129.el6.x86_64

How reproducible:
Always

Steps to Reproduce:
1.Add the following <sysinfo> section into a guest XML defination

<domain type='kvm' id='55'>
  <name>rhel6.1</name>
  <uuid>1849d614-50cf-b09d-dbf7-e5909a55c561</uuid>
  <memory>1048576</memory>
  <currentMemory>1048576</currentMemory>
  <vcpu>1</vcpu>
  <sysinfo type='smbios'>
    <bios>
      <entry name='vendor'>LENOVO</entry>
      <entry name='version'>6FET82WW (3.12 )</entry>
    </bios>
    <system>
      <entry name='manufacturer'>Fedora</entry>
      <entry name='product'>Virt-Manager</entry>
      <entry name='version'>0.8.2-3.fc14</entry>
      <entry name='serial'>32dfcb37-5af1-552b-357c-be8c3aa38310</entry>
      <entry name='sku'>1234567890</entry>
      <entry name='family'>Red Hat</entry>
    </system>
  </sysinfo>
  <os>
    <type arch='x86_64' machine='rhel6.0.0'>hvm</type>
    <boot dev='hd'/>
  </os>
  ...
2.Start the guest, run "dmidecode" command to examine appropriate seabios informatin

3.
  
Actual results:

# dmidecode 2.10
SMBIOS 2.4 present.
10 structures occupying 306 bytes.
Table at 0x7FFFFEC0.

Handle 0x0000, DMI type 0, 24 bytes
BIOS Information
        Vendor: Seabios
        Version: 0.5.1
        Release Date: 01/01/2007
        Address: 0xE8000
        Runtime Size: 96 kB
        ROM Size: 64 kB
        Characteristics:
                BIOS characteristics not supported
                Targeted content distribution is supported
        BIOS Revision: 1.0

Handle 0x0100, DMI type 1, 27 bytes
System Information
        Manufacturer: Red Hat
        Product Name: KVM
        Version: RHEL 6.0.0 PC
        Serial Number: Not Specified
        UUID: 877821DB-FC5B-E2A6-11C1-66C7B2CD6303
        Wake-up Type: Power Switch
        SKU Number: Not Specified
        Family: Red Hat Enterprise Linux
...

Expected results:

# dmidecode 2.10
SMBIOS 2.4 present.
10 structures occupying 306 bytes.
Table at 0x3FFFFEC0.

Handle 0x0000, DMI type 0, 24 bytes
BIOS Information
        Vendor: LENOVO
        Version: 6FET82WW (3.12 )
        Release Date: 01/01/2007
        Address: 0xE8000
        Runtime Size: 96 kB
        ROM Size: 64 kB
        Characteristics:
                BIOS characteristics not supported
                Targeted content distribution is supported
        BIOS Revision: 1.0

Handle 0x0100, DMI type 1, 27 bytes
System Information
        Manufacturer: Fedora
        Product Name: Virt-Manager
        Version: 0.8.2-3.fc14
        Serial Number: 32dfcb37-5af1-552b-357c-be8c3aa38310
        UUID: 784F384B-7FA0-29B9-2D9E-A99F0CEC6BF0
        Wake-up Type: Power Switch
        SKU Number: 1234567890
        Family: Red Hat

Additional info:

Comment 2 Daniel Veillard 2011-01-21 10:27:15 UTC
You also forgot to add 

  <smbios mode='sysinfo'/>

in the <os> section to pick the sysinfo informations and expose them
though smbios. Behaviour is normal.

Daniel