On 08/11/2011 02:36 PM, srinivas_g_go...@dell.com wrote: > Hi , > This is the patch that contains bug fixes for Non-Dell components
Hello, I can see all other patches in this patch set were accepted into upstream CVS except this one. Is there any reason? The patch does not contain any description regarding its purpose, so I can only guess what is does, but it looks to me as important in the patch set. Jan > > > ipmitool-1.8.11_8_dell_01_opensource.patch > > > diff -Naurp ipmitool_base/include/ipmitool/ipmi_fru.h > 01-ipmitool_opensource_fix/include/ipmitool/ipmi_fru.h > --- ipmitool_base/include/ipmitool/ipmi_fru.h 2011-08-04 16:48:43.000000000 > +0530 > +++ 01-ipmitool_opensource_fix/include/ipmitool/ipmi_fru.h 2011-08-04 > 16:48:42.000000000 +0530 > @@ -59,11 +59,14 @@ enum { > FRU_PRODUCT_SERIAL, > FRU_PRODUCT_ASSET, > }; > +#ifdef HAVE_PRAGMA_PACK > +#pragma pack(1) > +#endif > > struct fru_info { > uint16_t size; > uint8_t access:1; > -}; > +}ATTRIBUTE_PACKING; > > #ifdef HAVE_PRAGMA_PACK > #pragma pack(1) > @@ -596,5 +599,9 @@ static const char * chassis_type_desc[] <snip> ------------------------------------------------------------------------------ Virtualization & Cloud Management Using Capacity Planning Cloud computing makes use of virtualization - but cloud computing also focuses on allowing computing to be delivered as a service. http://www.accelacomm.com/jaw/sfnl/114/51521223/ _______________________________________________ Ipmitool-devel mailing list Ipmitool-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/ipmitool-devel