From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-it0-f69.google.com (mail-it0-f69.google.com [209.85.214.69]) by kanga.kvack.org (Postfix) with ESMTP id 9FE8A6B0033 for ; Tue, 12 Dec 2017 16:28:05 -0500 (EST) Received: by mail-it0-f69.google.com with SMTP id b11so844871itj.0 for ; Tue, 12 Dec 2017 13:28:05 -0800 (PST) Received: from mail-sor-f41.google.com (mail-sor-f41.google.com. [209.85.220.41]) by mx.google.com with SMTPS id t80sor71280ioi.142.2017.12.12.13.28.04 for (Google Transport Security); Tue, 12 Dec 2017 13:28:04 -0800 (PST) Date: Tue, 12 Dec 2017 13:28:01 -0800 (PST) From: David Rientjes Subject: Re: [patch 1/2] mm, mmu_notifier: annotate mmu notifiers with blockable invalidate callbacks In-Reply-To: <20171212200542.GJ5848@hpe.com> Message-ID: References: <20171212200542.GJ5848@hpe.com> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-linux-mm@kvack.org List-ID: To: Dimitri Sivanich Cc: Andrew Morton , Michal Hocko , Andrea Arcangeli , Benjamin Herrenschmidt , Paul Mackerras , Oded Gabbay , Alex Deucher , =?UTF-8?Q?Christian_K=C3=B6nig?= , David Airlie , Joerg Roedel , Doug Ledford , Jani Nikula , Mike Marciniszyn , Sean Hefty , Dimitri Sivanich , Boris Ostrovsky , =?UTF-8?Q?J=C3=A9r=C3=B4me_Glisse?= , Paolo Bonzini , =?UTF-8?Q?Radim_Kr=C4=8Dm=C3=A1=C5=99?= , linux-kernel@vger.kernel.org, linux-mm@kvack.org On Tue, 12 Dec 2017, Dimitri Sivanich wrote: > > --- a/drivers/misc/sgi-gru/grutlbpurge.c > > +++ b/drivers/misc/sgi-gru/grutlbpurge.c > > @@ -298,6 +298,7 @@ struct gru_mm_struct *gru_register_mmu_notifier(void) > > return ERR_PTR(-ENOMEM); > > STAT(gms_alloc); > > spin_lock_init(&gms->ms_asid_lock); > > + gms->ms_notifier.flags = 0; > > gms->ms_notifier.ops = &gru_mmuops; > > atomic_set(&gms->ms_refcnt, 1); > > init_waitqueue_head(&gms->ms_wait_queue); > > diff --git a/drivers/xen/gntdev.c b/drivers/xen/gntdev.c > > There is a kzalloc() just above this: > gms = kzalloc(sizeof(*gms), GFP_KERNEL); > > Is that not sufficient to clear the 'flags' field? > Absolutely, but whether it is better to explicitly document that the mmu notifier has cleared flags, i.e. there are no blockable callbacks, is another story. I can change it if preferred. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org