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

[Xen-changelog] [xen master] public/hvm_xs_strings.h: Fix ABI regression for OEM SMBios strings



commit 0f4cb23c3ea5b987c49c9a9368e7a0d505ec064f
Author:     Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
AuthorDate: Fri Aug 30 10:40:48 2013 +0200
Commit:     Jan Beulich <jbeulich@xxxxxxxx>
CommitDate: Fri Aug 30 10:40:48 2013 +0200

    public/hvm_xs_strings.h: Fix ABI regression for OEM SMBios strings
    
    The old code for OEM SMBios strings was:
    
            char path[20] = "bios-strings/oem-XX";
            path[(sizeof path) - 3] = '0' + ((i < 10) ? i : i / 10);
            path[(sizeof path) - 2] = (i < 10) ? '\0' : '0' + (i % 10);
    
    Where oem-1 thru 9 specifically had no leading 0.
    
    However, the definition of HVM_XS_OEM_STRINGS specifically requires leading
    0s.
    
    This regression was introduced by the combination of c/s 4d23036e709627 and
    e64c3f71ceb662
    
    I realise that this patch causes a change to the public headers.  However I
    feel it is justified as:
    
    * All toolstacks used to have to embed the magic string (and almost 
certainly
      still do)
    * If by some miriacle a new toolstack has started using the new define will
      continue to work.
    * The only intree consumer of the define is hvmloader itself.
    
    Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
    Acked-by: Keir Fraser <keir@xxxxxxx>
---
 xen/include/public/hvm/hvm_xs_strings.h |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/xen/include/public/hvm/hvm_xs_strings.h 
b/xen/include/public/hvm/hvm_xs_strings.h
index 4de5881..8aec935 100644
--- a/xen/include/public/hvm/hvm_xs_strings.h
+++ b/xen/include/public/hvm/hvm_xs_strings.h
@@ -75,6 +75,6 @@
 /* 1 to 99 OEM strings can be set in xenstore using values of the form
  * below. These strings will be loaded into the SMBIOS type 11 structure.
  */
-#define HVM_XS_OEM_STRINGS             "bios-strings/oem-%02d"
+#define HVM_XS_OEM_STRINGS             "bios-strings/oem-%d"
 
 #endif /* __XEN_PUBLIC_HVM_HVM_XS_STRINGS_H__ */
--
generated by git-patchbot for /home/xen/git/xen.git#master

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.