Re: [PATCH v2 7/8] powerpc/mm: remove linear mapping if __add_pages() fails in arch_add_memory()

To: Oscar Salvador <osalvador@xxxxxxx>
Subject: Re: [PATCH v2 7/8] powerpc/mm: remove linear mapping if __add_pages() fails in arch_add_memory()
From: David Hildenbrand <david@xxxxxxxxxx>
Date: Tue, 17 Nov 2020 16:53:31 +0100
Cc: linux-kernel@xxxxxxxxxxxxxxx, linux-mm@xxxxxxxxx, linuxppc-dev@xxxxxxxxxxxxxxxx, Michael Ellerman <mpe@xxxxxxxxxxxxxx>, Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx>, Paul Mackerras <paulus@xxxxxxxxx>, Rashmica Gupta <rashmica.g@xxxxxxxxx>, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>, Mike Rapoport <rppt@xxxxxxxxxx>, Michal Hocko <mhocko@xxxxxxxx>, Wei Yang <richard.weiyang@xxxxxxxxxxxxxxxxx>
In-reply-to: <20201117155125.GF15987@linux>
Organization: Red Hat GmbH
References: <20201111145322.15793-1-david@redhat.com> <20201111145322.15793-8-david@redhat.com> <20201117155125.GF15987@linux>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.6.0

On 17.11.20 16:51, Oscar Salvador wrote:

On Wed, Nov 11, 2020 at 03:53:21PM +0100, David Hildenbrand wrote:

Let's revert what we did in case seomthing goes wrong and we return an

"something" :-)


Thanks! :)

@Michael, I assume if I don't have to resend, this can be fixed up?

--
Thanks,

David / dhildenb

Follow-Ups:
- Re: [PATCH v2 7/8] powerpc/mm: remove linear mapping if __add_pages() fails in arch_add_memory()
  - From: Michael Ellerman

References:
- [PATCH v2 0/8] powernv/memtrace: don't abuse memory hot(un)plug infrastructure for memory allocations
  - From: David Hildenbrand
- [PATCH v2 7/8] powerpc/mm: remove linear mapping if __add_pages() fails in arch_add_memory()
  - From: David Hildenbrand
- Re: [PATCH v2 7/8] powerpc/mm: remove linear mapping if __add_pages() fails in arch_add_memory()
  - From: Oscar Salvador