Re: [PATCH] mm/mmu_notifier: rename mmu_notifier_synchronize() to <...>_barrier()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
Subject: Re: [PATCH] mm/mmu_notifier: rename mmu_notifier_synchronize() to <...>_barrier()
From: Matthew Wilcox <willy@xxxxxxxxxxxxx>
Date: Mon, 5 Nov 2018 13:14:55 -0800
Cc: Sean Christopherson <sean.j.christopherson@xxxxxxxxx>, linux-mm@xxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, Peter Zijlstra <peterz@xxxxxxxxxxxxx>, Jérôme Glisse <jglisse@xxxxxxxxxx>, Oded Gabbay <oded.gabbay@xxxxxxx>
In-reply-to: <20181105121833.200d5b53300a7ef4df7d349d@linux-foundation.org>
References: <20181105192955.26305-1-sean.j.christopherson@intel.com> <20181105121833.200d5b53300a7ef4df7d349d@linux-foundation.org>
User-agent: Mutt/1.9.2 (2017-12-15)

On Mon, Nov 05, 2018 at 12:18:33PM -0800, Andrew Morton wrote:
> > +++ b/mm/mmu_notifier.c
> 
> But as it has no callers, why retain it?

... and this patch missed the declaration of mmu_notifier_synchronize
in include/linux/mmu_notifier.h (whether we delete it or rename it,
that mention of it needs to be fixed)

Follow-Ups:
- Re: [PATCH] mm/mmu_notifier: rename mmu_notifier_synchronize() to <...>_barrier()
  - From: Sean Christopherson

References:
- [PATCH] mm/mmu_notifier: rename mmu_notifier_synchronize() to <...>_barrier()
  - From: Sean Christopherson
- Re: [PATCH] mm/mmu_notifier: rename mmu_notifier_synchronize() to <...>_barrier()
  - From: Andrew Morton

Prev by Date: Re: [PATCH] slab.h: Avoid using & for logical and of booleans
Next by Date: [mm PATCH v5 0/7] Deferred page init improvements
Previous by thread: Re: [PATCH] mm/mmu_notifier: rename mmu_notifier_synchronize() to <...>_barrier()
Next by thread: Re: [PATCH] mm/mmu_notifier: rename mmu_notifier_synchronize() to <...>_barrier()
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [IETF Annouce] [Bugtraq] [Linux OMAP] [Linux MIPS] [eCos] [Asterisk Internet PBX] [Linux API]