Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Message-ID: <a93a42c672171ed93557f9f9f3b5d64013980f26.camel@linux.ibm.com>
Subject: Re: [PATCH 1/1] powerpc/kernel: Enables memory hot-remove after
 reboot on pseries guests
From:   Leonardo Bras <leonardo@linux.ibm.com>
To:     Bharata B Rao <bharata.rao@gmail.com>
Cc:     ndfont@gmail.com,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        arbab@linux.ibm.com, Claudio Carvalho <cclaudio@linux.ibm.com>,
        Michael Roth <mdroth@linux.vnet.ibm.com>,
        Paul Mackerras <paulus@samba.org>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Thomas Gleixner <tglx@linutronix.de>,
        linuxppc-dev <linuxppc-dev@lists.ozlabs.org>,
        Hari Bathini <hbathini@linux.ibm.com>
Date:   Wed, 04 Mar 2020 19:05:43 -0300
In-Reply-To: <621dd76a8e0b449db66ba2c3ad20fb2c743a1f1b.camel@linux.ibm.com>
References: <20200228060439.52749-1-leonardo@linux.ibm.com>
         <CAGZKiBr=8i11YPDn+1y5j6YfGj+tVbbTKakoGje9QQ8TEw9g5g@mail.gmail.com>
         <621dd76a8e0b449db66ba2c3ad20fb2c743a1f1b.camel@linux.ibm.com>
Content-Type: multipart/signed; micalg="pgp-sha256";
        protocol="application/pgp-signature"; boundary="=-i+d4i/WWLm20T4rx7xB5"
User-Agent: Evolution 3.34.4 (3.34.4-1.fc31) 
MIME-Version: 1.0
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


--=-i+d4i/WWLm20T4rx7xB5
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Wed, 2020-03-04 at 04:18 -0300, Leonardo Bras wrote:
> Humm, this makes sense.
> But with mu change, these pieces of memory only get into ZONE_MOVABLE
> if the boot parameter 'movable_node' gets passed to guest kernel.=20

Humm, I think your patch also does that.

> So, even if we are unable to sort out some flag combination that work
> fine for both use-cases, if PowerVM don't pass 'movable_node' as boot
> parameter to kernel, it will behave just as today.

Also, another option would be adding a new 'removable' flag, given it
has a lot of free bytes. It would only be passed by qemu, so we would
be safe with PowerVM.=20

Then we would have=20
+	if(lmb->flags & DRCONF_MEM_REMOVABLE)=09
+		early_init_dt_mark_hotplug_memory_arch(base, size);

Do you know if it's possible?
We would need to update the LOPAPR?=20

Leonardo

--=-i+d4i/WWLm20T4rx7xB5
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: This is a digitally signed message part
Content-Transfer-Encoding: 7bit

-----BEGIN PGP SIGNATURE-----

iQIzBAABCAAdFiEEMdeUgIzgjf6YmUyOlQYWtz9SttQFAl5gJjcACgkQlQYWtz9S
ttSHtw/9EmfURu4xc28TKZKOR3RRVUWh04fm24UoyOjEiEQ//E0NsFKdT1sIx+uH
CSL9M3nNE0+DFz9YOtFwRKNwvrleSJUzivvaJyLU0jOsgZu0XmRHw3Sz3faYeIwU
fYke3nVYjqb+qrgJLhlX+EeVlAgpp0Wgr5gYxvMvjUrWhR163JOP8sXWZy2FW7vr
EgGOasvyrNiX/tvAa4H9mGSeL0ivmDx6O9FB/EKaZIRMXPCT9L0+jIR/upFPhNBH
3dmoCd26vpy/D/vs2thMQRU77WoIYQcGVTniM7/zEuJ2X3v0B95GnfblHh/3dQM7
D796j+eXq/Io6DQyBh4EEjMWOyYuDb3KM1G/ZJuHPs/RhTCl31DveMBC6kK7e9Yw
51q+hyU2opLZKDV3DI7EzzUIbUkC/i/wp7SmsswZQHZCpwOfCcf8/K24h9q5yUZn
yjce+OpcLy/VfhlZGhf0E5D8p+IOL0zJHFlpb3bGDAXtUecp1C8J82nLdXHFE0Pz
GitCSImCzjjwfoLEx8VYRV/TjBS9NjNbHOnG1GVVVgAlL1BauySvHYqTTdc0SlMG
f+NKuqE9U0b5hpG4+cu3PQMCnAngv4PzG9tiF4HW+2UQ4ETujvqUwMFilYJa94VT
bgB2swZfVuFydKx99Wy9FuBdsiNbzriiIccjv9Nn7MgXkJFl/+s=
=Sv41
-----END PGP SIGNATURE-----

--=-i+d4i/WWLm20T4rx7xB5--