[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2] xen/arm: implement GICD_I[S/C]ACTIVER reads


  • To: Julien Grall <julien@xxxxxxx>
  • From: Stefano Stabellini <stefano.stabellini@xxxxxxxxxx>
  • Date: Fri, 17 Apr 2020 09:31:57 -0700 (PDT)
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 149.199.60.83) smtp.rcpttodomain=xen.org smtp.mailfrom=xilinx.com; dmarc=bestguesspass action=none header.from=xilinx.com; dkim=none (message not signed); arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=PdIcjiLhfWOL6ylvSHUwMphztwroU6PYI+7VkROH3aU=; b=cp1ouxUNMPmeRYcOaZxDVSfJNhCICIiSjNvCEXh7WZ4u+EVGOO6mk72wxWa/fOSUDXKrp4zyeo+t7yB5QSTm65l5cF9B4lhV8536gk4PhxNd+4nIukeOUoYDv6VE4fewkmxSRl92f/DCVXUtZmLAevt2MzNU6kDiAfgNheBiIsfS+hUYS/5V04A/edpeuhdF8b9iFq60FrgTycq8yx1mEhrZGm1WhzVCbzvN2JPqFXyspp6nIE6h6HnXnUbgRhSDSgth/j1b6AlZHhc9fsQyeRbwRVeobgOcE1hYdokfJDGUwTym9Aeu7shlH5qmMA+PAu5AWYxHdUAP77MIRNe7iA==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Jc69E2zzooSJ5NNUFHfKRga0w26jQSod0NMW7MEehq2c4NSwg6RTrMxS/ITpafRw9pTHdpPORMdhqWUEUjLs506TlJKJmLSbYB/F/0+QWJK+TIekciiHQJowXSTZW2QiwzNjhOGrEAyGj4cCMF34fMqgUj7zXtW1L+CtO6YoQwpaGZViBpJ1XW7Qbnucr13Nddbf7AIqJwC5OMiKJJdD2WcjOT8g5Lu7KTRI2lsEpM/f+WzNLnOG1Az5ncICubIJn9BexWeBCaSHJ5i0igluJGg1lDSY8ZotbVsu6kuFTG78d8dNu+hDc/3Rg0pPYMZxF3SZ9zpdSKybaE/IOu+OLw==
  • Authentication-results: spf=pass (sender IP is 149.199.60.83) smtp.mailfrom=xilinx.com; xen.org; dkim=none (message not signed) header.d=none;xen.org; dmarc=bestguesspass action=none header.from=xilinx.com;
  • Cc: Peng Fan <peng.fan@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, "maz@xxxxxxxxxx" <maz@xxxxxxxxxx>, George Dunlap <George.Dunlap@xxxxxxxxxx>, Wei Xu <xuwei5@xxxxxxxxxxxxx>, Bertrand Marquis <Bertrand.Marquis@xxxxxxx>, nd <nd@xxxxxxx>, xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Stefano Stabellini <stefano.stabellini@xxxxxxxxxx>, Julien Grall <julien.grall.oss@xxxxxxxxx>
  • Delivery-date: Fri, 17 Apr 2020 16:32:12 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On Fri, 17 Apr 2020, Julien Grall wrote:
> Hi Bertrand,
> 
> On 17/04/2020 16:16, Bertrand Marquis wrote:
> > It seems that the problem is a bit bigger then expected and will need more
> > discussion and thinking.
> > I did some research on my side and there are several design possible
> > depending on what should be the goal performance or real-time behaviour
> > (going from just give the status we have to fire a service interrupts when
> > any interrupts is acked by a guest to refresh our internal status).
> > 
> > In the short term, could we not agree to fix the typo by returning always 0
> > and start a discussion on the vgic implementation ?
> 
> I have already pointed out multiple time now ([1], [2]) that I would not
> oppose the temporary solution. I think this is a matter of someone (Stefano?)
> to submit it :).
> 
> Cheers,
> 
> [1] https://lists.xenproject.org/archives/html/xen-devel/2019-11/msg01642.html
> [2] https://lists.xenproject.org/archives/html/xen-devel/2020-04/msg00459.html

I can submit it. Julien, would you prefer the plain always return 0
patch, or would you prefer if I tried to get the latest ISACTIVER
information (like this patch does) but only for the local processor?



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.