Thanks David,
I will fix this and upload next version.

Regards
-Vijay

On 11/21/18, 4:55 PM, "David Miller" <da...@davemloft.net> wrote:

    From: Vijay Khemka <vijaykhe...@fb.com>
    Date: Tue, 20 Nov 2018 12:35:16 -0800
    
    > +static int ncsi_oem_gma_handler_mlx(struct ncsi_cmd_arg *nca)
    > +{
    > + unsigned char data[NCSI_OEM_MLX_CMD_GMA_LEN];
    > + int ret = 0;
    > +
    > + nca->payload = NCSI_OEM_MLX_CMD_GMA_LEN;
    > +
    > + memset(data, 0, NCSI_OEM_MLX_CMD_GMA_LEN);
    > + *(unsigned int *)data = ntohl(NCSI_OEM_MFR_MLX_ID);
    
    'data' is not necessarily aligned on an unsigned int boundary.
    
    But second of all you want to use a fixed size type like "u32" or
    similar here, not "unsigned int".
    
    Altogether, something like:
    
        union {
                u8 data_u8[NCSI_OEM_MLX_CMD_GMA_LEN];
                u32 data_u32[[NCSI_OEM_MLX_CMD_GMA_LEN / sizeof(u32)];
        } u;
    
        memset(&u, 0, sizeof(u));
        u.data_u32[0] = ntohl(NCSI_OEM_MFR_MLX_ID);
        u.data_u8[5] = NCSI_OEM_MLX_CMD_GMA;
        u.data_u8[6] = NCSI_OEM_MLX_CMD_GMA_PARAM;
    
    If the rest of the ncsi driver has crud like this, it all needs
    to be fixed up similarly.
    

Reply via email to