From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:45250) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WIfvO-0007L2-PE for qemu-devel@nongnu.org; Wed, 26 Feb 2014 09:57:38 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WIfvF-0004Bt-Kg for qemu-devel@nongnu.org; Wed, 26 Feb 2014 09:57:30 -0500 Received: from e33.co.us.ibm.com ([32.97.110.151]:60386) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WIfvF-0004Bp-EQ for qemu-devel@nongnu.org; Wed, 26 Feb 2014 09:57:21 -0500 Received: from /spool/local by e33.co.us.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Wed, 26 Feb 2014 07:57:20 -0700 Received: from b03cxnp08025.gho.boulder.ibm.com (b03cxnp08025.gho.boulder.ibm.com [9.17.130.17]) by d03dlp03.boulder.ibm.com (Postfix) with ESMTP id A829819D8045 for ; Wed, 26 Feb 2014 07:57:15 -0700 (MST) Received: from d03av03.boulder.ibm.com (d03av03.boulder.ibm.com [9.17.195.169]) by b03cxnp08025.gho.boulder.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id s1QEvHUV8651106 for ; Wed, 26 Feb 2014 15:57:17 +0100 Received: from d03av03.boulder.ibm.com (localhost [127.0.0.1]) by d03av03.boulder.ibm.com (8.14.4/8.14.4/NCO v10.0 AVout) with ESMTP id s1QEvGDZ024863 for ; Wed, 26 Feb 2014 07:57:17 -0700 Message-ID: <530E00CB.2010001@linux.vnet.ibm.com> Date: Wed, 26 Feb 2014 09:57:15 -0500 From: "Jason J. Herne" MIME-Version: 1.0 References: <1390405693-15696-1-git-send-email-jjherne@us.ibm.com> <52F3A801.2030405@de.ibm.com> <530CA9F6.6010202@linux.vnet.ibm.com> <530CEBD5.3030308@de.ibm.com> <530CF063.3020505@suse.de> In-Reply-To: <530CF063.3020505@suse.de> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 8bit Subject: Re: [Qemu-devel] [PATCH] s390: Storage key global access Reply-To: jjherne@linux.vnet.ibm.com List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?ISO-8859-1?Q?Andreas_F=E4rber?= , Christian Borntraeger , "Jason J. Herne" Cc: agraf@suse.de, qemu-devel@nongnu.org On 02/25/2014 02:34 PM, Andreas Färber wrote: > Hi, > > Am 25.02.2014 20:15, schrieb Christian Borntraeger: >> On 25/02/14 15:34, Jason J. Herne wrote: >> >>> Christian, at one point you mentioned that it might be helpful to see this patch in the context of the rest of the hotplug patches. If you still feel this way let me know and I'll post the 4-patch series. If not, I still propose this one for s390-next. Thanks :). >> >> Do you feel your series is ready for upstream, then yet please post the whole series. >> Posting independent things is good, but I feel that the storage key rework makes more >> sense if the followup patches make clear why. > > I had requested changes to that series that apparently I could not > communicate in a form Jason could digest, and I have since been caught > in downstream work and a backlog of other patches, not getting to > writing the alternative myself yet nor will I the next few days. > Andreas, I believe I understood about 90% of what you were asking for. I had made many of the changes you requested. The last version of my patches can be seen here https://www.mail-archive.com/qemu-devel@nongnu.org/msg201796.html Most of the requested changes should be found within this set of patches. > An outline of the idea as far as I remember was dropping the ipi array > instead of refactoring it to dynamic allocation and - having discussed > that a topology will not be needed - add them as cpu[n] child > properties of /machine, allowing access via QOM property getters instead > of some self-cooked solution. Open question was link<> or child<> > property and, if link<>, whether some setter hook in QOM infrastructure > may be needed to trigger the hot-add or whether QOM realize event will > be sufficient. As per your original suggestion to try link<>, I wrote the code to do exactly that. Please see patch 6/8 in the above series. I have a version of these patches rebased for the latest master, but my test system is currently in use. I would like a chance to regression test them before I post them. I anticipate I should be able to do this today. At your option, you may opt to review the patches as posted at the above link. While some modifications have since been made, the overall design and function is the same. I look forward to your comments. Thank you for your time. -- -- Jason J. Herne (jjherne@linux.vnet.ibm.com)