From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:49331) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZPLye-0000qq-Fe for qemu-devel@nongnu.org; Tue, 11 Aug 2015 22:41:17 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZPLyc-0008Tt-PB for qemu-devel@nongnu.org; Tue, 11 Aug 2015 22:41:16 -0400 Date: Wed, 12 Aug 2015 11:32:38 +1000 From: David Gibson Message-ID: <20150812013238.GE19634@voom.fritz.box> References: <1438580143-587-1-git-send-email-bharata@linux.vnet.ibm.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="cPi+lWm09sJ+d57q" Content-Disposition: inline In-Reply-To: <1438580143-587-1-git-send-email-bharata@linux.vnet.ibm.com> Subject: Re: [Qemu-devel] [RFC PATCH v0 0/5] spapr-next: Memory hotplug updates List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Bharata B Rao Cc: mdroth@linux.vnet.ibm.com, nfont@linux.vnet.ibm.com, qemu-ppc@nongnu.org, qemu-devel@nongnu.org, agraf@suse.de --cPi+lWm09sJ+d57q Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Aug 03, 2015 at 11:05:38AM +0530, Bharata B Rao wrote: > This patchset includes some updates to sPAPR memory hotplug code that > currently resides in spapr-next branch of David Gibson's tree. >=20 > The main change here is to use drc-count hotplug identifier type for memo= ry > hotplug. Currently we use drc-index type of hotplug identifier for > memory, which means that we send one EPOW event to guest on hotplug > of each LMB. Guest kernel has limited buffer space to hold these events > and we can easily overflow this buffer leading to missing of memory > hotplug events. After we switch to drc-count type of hotplug identifier, > we will send only one event to guest on hotplugging each pc-dimm device > thereby reducing the chances of guest rtas buffer overflow. This change > allows us to hotplug huge amounts of memory (>16G) in one pc-dimm > hotplug request. >=20 > Another change here is to not include information about memory specified > using -m option in ibm,drc-names, ibm,drc-indexes and > ibm,dynamic-memory-reconfiguration nodes thus having only truly DR capable > memory in these DT nodes/properties. >=20 > Bharata B Rao (5): > spapr: Provide better error message when slots exceed max allowed > spapr: Populate ibm,associativity-lookup-arrays correctly for non-NUMA > spapr: Revert to memory@XXXX representation for non-hotplugged memory > spapr: Support hotplug by specifying DRC count > spapr: Move memory hotplug to RTAS_LOG_V6_HP_ID_DRC_COUNT type >=20 > hw/ppc/spapr.c | 58 +++++++++++++-------------------------------= ------ > hw/ppc/spapr_events.c | 47 ++++++++++++++++++++++++++++++++-------- > hw/ppc/spapr_pci.c | 4 ++-- > include/hw/ppc/spapr.h | 8 +++++-- > 4 files changed, 61 insertions(+), 56 deletions(-) I've merged patches 1..3/5 into spapr-next. 4 & 5 I need to think about a bit more. --=20 David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson --cPi+lWm09sJ+d57q Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJVyqI2AAoJEGw4ysog2bOSNNYQANVv/AXVp8worAo3ub7/c68l XIfkW77V8DXyMsCGqYrLClRKC/325C0KoUsiQrvmgHI3TGzQBIv7s/rUCFcfEded CDUaoUolUG8smD166nxSe5/GN13DpUr3yr+h1yhU36MMYAfNQO0i0xWof72DUHqW 8JySHGMm2Y+RbZKxbuy0b8n3m84N83dJBwyQGH166HLRx66n8CD8/bBjjy1XQsBu CtPpzbofEIsD6aS/2wX9BDySOEHVyeo7a1AWU9sF0MzQXSJ5QE7NOJOqffJJFkMX XDHhSUHAsVUc0yxrc5Jh33Hodu5zZB5pWwoDJ6u1eAB2kXgfJRtchBeTAyCV93m1 zu368qrdnQMgxUJvWFgR7tIMiMBHjJW2dXYeyc1EwLJ71cQBPkkuswDu2r8nUd2l wfkinj6YWJ2BgS7hMy1hA9sKjqVNi1RzmFUbLeoqRCfdlQeMyAnb8gNONKNMqyau 6JkRyiq1j1sLp17MKpUc6Vn9TP/TOoKbLHQCW7Qag39DsKRgrsAM0zH9l5niRhNf MVDgrwswd0oRu5V6LT7LZNySPnkeVglE3anu8C3616YqT1FmiZLCMCDZCQ8toOpJ mImMtnK7Xpp38fakb6uz2lLvdWqYW2Sr/fNlXcpLlNNwzCCt1eV8UJWSp//5ccn2 2WwZz8izwxGYRpxaBoFV =PKs5 -----END PGP SIGNATURE----- --cPi+lWm09sJ+d57q--