Hi Borislav,
On 10 August 2016 at 19:10, Borislav Petkov bp@suse.de wrote:
On Wed, Aug 10, 2016 at 07:01:05PM +0800, Fu Wei wrote:
- if (!acpi_disable_cmcff)
- apei_hest_parse(hest_parse_cmc, NULL);
rc =apei_hest_parse(hest_parse_cmc, NULL);
if (rc)
goto err;
if (!ghes_disable) { rc = apei_hest_parse(hest_parse_ghes_count, &ghes_count);
Do you have other suggestion? then I can post v14 to fix it :-)
Nah, looks good :)
Let's keep it simple and add more involved handling when it is really needed. Thanks!
Thanks for your help, this have been fixed in v14(just posted).
Btw, that comment:
+/*
- acpi_disable_cmcff is used in drivers/acpi/apei/hest.c for disabling
- IA-32 Architecture Corrected Machine Checkx(cmc) FIRMWARE_FIRST mode by
- boot parameter(acpi=nocmcff). But we don't have this IA-32 specific
- feature on ARM64, this definition is only for compatibility.
- */
+#define acpi_disable_cmcff 1
looks ok, except the small typo:
Architecture Corrected Machine Checkx(cmc) ^ Make that
"IA-32 Architecture Corrected Machine Check (CMC) Firmware-First mode"
Done! Thanks :-)
Thanks.
-- Regards/Gruss, Boris.
ECO tip #101: Trim your mails when you reply.
SUSE Linux GmbH, GF: Felix Imendörffer, Jane Smithard, Graham Norton, HRB 21284 (AG Nürnberg)