Thread (5 messages) 5 messages, 4 authors, 2017-05-02

Re: [PATCH] powerpc/pseries hotplug: prevent the reserved mem from removing

From: Hari Bathini <hidden>
Date: 2017-04-27 18:07:22

Hi Pingfan,


On Thursday 27 April 2017 01:13 PM, Pingfan Liu wrote:
quoted hunk ↗ jump to hunk
E.g after fadump reserves mem regions, these regions should not be removed
before fadump explicitly free them.
Signed-off-by: Pingfan Liu <redacted>
---
  arch/powerpc/platforms/pseries/hotplug-memory.c | 5 +++--
  1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/arch/powerpc/platforms/pseries/hotplug-memory.c b/arch/powerpc/platforms/pseries/hotplug-memory.c
index e104c71..201be23 100644
--- a/arch/powerpc/platforms/pseries/hotplug-memory.c
+++ b/arch/powerpc/platforms/pseries/hotplug-memory.c
@@ -346,6 +346,8 @@ static int pseries_remove_memblock(unsigned long base, unsigned int memblock_siz

  	if (!pfn_valid(start_pfn))
  		goto out;
+	if (memblock_is_reserved(base))
+		return -EINVAL;
I think memblock reserved regions are not hot removed even without this 
patch.
So, can you elaborate on when/why this patch is needed?

Thanks
Hari
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help