Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1221114

Re: [PATCH V2 1/2] efi: print unrecognized CPER section

Show all headers | View raw

On 09/08/2015 04:29 PM, Jonathan (Zhixiong) Zhang wrote:

> Change-Id: I663a6e3ae6dcf68e4e389f76d555e9106ffee165

You need to strip out the Change-Id's before posting the patch.

> +#define cper_print_hex(pfx, buf, len)				\
> +	print_hex_dump(pfx, "",					\
> +		DUMP_PREFIX_OFFSET, ROW_SIZE, GROUP_SIZE,	\
> +		buf, len, 0)

		(buf), (len), 0)

is safer

> +	} else {
> +		const void *raw_err = gdata + 1;
> +		printk("%ssection type: %pUl\n", pfx, sec_type);
> +		printk("%ssection length: %d\n", pfx, len);
> +		cper_print_hex(newpfx, raw_err, len);

You don't need raw_err.  This should work fine:

		cper_print_hex(newpfx, gdata + 1, len);

-- 
Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the
Code Aurora Forum, a Linux Foundation Collaborative Project.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH V2 0/2] process unrecognized CPER error section "Jonathan (Zhixiong) Zhang" <zjzhang@codeaurora.org> - 2015-09-08 23:30 +0200
  [PATCH V2 1/2] efi: print unrecognized CPER section "Jonathan (Zhixiong) Zhang" <zjzhang@codeaurora.org> - 2015-09-08 23:40 +0200
    Re: [PATCH V2 1/2] efi: print unrecognized CPER section Timur Tabi <timur@codeaurora.org> - 2015-09-09 00:00 +0200
    Re: [PATCH V2 1/2] efi: print unrecognized CPER section Borislav Petkov <bp@alien8.de> - 2015-09-10 20:30 +0200

csiph-web