lkml.org 
[lkml]   [2018]   [Apr]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH v2 2/2] x86/mm: implement free pmd/pte page interfaces
Date
On Mon, 2018-04-30 at 13:00 +0530, Chintan Pandya wrote:
>
> On 4/29/2018 2:24 AM, Kani, Toshi wrote:
> > On Sat, 2018-04-28 at 11:02 +0200, joro@8bytes.org wrote:
> > > On Fri, Apr 27, 2018 at 02:31:51PM +0000, Kani, Toshi wrote:
> > > > So, we can add the step 2 on top of this patch.
> > > > 1. Clear pud/pmd entry.
> > > > 2. System wide TLB flush <-- TO BE ADDED BY NEW PATCH
> > > > 3. Free its underlining pmd/pte page.
> > >
> > > This still lacks the page-table synchronization and will thus not fix
> > > the BUG_ON being triggered.
> >
> > The BUG_ON issue is specific to PAE that it syncs at the pmd level.
> > x86/64 does not have this issue since it syncs at the pgd or p4d level.
> >
> > > > We do not need to revert this patch. We can make the above change I
> > > > mentioned.
> > >
> > > Please note that we are not in the merge window anymore and that any fix
> > > needs to be simple and obviously correct.
> >
> > Understood. Changing the x86/32 sync point is risky. So, I am going to
> > revert the free page handling for PAE.
>
> Will this affect pmd_free_pte_page() & pud_free_pmd_page() 's existence
> or its parameters ? I'm asking because, I've similar change for arm64
> and ready to send v9 patches.

No, it won't. The change is only to the x86 side.

> I'm thinking to share my v9 patches in any case. If you are going to do
> TLB invalidation within these APIs, my first patch will help.

I will make my change on top of your v9 1/4 patch so that we can avoid
merge conflict.

Thanks,
-Toshi
\
 
 \ /
  Last update: 2018-04-30 15:44    [W:0.085 / U:0.240 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site