Hi Vijay,

On 19/03/2015 14:37, vijay.kil...@gmail.com wrote:
From: Vijaya Kumar K <vijaya.ku...@caviumnetworks.com>

Add ITS support for arm. Following major features
are supported
  - GICv3 ITS support for arm64 platform
  - Supports multi ITS node
  - LPI descriptors are allocated on-demand
  - Only ITS Dom0 is supported

Tested with single ITS node.

Some though about the whole design:

Your vGIC ITS driver does too much things. In general a virtual driver should only emulate the hardware for the domain and forward the request to the physical driver.

Your series adds device management (create/free) in the vITS, which is wrong.

How do you check if the domain can use the device?
Currently, you allow any domain to use any device. That would bring a big mess with guest using passthrough.

Also, does the guess will always pass the correct devid? If not how do you plan to handle it?

Regards,

--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Reply via email to