From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:57336) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Ywn1v-0001qN-FL for qemu-devel@nongnu.org; Mon, 25 May 2015 03:42:39 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Ywn1q-0000kJ-BP for qemu-devel@nongnu.org; Mon, 25 May 2015 03:42:35 -0400 Received: from mailout1.w1.samsung.com ([210.118.77.11]:22471) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Ywn1q-0000jS-4s for qemu-devel@nongnu.org; Mon, 25 May 2015 03:42:30 -0400 Received: from eucpsbgm2.samsung.com (unknown [203.254.199.245]) by mailout1.w1.samsung.com (Oracle Communications Messaging Server 7.0.5.31.0 64bit (built May 5 2014)) with ESMTP id <0NOW00D9KAQQQR90@mailout1.w1.samsung.com> for qemu-devel@nongnu.org; Mon, 25 May 2015 08:42:26 +0100 (BST) From: Pavel Fedin References: <1430921082-16779-1-git-send-email-shlomopongratz@gmail.com> <1430921082-16779-5-git-send-email-shlomopongratz@gmail.com> <555B4B0C.1070506@linaro.org> In-reply-to: Date: Mon, 25 May 2015 10:42:25 +0300 Message-id: <008c01d096be$583d7640$08b862c0$@samsung.com> MIME-version: 1.0 Content-type: text/plain; charset=us-ascii Content-transfer-encoding: 7bit Content-language: ru Subject: Re: [Qemu-devel] [PATCH RFC V2 4/4] Add virtv2 machine that uses GIC-500 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: 'Shlomo Pongratz' , 'Eric Auger' , shlomopongratz@gmail.com, qemu-devel@nongnu.org Cc: peter.maydell@linaro.org, ashoks@broadcom.com Hello! > I accept that virtv2 is not needed and I'm currently using Pavel's patch > https://lists.gnu.org/archive/html/qemu-devel/2015-05/msg02930.html with small > modifications. > However If there is a consensus I'll move to Ashok's virt instead of Pavel's one. > > I'm looking forward for a decision regarding which virt is to be used. I have just tried Ashok's approach and unfortunately it does not work well. The problem is mc->max_cpus. Ashok's patch does not do anything with it because he seemed to be satisfied with the maximum of 8. And it looks like CPU number limitation is evaluated before machine is instantiated and properties are evaluated. I tried to change this value inside virt_set_gic_version() (see Ashok's code), but it simply does not work, and i still get " Maximum CPUs greater than specified machine type limit" from qemu. I believe there can be some fix, but it seems to require much more changes than simply subclassing a machine. Does it worth that? And, by the way, libvirt has now added recognition of "virt-" prefix too. Kind regards, Pavel Fedin Expert Engineer Samsung Electronics Research center Russia