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

Reply via email to