Subject: Re: [RESEND 1/3] powerpc/mce: Bug fixes for MCE handling in kernel space

From: Balbir Singh <[email-masked]>
Date: Thu, 5 Apr 2018 11:11:56 +1000

On Thu, 5 Apr 2018 09:49:00 +1000
Nicholas Piggin <[email-masked]> wrote:

> On Thu,  5 Apr 2018 09:19:41 +1000
> Balbir Singh <[email-masked]> wrote:
> 
> > The code currently assumes PAGE_SHIFT as the shift value of
> > the pfn, this works correctly (mostly) for user space pages,
> > but the correct thing to do is  
> 
> It would be good to actually explain the problem in the
> changelog. I would have thought pte_pfn returns a
> PAGE_SIZE based pfn value?
>

The issue is hidden inside of hugepte_offset() as invoked by __find_linux_pte().
I will send a new version because the code needs to do
<< (shift - PAGE_SHIFT) for instruction address.

> > 
> > 1. Extrace the shift value returned via the pte-walk API's  
> 
>       ^^^ extract?

Thanks, yes, typo!

Balbir Singh.