From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756828AbcBWCce (ORCPT ); Mon, 22 Feb 2016 21:32:34 -0500 Received: from eu-smtp-delivery-143.mimecast.com ([207.82.80.143]:45441 "EHLO eu-smtp-delivery-143.mimecast.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756699AbcBWCcc convert rfc822-to-8bit (ORCPT ); Mon, 22 Feb 2016 21:32:32 -0500 Date: Tue, 23 Feb 2016 10:31:57 +0800 From: Huang Shijie To: Julien Grall CC: , , , , , , , , Subject: Re: [PATCH v2 0/6] arm64: Add support of KVM with ACPI Message-ID: <20160223023156.GA11645@sha-win-210.asiapac.arm.com> References: <1455204804-31830-1-git-send-email-julien.grall@arm.com> MIME-Version: 1.0 In-Reply-To: <1455204804-31830-1-git-send-email-julien.grall@arm.com> User-Agent: Mutt/1.5.24 (2015-08-30) X-Originating-IP: [217.140.104.200] X-ClientProxiedBy: TY1PR01CA0013.jpnprd01.prod.outlook.com (25.161.131.151) To VI1PR08MB0448.eurprd08.prod.outlook.com (25.162.16.142) X-Microsoft-Exchange-Diagnostics: 1;VI1PR08MB0448;2:BBFHW3gfNjwv5xg77zA19jaXunxqzF3qR6Bu2opuGgdtiA+FvRcXgOByMt49nAAElHqEenayrkrDY9oxAzhSJMRxdwEd4cLD3PJErrQ1Fu7miT5VKI8gHb87IStFPuuAVzvYYCmb4/VJwRoB4MDFhg==;3:fkOukLnBo495ALxkPXQoTENfpTBvYUxm99VkZuME4h8CwHaXfj5jNh/zjieXMQRBqSXTEah2laRuuQasWMNb8E4G2VKZwg1OoDcBblGAyHOfCC/4m5uoI8ISLEsKKVXC;25:6INpnXXUH+Oo01sqMuGIVTubEyBWG+b3JXk3nTfTBZGYN7ikhm+UEfARIj4S0q4NPbwkhVBW8Tnh5s8UgO2R90nYH+KpaybenLl8KT2JOVPf/nxvPFcUsTxPVvtAT3WcmLhniljAr1M3r+t3q+tlQlU+rv8qfHBtHE2nNKkvaUavsi6GkfQl765c0WYHKqWpaUGdejiRvOvEmz5ow7UL1abXhC63jvWS2yKUu34UjZPq/0c7IDvbqAiSiQ12zfgjh053ssHd27zN9sdyXG48xX8VWmESN11We8/G6PwpAtBdCeLFCTWr4/bU23baJoIqOuZakxVDK/aiyrFhJlSS+8e03yZEIJVkMRbx194oHGM=;20:OioSv60lh8ZwiJR6Fp/v9XgvJfyY+DtEi1ugvPoUHSfTn0bBEGMrD1Kb0ANb/+3fR3eJdD0t27/2SbsgjhrsgOcgkxaTvke0IAh426ZVLGV4Wza2wguIMAWuGwPIPjCyZ44oE5M5BKIf+UGe9x1oEwo9IIJPYXta67vBvhbHcrmV8+6A2uiyTZWaORyISgscTQ3WYatt+PY2sNGTMKuwoYtignXpXKJFELfgMhIZcOqI02xafSplKNaRHpB8zRob X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:VI1PR08MB0448; X-MS-Office365-Filtering-Correlation-Id: a5cf9e6d-d6cf-48d0-0c37-08d33bf991ee X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(601004)(2401047)(8121501046)(5005006)(10201501046)(3002001);SRVR:VI1PR08MB0448;BCL:0;PCL:0;RULEID:;SRVR:VI1PR08MB0448; X-Microsoft-Exchange-Diagnostics: 1;VI1PR08MB0448;4:8ru8CBG7KE24qYTHGm2RINFGESDOr+BCTgQBaCtp5meSR3DChbdZ4Sc8lp1IYQnyikryU5caZBo06U9MljSeZKIgSZ+2yvM3Vp5YvletqtCdMMmq+26BSiZNch7e+y39PAT2JwHREvOsmH5gJhM9Y8DXczCQhvjma/VTsi7QE5CyzQzfs3TCeKdM/SMiTx/fCYWaVduSYkHovRRwJjzGPEN72iH3AxYbt1uQXsYKpEtpLGi4oPcMyewZHhGJmylaxgOaRnt/ka9QNY8Pl+Dtfos9l+6edqqEBPMiHGJYh6b2zimonO6aUiHV547G3Oxzdmdf0KVdkQZ401ZwBWeus+qvwND567k35nPcDuJNbYoPxgIYq/gzE49svB6JOgJs X-Forefront-PRVS: 08617F610C X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10009020)(4630300001)(6009001)(24454002)(40434004)(40100003)(87976001)(122386002)(54356999)(86362001)(76176999)(50986999)(42186005)(97756001)(47776003)(5004730100002)(5008740100001)(6116002)(586003)(66066001)(92566002)(1076002)(23726003)(33656002)(1096002)(3846002)(189998001)(4326007)(15975445007)(46406003)(50466002)(4001350100001)(83506001)(19580395003)(110136002)(19580405001)(2950100001)(2906002)(5890100001)(77096005);DIR:OUT;SFP:1101;SCL:1;SRVR:VI1PR08MB0448;H:sha-win-210.asiapac.arm.com;FPR:;SPF:None;MLV:sfv;LANG:en; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;VI1PR08MB0448;23:YiEJyhVyg1G3ZUdCXf4//0Zgb0/XpxC4pU/F/dgpP?= =?us-ascii?Q?BUvo0k3Pvfo67UuOM4yY5+PTNJydwb5n1KHtf/p0XmoFTlct0Xo8bcodlUqf?= =?us-ascii?Q?ZNS0tj8Y6A8+whEdRdxlhPobqzKAICF8NEFeomAgSkYGtr4TglXvz4IJonoa?= =?us-ascii?Q?Wel7alR6+0a84La3cUlm0bsvzYsApi9yIuopmatqagOwzEfUsCtTqgXHgjRY?= =?us-ascii?Q?ca8ACUBbLKohhk7mRtXUsO3VhHRl+lpcGwZDIuKM42y0WVFyTuE+RUwXPDe+?= =?us-ascii?Q?eMkLrLZmik3tPi/a/tocHGsdnUFX3/AjtKzLD5TonUpBTpmwCcagUMtIxViK?= =?us-ascii?Q?mQn2eeY3EFGgFY0h2JHM1K9WOHItD7+mp9e/UkBGDSILwUxFHpYh7sfljw1a?= =?us-ascii?Q?w4CFdOPkg3NJWDJKWQvePMJpEmbT+LH1GUImKXtm9JeXqNDaJ6TgGCdAnisL?= =?us-ascii?Q?/DltboLgurFhHf6NdjlFlLyuEytjFm45I8roiVtDK/TWP1CouzW59+Me3/vz?= =?us-ascii?Q?QXJ2ij7Q0vHMbIsXnZiz9vrxjcd/3MaSqOKHOFlX5mtB/I+AUlgCpF1cFtlx?= =?us-ascii?Q?JsweyHnI1xEHiIWCsVtpaByhSeZnn57qym9TyxvzhxkmYUTLqENbtTh/c0QM?= =?us-ascii?Q?RyVkcVkvgR2cQt/CXSJiMnYMDygFIog+PxaTtkm0izJXANeNVd0LEVfxyQfe?= =?us-ascii?Q?DwtroZts/1fQfqkgyKb2N+gE9oTCN7yjEdZlGVxzgKFjEzsQJnwUOHxNReeZ?= =?us-ascii?Q?xtFsAmM7b8s90hfbNKM94836xL0E3tOxbSjB2hfDt8n+EzJG67xnIy7BtugD?= =?us-ascii?Q?PZiD+j/UirzUXngYVYOEAahCTqNXbCYxiy/vnSv99NEPdpU/Hvh9Pem3cWdC?= =?us-ascii?Q?msXv+WGSQjoMM7pRZMeO3VTBcKyB1V9CyjJjy52A9prfaig9ot89WT9E3otS?= =?us-ascii?Q?W1qEnejRpXC3bIjLK9o7CI9i+KCN89TKZHvMekuoLlUmMhWHXE76+hf1e0c5?= =?us-ascii?Q?Ql/Q3+lbwMLxeyTD9xHxwMdS0di8BhNAi28hvA/B6YAgBtCb2vyRRq73rp/v?= =?us-ascii?Q?fJrSvE=3D?= X-Microsoft-Exchange-Diagnostics: 1;VI1PR08MB0448;5:O+dy2RPSw39+F9AggApvuijjS3/9xYYbb7s7KK/xcehqWzO2bRGb2QjGUw1H3uJSXxh6NSBEtDufKfHcXFZCsJII3RNXe4vTtp7PPBkbE/bgD1Qvm6iwa3rjgJfOVm6gHgkvVKvKw4cBHXEu9UwvQg==;24:enKHjwGdd4mRwlUocLg5H9MjU2gYDaBXRpaeAzycPR5xRf18Wr2Px5hm8h/TnjhJmI/RkJSAL9wV3wrC3MJa75gQaudHLT4NvFmxviNO+bk=;20:6r1mltzo94gd6ZSZ+9Hv8bZBftp7CqSeGhwE801imkz/DMTzIEGQbnv8pp25PcKjmVdi0uWEvb8UK6jIMAZqCd3BajwoGsaPbtcFlXd7FFpndEI9OkXznw34wylmIAZxIjzwepX3Dnum04SX+nEYgzA4uROi2FbTRx2htY4m5+g= SpamDiagnosticOutput: 1:23 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: arm.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Feb 2016 02:32:23.4938 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR08MB0448 X-MC-Unique: QlFs95m1SbKoav1muG1mDw-1 Content-Type: text/plain; charset=WINDOWS-1252 Content-Transfer-Encoding: 8BIT Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Feb 11, 2016 at 03:33:18PM +0000, Julien Grall wrote: > Hello, > > This small series allows an ARM64 ACPI based platform to use KVM. > > Currently the KVM code has to parse the firmware table to get the necessary > information to setup the virtual timer and virtual GIC. > > However the parsing of those tables are already done in the GIC and arch > timer drivers. > > This patch series introduces different helpers to retrieve the information > from different drivers avoiding to duplicate the parsing code. > > Note there is patch series ([1] and [2]) adding support of KVM on ACPI, > although the approach chosen is completely different. The code to parse > the firmware tables are duplicated which I think make more complex to > support new firmware tables. > > See the changes since v1 in the different patches. > > Regards, > > [1] https://lists.cs.columbia.edu/pipermail/kvmarm/2016-February/018482.html > [2] https://lists.cs.columbia.edu/pipermail/kvmarm/2016-February/018355.html > > Julien Grall (6): > KVM: arm/arm64: arch_timer: Gather KVM specific information in a > structure > KVM: arm/arm64: arch_timer: Rely on the arch timer to parse the > firmware tables > irqchip/gic-v2: Gather ACPI specific data in a single structure > irqchip/gic-v2: Parse and export virtual GIC information > irqchip/gic-v3: Parse and export virtual GIC information Beside this patch (I do not have the gic-v3 hardware now), I tested this patch set with my JUNo-r1 board with kvmtool. It works fine. Tested-by: Huang Shijie > KVM: arm/arm64: vgic: Rely on the GIC driver to parse the firmware > tables IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.