From: Sasha Levin <sashal@kernel•org>
To: linux-kernel@vger•kernel.org, stable@vger•kernel.org
Cc: Kate Stewart <kstewart@linuxfoundation•org>,
Michal Hocko <mhocko@suse•com>,
David Hildenbrand <david@redhat•com>,
Heiko Carstens <heiko.carstens@de•ibm.com>,
Paul Mackerras <paulus@samba•org>,
Rashmica Gupta <rashmica.g@gmail•com>,
"K. Y. Srinivasan" <kys@microsoft•com>,
Boris Ostrovsky <boris.ostrovsky@oracle•com>,
Sasha Levin <sashal@kernel•org>,
Michael Neuling <mikey@neuling•org>,
Stephen Hemminger <sthemmin@microsoft•com>,
Jonathan Corbet <corbet@lwn•net>,
"Rafael J . Wysocki" <rafael.j.wysocki@intel•com>,
Nathan Fontenot <nfont@linux•vnet.ibm.com>,
Len Brown <lenb@kernel•org>,
Pavel Tatashin <pavel.tatashin@microsoft•com>,
Haiyang Zhang <haiyangz@microsoft•com>,
linuxppc-dev@lists•ozlabs.org,
YASUAKI ISHIMATSU <yasu.isimatu@gmail•com>,
Dan Williams <dan.j.williams@intel•com>,
Andrew Morton <akpm@linux-foundation•org>,
Vlastimil Babka <vbabka@suse•cz>,
Oscar Salvador <osalvador@suse•de>,
Juergen Gross <jgross@suse•com>,
Mathieu Malaterre <malat@debian•org>,
Greg Kroah-Hartman <gregkh@linuxfoundation•org>,
"Rafael J. Wysocki" <rjw@rjwysocki•net>,
John Allen <jallen@linux•vnet.ibm.com>,
Thomas Gleixner <tglx@linutronix•de>,
Philippe Ombredanne <pombredanne@nexb•com>,
Martin Schwidefsky <schwidefsky@de•ibm.com>,
Joonsoo Kim <iamjoonsoo.kim@lge•com>,
Linus Torvalds <torvalds@linux-foundation•org>
Subject: [PATCH AUTOSEL 4.14 04/56] powerpc/powernv: hold device_hotplug_lock when calling memtrace_offline_pages()
Date: Fri, 18 Oct 2019 18:07:01 -0400 [thread overview]
Message-ID: <20191018220753.10002-4-sashal@kernel.org> (raw)
In-Reply-To: <20191018220753.10002-1-sashal@kernel.org>
From: David Hildenbrand <david@redhat•com>
[ Upstream commit 5666848774ef43d3db5151ec518f1deb63515c20 ]
Let's perform all checking + offlining + removing under
device_hotplug_lock, so nobody can mess with these devices via sysfs
concurrently.
[david@redhat•com: take device_hotplug_lock outside of loop]
Link: http://lkml.kernel.org/r/20180927092554.13567-6-david@redhat.com
Link: http://lkml.kernel.org/r/20180925091457.28651-6-david@redhat.com
Signed-off-by: David Hildenbrand <david@redhat•com>
Reviewed-by: Pavel Tatashin <pavel.tatashin@microsoft•com>
Reviewed-by: Rashmica Gupta <rashmica.g@gmail•com>
Acked-by: Balbir Singh <bsingharora@gmail•com>
Cc: Benjamin Herrenschmidt <benh@kernel•crashing.org>
Cc: Paul Mackerras <paulus@samba•org>
Cc: Michael Ellerman <mpe@ellerman•id.au>
Cc: Rashmica Gupta <rashmica.g@gmail•com>
Cc: Michael Neuling <mikey@neuling•org>
Cc: Boris Ostrovsky <boris.ostrovsky@oracle•com>
Cc: Dan Williams <dan.j.williams@intel•com>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation•org>
Cc: Haiyang Zhang <haiyangz@microsoft•com>
Cc: Heiko Carstens <heiko.carstens@de•ibm.com>
Cc: John Allen <jallen@linux•vnet.ibm.com>
Cc: Jonathan Corbet <corbet@lwn•net>
Cc: Joonsoo Kim <iamjoonsoo.kim@lge•com>
Cc: Juergen Gross <jgross@suse•com>
Cc: Kate Stewart <kstewart@linuxfoundation•org>
Cc: "K. Y. Srinivasan" <kys@microsoft•com>
Cc: Len Brown <lenb@kernel•org>
Cc: Martin Schwidefsky <schwidefsky@de•ibm.com>
Cc: Mathieu Malaterre <malat@debian•org>
Cc: Michal Hocko <mhocko@suse•com>
Cc: Nathan Fontenot <nfont@linux•vnet.ibm.com>
Cc: Oscar Salvador <osalvador@suse•de>
Cc: Philippe Ombredanne <pombredanne@nexb•com>
Cc: Rafael J. Wysocki <rafael.j.wysocki@intel•com>
Cc: "Rafael J. Wysocki" <rjw@rjwysocki•net>
Cc: Stephen Hemminger <sthemmin@microsoft•com>
Cc: Thomas Gleixner <tglx@linutronix•de>
Cc: Vlastimil Babka <vbabka@suse•cz>
Cc: YASUAKI ISHIMATSU <yasu.isimatu@gmail•com>
Signed-off-by: Andrew Morton <akpm@linux-foundation•org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation•org>
Signed-off-by: Sasha Levin <sashal@kernel•org>
---
arch/powerpc/platforms/powernv/memtrace.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/arch/powerpc/platforms/powernv/memtrace.c b/arch/powerpc/platforms/powernv/memtrace.c
index c9a6d4f3403ce..cfbd242c3e011 100644
--- a/arch/powerpc/platforms/powernv/memtrace.c
+++ b/arch/powerpc/platforms/powernv/memtrace.c
@@ -99,6 +99,7 @@ static int change_memblock_state(struct memory_block *mem, void *arg)
return 0;
}
+/* called with device_hotplug_lock held */
static bool memtrace_offline_pages(u32 nid, u64 start_pfn, u64 nr_pages)
{
u64 end_pfn = start_pfn + nr_pages - 1;
@@ -139,6 +140,7 @@ static u64 memtrace_alloc_node(u32 nid, u64 size)
/* Trace memory needs to be aligned to the size */
end_pfn = round_down(end_pfn - nr_pages, nr_pages);
+ lock_device_hotplug();
for (base_pfn = end_pfn; base_pfn > start_pfn; base_pfn -= nr_pages) {
if (memtrace_offline_pages(nid, base_pfn, nr_pages) == true) {
/*
@@ -147,7 +149,6 @@ static u64 memtrace_alloc_node(u32 nid, u64 size)
* we never try to remove memory that spans two iomem
* resources.
*/
- lock_device_hotplug();
end_pfn = base_pfn + nr_pages;
for (pfn = base_pfn; pfn < end_pfn; pfn += bytes>> PAGE_SHIFT) {
remove_memory(nid, pfn << PAGE_SHIFT, bytes);
@@ -156,6 +157,7 @@ static u64 memtrace_alloc_node(u32 nid, u64 size)
return base_pfn << PAGE_SHIFT;
}
}
+ unlock_device_hotplug();
return 0;
}
--
2.20.1
parent reply other threads:[~2019-10-18 22:12 UTC|newest]
Thread overview: expand[flat|nested] mbox.gz Atom feed
[parent not found: <20191018220753.10002-1-sashal@kernel.org>]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20191018220753.10002-4-sashal@kernel.org \
--to=sashal@kernel$(echo .)org \
--cc=akpm@linux-foundation$(echo .)org \
--cc=boris.ostrovsky@oracle$(echo .)com \
--cc=corbet@lwn$(echo .)net \
--cc=dan.j.williams@intel$(echo .)com \
--cc=david@redhat$(echo .)com \
--cc=gregkh@linuxfoundation$(echo .)org \
--cc=haiyangz@microsoft$(echo .)com \
--cc=heiko.carstens@de$(echo .)ibm.com \
--cc=iamjoonsoo.kim@lge$(echo .)com \
--cc=jallen@linux$(echo .)vnet.ibm.com \
--cc=jgross@suse$(echo .)com \
--cc=kstewart@linuxfoundation$(echo .)org \
--cc=kys@microsoft$(echo .)com \
--cc=lenb@kernel$(echo .)org \
--cc=linux-kernel@vger$(echo .)kernel.org \
--cc=linuxppc-dev@lists$(echo .)ozlabs.org \
--cc=malat@debian$(echo .)org \
--cc=mhocko@suse$(echo .)com \
--cc=mikey@neuling$(echo .)org \
--cc=nfont@linux$(echo .)vnet.ibm.com \
--cc=osalvador@suse$(echo .)de \
--cc=paulus@samba$(echo .)org \
--cc=pavel.tatashin@microsoft$(echo .)com \
--cc=pombredanne@nexb$(echo .)com \
--cc=rafael.j.wysocki@intel$(echo .)com \
--cc=rashmica.g@gmail$(echo .)com \
--cc=rjw@rjwysocki$(echo .)net \
--cc=schwidefsky@de$(echo .)ibm.com \
--cc=stable@vger$(echo .)kernel.org \
--cc=sthemmin@microsoft$(echo .)com \
--cc=tglx@linutronix$(echo .)de \
--cc=torvalds@linux-foundation$(echo .)org \
--cc=vbabka@suse$(echo .)cz \
--cc=yasu.isimatu@gmail$(echo .)com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox