From mboxrd@z Thu Jan 1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:51864)
by lists.gnu.org with esmtp (Exim 4.71)
(envelope-from
) id 1ZL8EZ-0007CM-GY
for qemu-devel@nongnu.org; Fri, 31 Jul 2015 07:12:16 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
(envelope-from ) id 1ZL8EU-0002Ab-CE
for qemu-devel@nongnu.org; Fri, 31 Jul 2015 07:12:15 -0400
Received: from mailout4.w1.samsung.com ([210.118.77.14]:57920)
by eggs.gnu.org with esmtp (Exim 4.71)
(envelope-from ) id 1ZL8EU-00027G-6R
for qemu-devel@nongnu.org; Fri, 31 Jul 2015 07:12:10 -0400
Received: from eucpsbgm2.samsung.com (unknown [203.254.199.245])
by mailout4.w1.samsung.com
(Oracle Communications Messaging Server 7.0.5.31.0 64bit (built May 5
2014)) with ESMTP id <0NSC007Q5N44VQ80@mailout4.w1.samsung.com> for
qemu-devel@nongnu.org; Fri, 31 Jul 2015 12:12:05 +0100 (BST)
From: Pavel Fedin
References:
<1dbab1e73e54f0fca4d6d05d50986d2dd396f867.1437731107.git.p.fedin@samsung.com>
<026901d0cb73$e4708000$ad518000$@samsung.com>
In-reply-to:
Date: Fri, 31 Jul 2015 14:12:02 +0300
Message-id: <02bd01d0cb81$bace5b10$306b1130$@samsung.com>
MIME-version: 1.0
Content-type: text/plain; charset=UTF-8
Content-transfer-encoding: quoted-printable
Content-language: ru
Subject: Re: [Qemu-devel] [PATCH v7 5/6] Initial implementation of vGICv3
List-Id:
List-Unsubscribe: ,
List-Archive:
List-Post:
List-Help:
List-Subscribe: ,
To: 'Peter Maydell'
Cc: 'Eric Auger' , 'Marc Zyngier' , 'Shlomo Pongratz' , 'Shlomo Pongratz' , 'QEMU Developers' , 'Christoffer Dall'
Hello!
> >> I intentionally put some tracing to these conditions. On my
> >> system KVM_DEV_ARM_VGIC_GRP_NR_IRQS is supported and
> >> KVM_DEV_ARM_VGIC_CTRL_INIT is not. So will it always be this way?
> >
> > That is weird. I thought that the kernel devs had agreed that all
> > new VGIC code should require explicit initialization (ie that
> > lazy init-on-first-time-it's-needed was only there for legacy
> > guests and the old GICv2). The lazy-init stuff was a big source
> > of bugs where things weren't inited at the point where you
> > expected, which is why it should have gone away...
>=20
> I just checked with Marc, and he agreed that all kernels
> with GICv3 support should support VGIC_CTRL_INIT. What kernel
> are you running?
v3.18 with backported GICv3 support. It is a choice of our HW vendor =
and their current official kernel.
Kind regards,
Pavel Fedin
Expert Engineer
Samsung Electronics Research center Russia