From mboxrd@z Thu Jan 1 00:00:00 1970 From: Julien Grall Subject: Re: [RFC PATCH 00/19] xen/arm: Add ITS support Date: Mon, 02 Mar 2015 13:19:24 +0000 Message-ID: <54F4635C.2050200@linaro.org> References: <1425299435-3278-1-git-send-email-vijay.kilari@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1425299435-3278-1-git-send-email-vijay.kilari@gmail.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: vijay.kilari@gmail.com, Ian.Campbell@citrix.com, stefano.stabellini@eu.citrix.com, stefano.stabellini@citrix.com, tim@xen.org, xen-devel@lists.xen.org Cc: Prasun.Kapoor@caviumnetworks.com, vijaya.kumar@caviumnetworks.com, manish.jaggi@caviumnetworks.com List-Id: xen-devel@lists.xenproject.org Hi Vijay, On 02/03/15 12:30, vijay.kilari@gmail.com wrote: > From: Vijaya Kumar K > > Add ITS support for arm. Following major features > are supported > - GICv3 ITS support for arm64 platform > - Supports only single ITS node Why only one ITS node supported? I though Cavium was using 2 ITS... > - LPI descriptors are allocated on-demand > - Only ITS Dom0 is supported Any plan to support guest? > Vijaya Kumar K (19): > xen/arm: add linked list apis > xen/arm: its: Import GICv3 ITS driver from linux > xen/arm: its: Port ITS driver to xen A general comment (I haven't read closely the patches). The GICv3 ITS taken from Linux is modified so heavily (rename function, move out code, dropping unused code...) that your assumption in patch #1 [1] is wrong. At the end of this series it would make impossible to backport patch from Linux. Regards, [1] "This is actual GICv3 ITS driver from Linux. [..] This helps to import any issues found in Linux" -- Julien Grall