[PATCH v2] x86/mce: retrieve poison range from hardware whenever supported

2022-07-15 Thread Jane Chu
With Commit 7917f9cdb503 ("acpi/nfit: rely on mce->misc to determine poison granularity") that changed nfit_handle_mce() callback to report badrange according to 1ULL << MCI_MISC_ADDR_LSB(mce->misc), it's been discovered that the mce->misc LSB field is 0x1000 bytes, hence injecting 2 back-to-back

Re: [PATCH] acpi/nfit: badrange report spill over to clean range

2022-07-15 Thread Dan Williams
[ add Tony ] Jane Chu wrote: > On 7/14/2022 6:19 PM, Dan Williams wrote: > > Jane Chu wrote: > >> I meant to say there would be 8 calls to the nfit_handle_mce() callback, > >> one call for each poison with accurate address. > >> > >> Also, short ARS would find 2 poisons. > >> > >> I attached the

Re: [PATCH] acpi/nfit: badrange report spill over to clean range

2022-07-15 Thread Jane Chu
On 7/15/2022 12:17 PM, Dan Williams wrote: > [ add Tony ] > > Jane Chu wrote: >> On 7/14/2022 6:19 PM, Dan Williams wrote: >>> Jane Chu wrote: I meant to say there would be 8 calls to the nfit_handle_mce() callback, one call for each poison with accurate address. Also, short

RE: [PATCH v2] x86/mce: retrieve poison range from hardware whenever supported

2022-07-15 Thread Dan Williams
Jane Chu wrote: > With Commit 7917f9cdb503 ("acpi/nfit: rely on mce->misc to determine > poison granularity") that changed nfit_handle_mce() callback to report > badrange according to 1ULL << MCI_MISC_ADDR_LSB(mce->misc), it's been > discovered that the mce->misc LSB field is 0x1000 bytes, hence

Re: [PATCH] acpi/nfit: badrange report spill over to clean range

2022-07-15 Thread Jane Chu
On 7/14/2022 5:58 PM, Dan Williams wrote: [..] >>> > However, the ARS engine likely can return the precise error ranges so I > think the fix is to just use the address range indicated by 1UL << > MCI_MISC_ADDR_LSB(mce->misc) to filter the results from a short ARS > scrub request to

Re: [PATCH] acpi/nfit: badrange report spill over to clean range

2022-07-15 Thread Jane Chu
On 7/14/2022 6:19 PM, Dan Williams wrote: > Jane Chu wrote: >> I meant to say there would be 8 calls to the nfit_handle_mce() callback, >> one call for each poison with accurate address. >> >> Also, short ARS would find 2 poisons. >> >> I attached the console output, my annotation is prefixed with