Re: [Xen-devel] [PATCH v1 2/3] xen/arm: smmu: Renaming arm_smmu_xen_domain with, domain_iommu_info

2015-03-27 Thread Julien Grall
to differentiate domain_iommu_info and iommu_domain. Regards, Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v1 3/3] xen/arm: smmu: Renaming struct iommu_domain *domain to, struct iommu_domain *iommu_domain

2015-03-27 Thread Julien Grall
-by: Julien Grall julien.gr...@linaro.org Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v1 1/3] xen/arm: smmu: Rename arm_smmu_xen_device with, device_iommu_info

2015-03-27 Thread Julien Grall
-archdata.iommu = xzalloc(struct arm_smmu_xen_device); +dev-archdata.iommu = xzalloc(struct device_iommu_info); if (!dev-archdata.iommu) return -ENOMEM; } -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org

Re: [Xen-devel] [PATCH v1 1/3] xen/arm: smmu: Rename arm_smmu_xen_device with, device_iommu_info

2015-03-27 Thread Julien Grall
On 27/03/15 13:21, Jaggi, Manish wrote: Regards, Manish Jaggi Could you please try to configure you email client correctly? It's rather confusing the regards, Manish Jaggi at the beginning of the mail. From: Julien Grall julien.gr...@linaro.org

Re: [Xen-devel] [PATCH v1 2/3] xen/arm: smmu: Renaming arm_smmu_xen_domain with, domain_iommu_info

2015-03-27 Thread Julien Grall
arm_smmu_domain and arm_smmu_xen_domain, I cannot find it based on name. Again, did you read the common above the structure? How do you different domain_iommu_info vs iommu_domain? Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org

Re: [Xen-devel] [PATCH v1 3/3] xen/arm: smmu: Renaming struct iommu_domain *domain to, struct iommu_domain *iommu_domain

2015-03-27 Thread Julien Grall
. Only one was added for our purpose (arm_smmu_xen_domain). If you don't like the name of the 2 others, please complain on the Linux ML. The previous SMMU drivers was diverging from the Linux code and was hard to backport patch. So my nack is not changed on this point. Rgards, -- Julien Grall

Re: [Xen-devel] [PATCH v4 25/33] xen/xsm: Add helpers to check permission for device tree passthrough

2015-03-31 Thread Julien Grall
Hi Daniel, On 31/03/15 18:12, Daniel De Graaf wrote: On 03/19/2015 03:29 PM, Julien Grall wrote: This is a follow-up of commit 525ee49 xsm: add device tree labeling support which add support for device tree labelling in flask. Those helpers will be use latter when non-pci passthrough (i.e

Re: [Xen-devel] [PATCH v4 07/33] xen: guestcopy: Provide an helper to safely copy string from guest

2015-03-31 Thread Julien Grall
ERR_PTR(-ENOMEM); + +if ( copy_from_guest(tmp, u_buf, size) ) +{ +xfree(tmp); +return ERR_PTR(-EFAULT); +} +tmp[size] = 0; '\0' please. Ok. Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel

Re: [Xen-devel] [PATCH v5 0/8] Display IO topology when PXM data is available (plus some cleanup)

2015-03-23 Thread Julien Grall
be the testing value here given that cpu_to_core is not yet correctly implemented on ARM. It would be nice to at least build test it ARM/ARM64 before pushing patches which modify common code. Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel

Re: [Xen-devel] [RFC PATCH v2 00/22] xen/arm: Add ITS support

2015-03-23 Thread Julien Grall
On 23/03/15 12:37, Vijay Kilari wrote: On Fri, Mar 20, 2015 at 9:53 PM, Julien Grall julien.gr...@linaro.org wrote: 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

Re: [Xen-devel] [RFC PATCH v2 06/22] xen/arm: its: Port ITS driver to xen

2015-03-23 Thread Julien Grall
GICR_PIDR2_ARCH_REV_MASK GIC_PIDR2_ARCH_REV_MASK Why this change? GICD_PIDR2_ARCH_REV_MASK still exists... gic-v3.c still uses this. You define GICD_PIDR2_ARCH_REV_MASK with GIC_PIDR2_ARCH_REV_MASK. So the pre-processor will replace by the correct value. Regards, -- Julien Grall

Re: [Xen-devel] [PATCH v5 0/8] Display IO topology when PXM data is available (plus some cleanup)

2015-03-23 Thread Julien Grall
think it's a good solution, building Xen ARM take less than 2 minutes Cheers, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v5 0/8] Display IO topology when PXM data is available (plus some cleanup)

2015-03-23 Thread Julien Grall
Hi, On 23/03/15 13:47, Boris Ostrovsky wrote: On 03/23/2015 08:42 AM, Julien Grall wrote: On 19/03/15 21:53, Boris Ostrovsky wrote: A few patches that add interface for querying hypervisor about device topology and allow 'xl info -n' display this information if PXM object is provided by ACPI

Re: [Xen-devel] [RFC PATCH v2 13/22] xen/arm: its: Add virtual ITS command support

2015-03-23 Thread Julien Grall
); +unsigned long *its_lpi_alloc_chunks(int nirqs, int *base, int *nr_ids); +uint32_t its_get_pta_type(void); +struct its_node * its_get_phys_node(uint32_t dev_id); #endif /* __ASM_ARM_GIC_ITS_H__ */ /* Regards, -- Julien Grall ___ Xen-devel

[Xen-devel] [PATCH v5 p1 04/14] xen/arm: vgic: Introduce a function to initialize pending_irq

2015-04-01 Thread Julien Grall
From: Julien Grall julien.gr...@linaro.org The structure pending_irq is initialized in the same way in 2 different places. Introduce vgic_init_pending_irq to avoid code duplication. Also move the setting of the irq field into this function as we need to initialize it once rather than every time

[Xen-devel] [PATCH v5 p1 09/14] xen/arm: Allow virq != irq

2015-04-01 Thread Julien Grall
From: Julien Grall julien.gr...@linaro.org Currently, Xen is assuming that the virtual IRQ will always be the same as IRQ. Modify route_guest_irq to take the virtual IRQ in parameter which allow Xen to assign a different IRQ number. Also store the vIRQ in the desc action to easily retrieve

[Xen-devel] [PATCH v5 p1 07/14] MAINTAINERS: move drivers/passthrough/device_tree.c in DEVICE TREE

2015-04-01 Thread Julien Grall
From: Julien Grall julien.gr...@linaro.org Suggested-by: Jan Beulich jbeul...@suse.com Signed-off-by: Julien Grall julien.gr...@linaro.org Acked-by: Ian Campbell ian.campb...@citrix.com Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Keir Fraser k...@xen.org --- Changes in v5: - Add

[Xen-devel] [PATCH v5 p1 00/14] xen/arm: Add support for non-PCI passthrough

2015-04-01 Thread Julien Grall
Hello all, This is the first part of the non-PCI passthrough series. It contains all patches acked than can go now in Xen upstream. Another patch series will be sent later in order to fully support non-PCI passthrough. Regards, Julien Grall (14): xen/arm: Divide GIC initialization in 2 parts

[Xen-devel] [PATCH v5 p1 01/14] xen/arm: Divide GIC initialization in 2 parts

2015-04-01 Thread Julien Grall
From: Julien Grall julien.gr...@linaro.org Currently the function to translate IRQ from the device tree is set unconditionally to be able to be able to retrieve serial/timer IRQ before the GIC has been initialized. It assumes that the xlate function won't ever changed. We may also need to have

[Xen-devel] [PATCH v5 p1 06/14] xen/arm: Introduce xen, passthrough property

2015-04-01 Thread Julien Grall
From: Julien Grall julien.gr...@linaro.org When a device is marked for passthrough (via the new property xen,passthrough), dom0 must not access to the device (i.e not loading a driver), but should be able to manage the MMIO/interrupt of the passthrough device. The latter part will allow

Re: [Xen-devel] [PATCH 1/2] xen: arm: log warning for interrupt configuration mismatch

2015-03-03 Thread Julien Grall
Hi Ian, On 02/03/2015 17:01, Ian Campbell wrote: On Mon, 2015-03-02 at 12:56 +, Julien Grall wrote: From 852f6e3fe49f7fab801f2857b8b505922556d746 Mon Sep 17 00:00:00 2001 From: Ian Campbell ian.campb...@citrix.com Date: Mon, 2 Mar 2015 11:09:35 + Subject: [PATCH] xen: arm: Assume

Re: [Xen-devel] [PATCH v5] xen/iommu: smmu: Advertise when the SMMU support coherent table walk

2015-03-03 Thread Julien Grall
On 02/03/2015 17:04, Ian Campbell wrote: On Mon, 2015-03-02 at 15:42 +, Julien Grall wrote: I've just noticed that the support on the previous driver (i.e in Xen 4.5) may incorrectly expose this feature when all the SMMUs is not supporting coherent table walk. I'm

Re: [Xen-devel] [PATCH v3 12/24] xen/arm: Release IRQ routed to a domain when it's destroying

2015-03-03 Thread Julien Grall
Hi Ian, On 20/02/2015 17:41, Julien Grall wrote: +/* Only SPIs are supported */ +if ( virq 32 || virq = vgic_num_irqs(d) ) +return -EINVAL; + +p = irq_to_pending(d-vcpu[0], virq); +if ( !p-desc ) +return -EINVAL; Are we seeing this pattern a lot? It seems so

Re: [Xen-devel] [PATCH v8 3/4] xen/arm: Make gic-v2 code handle hip04-d01 platform

2015-03-03 Thread Julien Grall
= hip04gic_dt_match, +.init = hip04gic_init, DT_DEVICE_END Please keep the same indentation as before. I was wondering why the indentation is different. Ok I'm not sure why ... but it looks like we use the same indentation everywhere for DT_DEVICE_START. Regards, -- Julien Grall

Re: [Xen-devel] [PATCH v8 2/4] xen/arm: Check for interrupt controller directly

2015-03-03 Thread Julien Grall
On 03/03/15 15:07, Frediano Ziglio wrote: 2015-03-03 14:45 GMT+00:00 Julien Grall julien.gr...@linaro.org: Hello Frediano, On 03/03/15 11:19, Frediano Ziglio wrote: This check allow to detect mail interrupt controller even if it does main not match one of the standard ones. This allow

Re: [Xen-devel] [PATCH v8 3/4] xen/arm: Make gic-v2 code handle hip04-d01 platform

2015-03-03 Thread Julien Grall
, DT_DEVICE_END Please keep the same indentation as before. Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

[Xen-devel] [PATCH] xen/arm: gic: Correctly check if an IRQ is valid in gic_interrupt

2015-03-03 Thread Julien Grall
The IRQ 1020 is reserved for special purpose, so the check should be irq 1020 rather than irq 1021. Signed-off-by: Julien Grall julien.gr...@linaro.org --- xen/arch/arm/gic.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xen/arch/arm/gic.c b/xen/arch/arm/gic.c index

[Xen-devel] [PATCH] xen/arm: gic: GICv2 GICv3 only supports 1020 physical interrupts

2015-03-03 Thread Julien Grall
GICD_TYPER.ITLinesNumber can encode up to 1024 interrupts. Although, IRQ 1020-1023 are reserved for special purpose. This helps to check when an IRQ is valid or not. Signed-off-by: Julien Grall julien.gr...@linaro.org --- xen/arch/arm/gic-v2.c | 16 ++-- xen/arch/arm/gic-v3.c | 16

Re: [Xen-devel] [PATCH 1/2] xen: arm: log warning for interrupt configuration mismatch

2015-02-28 Thread Julien Grall
the other possibilities (edge falling and level low) for PPIs. Sorry I haven't spot those errors until now. Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH 2/2] xen: arm: Warn if timer interrupts are not level triggered

2015-02-28 Thread Julien Grall
on this platform. I will send a patch Monday to fix it. Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v4 6/8] xen/iommu: smmu: Add Xen specific code to be able to use the driver

2015-03-02 Thread Julien Grall
Hi Ian, On 02/03/15 13:17, Ian Campbell wrote: On Wed, 2015-02-25 at 18:52 +, Julien Grall wrote: The main goal is to modify as little the Linux code to be able to port easily new feature added in Linux repo for the driver. To achieve that we: - Add helpers to Linux function

Re: [Xen-devel] [RFC PATCH 09/19] xen/arm: Update irq descriptor for LPIs support

2015-03-02 Thread Julien Grall
https://patches.linaro.org/43012/ Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH 1/2] xen: arm: log warning for interrupt configuration mismatch

2015-03-02 Thread Julien Grall
On 02/03/15 13:53, Ian Campbell wrote: On Mon, 2015-03-02 at 13:48 +, Julien Grall wrote: Given that, I was wondering if it would be useful to only store EDGE/LEVEL in the desc-arch.type and, therefore, ignoring rising/falling and low/high. If we don't ever need to know about

Re: [Xen-devel] [PATCH 1/2] xen: arm: log warning for interrupt configuration mismatch

2015-03-02 Thread Julien Grall
). desc-arch.type is only used in Xen for IRQ configuration. Given that, I was wondering if it would be useful to only store EDGE/LEVEL in the desc-arch.type and, therefore, ignoring rising/falling and low/high. That would make the code simpler. Regards, -- Julien Grall

Re: [Xen-devel] [RFC PATCH 00/19] xen/arm: Add ITS support

2015-03-02 Thread Julien Grall
] This is actual GICv3 ITS driver from Linux. [..] This helps to import any issues found in Linux -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH 1/2] xen: arm: log warning for interrupt configuration mismatch

2015-03-02 Thread Julien Grall
Hi Ian, On 02/03/15 11:12, Ian Campbell wrote: On Sat, 2015-02-28 at 22:12 +, Julien Grall wrote: Hi Ian, On 19/02/2015 15:24, Ian Campbell wrote: The ICFGR register is not necessarily writeable, in particular it is IMPLEMENTATION DEFINED for a PPI if the configuration register

Re: [Xen-devel] [PATCH v4 7/8] xen/iommu: smmu: Advertise when the SMMU support coherent table walk

2015-03-02 Thread Julien Grall
prefer to provide a helper function to lookup an smmu by dev-dev. I would like to keep the BUG(). Because this would be a mistake to not find the last SMMU added. I will move to an helper function. Regards, -- Julien Grall ___ Xen-devel mailing

[Xen-devel] [PATCH] xen/arm: timer: Don't warn if the timer interrupts are high-level

2015-03-02 Thread Julien Grall
triggered). As high-level interrupt is valid, don't warn on both high-level and low-level interrupt. Signed-off-by: Julien Grall julien.gr...@linaro.org --- The Seattle DT is using high-level interrupt. Also I haven't tried to boot the latest Xen on it... --- xen/arch/arm/time.c | 2 +- 1 file changed

[Xen-devel] [PATCH v5] xen/iommu: smmu: Advertise when the SMMU support coherent table walk

2015-03-02 Thread Julien Grall
coherent table walk. If not, we may need to enable this feature per-domain. Signed-off-by: Julien Grall julien.gr...@linaro.org --- I've just noticed that the support on the previous driver (i.e in Xen 4.5) may incorrectly expose this feature when all the SMMUs is not supporting coherent

Re: [Xen-devel] [PATCH v3 08/24] xen/arm: Allow virq != irq

2015-02-27 Thread Julien Grall
irq == virq in case of DOM0. Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] ./configure: line 7058: python-config: command not found after commit 0013245

2015-03-03 Thread Julien Grall
On 03/03/2015 18:48, Wei Liu wrote: On Tue, Mar 03, 2015 at 05:46:08PM +, Julien Grall wrote: On 03/03/15 17:36, Wei Liu wrote: On Tue, Mar 03, 2015 at 05:22:39PM +, Julien Grall wrote: Hi, I've tried to use ./configure on the latest master and got the below error: checking

Re: [Xen-devel] [RFC PATCH 17/19] xen/arm: its: Support ITS interrupt handling

2015-03-03 Thread Julien Grall
... I've already got a patch for making virq != pirq : https://patches.linaro.org/43012/ That would drop the static 8K added because of the changes in irq_desc. Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org

Re: [Xen-devel] [PATCH v3 3/4] xen/arm: Add early printk support for ThunderX platform

2015-03-05 Thread Julien Grall
Hello Vijay, On 4 Mar 2015 06:06, vijay.kil...@gmail.com wrote: From: Vijaya Kumar K vijaya.ku...@caviumnetworks.com ThunderX platform uses pl011 uart. You need to update docs/misc/arm/early-printk.txt. Regards, Signed-off-by: Vijaya Kumar K vijaya.ku...@caviumnetworks.com Acked-by: Ian

Re: [Xen-devel] [PATCH 5/5] xen: arm: handle PCI DT node ranges and interrupt-map properties

2015-03-05 Thread Julien Grall
Hi Ian, On 05/03/2015 14:43, Ian Campbell wrote: On Wed, 2015-02-18 at 15:16 +, Julien Grall wrote: ranges; means there is not translation necessary. But nothing prevent to have a the property ranges set. I don't suppose you know of a system where this translation is needed do you? So

[Xen-devel] ./configure: line 7058: python-config: command not found after commit 0013245

2015-03-03 Thread Julien Grall
brokenly enables -D_FORTIFY_SOURCE... ./configure: line 7058: python-config: command not found no checking for xgettext... /usr/bin/xgettext Shall we require python-config installed on the build-platform? Regards, -- Julien Grall ___ Xen-devel mailing

Re: [Xen-devel] [RFC PATCH 14/19] xen/arm: vits: Map ITS translation space

2015-03-03 Thread Julien Grall
with a virtual GITS_TRANSLATOR address different from the physical translator address. FWIW, a 1:1 mapping won't work for guest. Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] ./configure: line 7058: python-config: command not found after commit 0013245

2015-03-03 Thread Julien Grall
On 03/03/15 17:36, Wei Liu wrote: On Tue, Mar 03, 2015 at 05:22:39PM +, Julien Grall wrote: Hi, I've tried to use ./configure on the latest master and got the below error: checking for python-config... no checking Python.h usability... yes checking Python.h presence... yes checking

Re: [Xen-devel] ./configure: line 7058: python-config: command not found after commit 0013245

2015-03-04 Thread Julien Grall
On 04/03/15 12:35, Ian Campbell wrote: On Wed, 2015-03-04 at 12:27 +, Julien Grall wrote: Hi Ian, On 04/03/15 09:07, Ian Campbell wrote: On Tue, 2015-03-03 at 19:41 +, Julien Grall wrote: python-dev is not installed. Although I have libpython-dev installed. And this used to work I

Re: [Xen-devel] [PATCH v3 13/24] xen/arm: Implement hypercall PHYSDEVOP_{, un}map_pirq

2015-03-04 Thread Julien Grall
. Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v1] xen/arm: Do not allocate pte entries for MAP_SMALL_PAGES

2015-03-03 Thread Julien Grall
Hi Ian, On 03/03/2015 10:27, Ian Campbell wrote: On Tue, 2015-03-03 at 13:28 +0530, Vijay Kilari wrote: On Tue, Feb 24, 2015 at 6:29 PM, Julien Grall julien.gr...@linaro.org wrote: On 24/02/15 10:26, Ian Campbell wrote: On Tue, 2015-02-24 at 09:38 +, Julien Grall wrote: Hi Ian, On 24

Re: [Xen-devel] [RFC PATCH 00/19] xen/arm: Add ITS support

2015-03-03 Thread Julien Grall
Hello Vijay, On 03/03/2015 03:55, Vijay Kilari wrote: On Mon, Mar 2, 2015 at 6:49 PM, Julien Grall julien.gr...@linaro.org wrote: On 02/03/15 12:30, vijay.kil...@gmail.com wrote: From: Vijaya Kumar K vijaya.ku...@caviumnetworks.com Add ITS support for arm. Following major features

Re: [Xen-devel] [PATCH v1] xen/arm: Do not allocate pte entries for MAP_SMALL_PAGES

2015-03-03 Thread Julien Grall
WRITEALLOC will never be set. I think it would be better to have a separate helper (as you suggested on a previous mail). Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v8 2/4] xen/arm: Check for interrupt controller directly

2015-03-03 Thread Julien Grall
mail, I would prefer to introduce a new callback to check if the node is a GIC. Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] RFC: [PATCH 1/3] Enhance platform support for PCI

2015-02-23 Thread Julien Grall
the hypercall pci_device_add is called? What about ACPI case? Does everything necessary live in static table? Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v3 11/13] xen/iommu: smmu: Introduce automatic stream-id-masking

2015-02-23 Thread Julien Grall
upstream ( 3.19)... They reworked a big part of the code (mostly P2M handling) in this version. Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] RFC: [PATCH 1/3] Enhance platform support for PCI

2015-02-23 Thread Julien Grall
On 23/02/15 11:50, Manish Jaggi wrote: On 23/02/15 4:44 pm, Julien Grall wrote: On 23/02/2015 10:59, Manish Jaggi wrote: On 20/02/15 8:09 pm, Ian Campbell wrote: On Fri, 2015-02-20 at 19:44 +0530, Manish Jaggi wrote: Another option might be a new hypercall (assuming one doesn't already

Re: [Xen-devel] [PATCH v3 10/24] xen/arm: gic: Add sanity checks gic_route_irq_to_guest

2015-02-23 Thread Julien Grall
Hi Ian, On 23/02/15 15:20, Ian Campbell wrote: On Fri, 2015-02-20 at 17:28 +, Julien Grall wrote: The priority is controlled by route_irq_to_guest and set statically using GIC_PRI_IRQ. If we decide to hardcoded the priority here, we should drop the parameter on gic_route_irq_guest

Re: [Xen-devel] [PATCH v3 07/24] xen/arm: Introduce xen, passthrough property

2015-02-23 Thread Julien Grall
Hi Ian, On 23/02/15 15:12, Ian Campbell wrote: On Fri, 2015-02-20 at 17:01 +, Julien Grall wrote: diff --git a/docs/misc/arm/device-tree/passthrough.txt b/docs/misc/arm/device-tree/passthrough.txt new file mode 100644 index 000..04645b3 --- /dev/null +++ b/docs/misc/arm/device

Re: [Xen-devel] [PATCH v3 07/24] xen/arm: Introduce xen, passthrough property

2015-02-23 Thread Julien Grall
On 23/02/15 15:15, Ian Campbell wrote: On Fri, 2015-02-20 at 17:03 +, Julien Grall wrote: On 20/02/15 15:42, Ian Campbell wrote: On Tue, 2015-01-13 at 14:25 +, Julien Grall wrote: @@ -919,8 +943,14 @@ static int make_timer_node(const struct domain *d, void *fdt, return res

Re: [Xen-devel] [PATCH v3 11/24] xen/arm: Let the toolstack configure the number of SPIs

2015-02-23 Thread Julien Grall
Hi Ian, On 23/02/15 15:22, Ian Campbell wrote: On Fri, 2015-02-20 at 17:29 +, Julien Grall wrote: On 20/02/15 16:08, Ian Campbell wrote: On Wed, 2015-01-28 at 18:26 +, Stefano Stabellini wrote: +int spi = irq - 32; unsigned int and underflow? No because there is a check

Re: [Xen-devel] [PATCH] xen/arm: Implement dynamic allocation of irq descriptors

2015-02-23 Thread Julien Grall
Hello Vijay, On 23/02/15 13:04, Vijay Kilari wrote: On Thu, Feb 19, 2015 at 7:33 PM, Julien Grall julien.gr...@linaro.org wrote: On 19/02/15 07:17, vijay.kil...@gmail.com wrote: From: Vijaya Kumar K vijaya.ku...@caviumnetworks.com For arm memory for 1024 irq descriptors are allocated

Re: [Xen-devel] [PATCH v3 13/24] xen/arm: Implement hypercall PHYSDEVOP_{, un}map_pirq

2015-02-23 Thread Julien Grall
passthrough). Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v3 18/24] xen/passthrough: iommu_deassign_device_dt: By default reassign device to nobody

2015-02-23 Thread Julien Grall
. Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v3 16/24] xen/passthrough: Introduce iommu_construct

2015-02-23 Thread Julien Grall
Hi Ian, On 23/02/15 15:31, Ian Campbell wrote: On Fri, 2015-02-20 at 17:45 +, Julien Grall wrote: On 20/02/15 16:58, Ian Campbell wrote: On Tue, 2015-01-13 at 14:25 +, Julien Grall wrote: This new function will correctly initialize the IOMMU page table for the current domain. Also

Re: [Xen-devel] [PATCH v3 13/24] xen/arm: Implement hypercall PHYSDEVOP_{, un}map_pirq

2015-02-23 Thread Julien Grall
On 23/02/15 16:04, Ian Campbell wrote: On Mon, 2015-02-23 at 15:53 +, Julien Grall wrote: Hi Ian, On 23/02/15 15:28, Ian Campbell wrote: On Mon, 2015-02-23 at 09:33 +, Jan Beulich wrote: On 20.02.15 at 17:53, ian.campb...@citrix.com wrote: Jan, do you have any feeling for how

Re: [Xen-devel] [PATCH] xen/arm: Implement dynamic allocation of irq descriptors

2015-02-23 Thread Julien Grall
On 23/02/15 16:09, Vijay Kilari wrote: On Mon, Feb 23, 2015 at 9:10 PM, Julien Grall julien.gr...@linaro.org wrote: Hello Vijay, On 23/02/15 13:04, Vijay Kilari wrote: On Thu, Feb 19, 2015 at 7:33 PM, Julien Grall julien.gr...@linaro.org wrote: On 19/02/15 07:17, vijay.kil...@gmail.com

Re: [Xen-devel] [PATCH v3 14/24] xen/dts: Use unsigned int for MMIO and IRQ index

2015-02-23 Thread Julien Grall
Hi Ian, On 20/02/15 16:55, Ian Campbell wrote: On Tue, 2015-01-13 at 14:25 +, Julien Grall wrote: There is no reason to use signed integer for an index. Did you check for now pointless if ( uthing 0 ) which a picky compiler might whinge about? I just checked and we don't have

Re: [Xen-devel] [PATCH v3 15/24] xen/dts: Provide an helper to get a DT node from a path provided by a guest

2015-02-23 Thread Julien Grall
Hi Ian, On 23/02/15 15:30, Ian Campbell wrote: On Tue, 2015-01-13 at 14:25 +, Julien Grall wrote: +/* This limit is used by the hypercalls to restrict the size of the path */ +#define DEVICE_TREE_MAX_PATHLEN 1024 Is this something you've made up or derived from the DT spec/ePAPR etc

Re: [Xen-devel] [PATCH v3 10/24] xen/arm: gic: Add sanity checks gic_route_irq_to_guest

2015-02-23 Thread Julien Grall
On 23/02/15 15:52, Ian Campbell wrote: On Mon, 2015-02-23 at 15:47 +, Julien Grall wrote: Hi Ian, On 23/02/15 15:20, Ian Campbell wrote: On Fri, 2015-02-20 at 17:28 +, Julien Grall wrote: The priority is controlled by route_irq_to_guest and set statically using GIC_PRI_IRQ. If we

Re: [Xen-devel] [PATCH v3 24/24] xl: Add new option dtdev

2015-02-23 Thread Julien Grall
Hi Ian, On 23/02/2015 14:45, Ian Campbell wrote: +is the absolute path in the device tree. Can it be an alias? Right now, no. But I can change it if you think it's useful. Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel

Re: [Xen-devel] [PATCH v2 3/3] xen/arm: Skip parsing psci-0.2 from host device tree

2015-02-23 Thread Julien Grall
-off-by: Vijaya Kumar K vijaya.ku...@caviumnetworks.com Acked-by: Ian Campbell ian.campb...@citrix.com Seems like a candidate for backport to 4.5. I think so. Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http

Re: [Xen-devel] [PATCH v2 4/9] xen: arm: correctly handle vtimer traps from userspace

2015-02-25 Thread Julien Grall
On 25/02/15 14:32, Ian Campbell wrote: On Thu, 2015-02-19 at 15:13 +, Ian Campbell wrote: On Thu, 2015-02-19 at 14:42 +, Julien Grall wrote: Although, I think the debug message in bad_trap is useful to keep. It may be handy to have the HSR and the guest stack trace printed if Xen

Re: [Xen-devel] [PATCH] xen/arm: Handle translated addresses for hardware domains in GICv2

2015-02-25 Thread Julien Grall
, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v2 0/3] arm/arm64: Detect Xen support earlier

2015-02-25 Thread Julien Grall
On 25/02/15 16:34, Will Deacon wrote: On Wed, Feb 25, 2015 at 04:22:33PM +, Julien Grall wrote: Hello, Ping? Any comments from ARM's maintainers on theses patches (at least #2)? I couldn't care less :) The arm64 part is boring in the good sense of the word. I will still need a ack

Re: [Xen-devel] [PATCH v2 0/3] arm/arm64: Detect Xen support earlier

2015-02-25 Thread Julien Grall
Hello, Ping? Any comments from ARM's maintainers on theses patches (at least #2)? Regards, On 18/02/15 13:51, Julien Grall wrote: Hello, This small patch series moves the detection of running on Xen earlier. This is required in order to support earlyprintk via Xen and selecting preferred

[Xen-devel] [PATCH v4 5/8] xen/iommu: arm: Import the SMMU driver from Linux

2015-02-25 Thread Julien Grall
Based on commit e6b5be2be4e30037eb551e0ed09dd97bd00d85d3. It's a verbatim of the Linux SMMU drivers code. No Xen code has yet been added and the file is not built. Compare to the previous drivers it gains support of PCI. Though it will need a bit of plumbing for Xen. Signed-off-by: Julien Grall

[Xen-devel] [PATCH v4 2/8] xen/arm: Introduce a generic way to describe device

2015-02-25 Thread Julien Grall
a new field for this place. Signed-off-by: Julien Grall julien.gr...@linaro.org Acked-by: Jan Beulich jbeul...@suse.com Acked-by: Stefano Stabellini stefano.stabell...@eu.citrix.com CC: Keir Fraser k...@xen.org CC: Andrew Cooper andrew.coop...@citrix.com --- Changes in v4: - Add Jan

[Xen-devel] [PATCH v4 3/8] xen/iommu: Consolidate device assignment ops into a single set

2015-02-25 Thread Julien Grall
of the former. Signed-off-by: Julien Grall julien.gr...@linaro.org Acked-by: Jan Beulich jbeul...@suse.com Acked-by: Stefano Stabellini stefano.stabell...@eu.citrix.com\ Acked-by: Ian Campbell ian.campb...@citrix.com --- Changes in v4: - Add Ian's ack Changes in v3: - Add

[Xen-devel] [PATCH v4 0/8] xen/arm: Resync the SMMU driver with the Linux one

2015-02-25 Thread Julien Grall
summarized below the patch acked/modified: A: Patch acked M: Patch modified in this version Sincerely yours, Julien Grall (8): xen/iommu: arm: Remove temporary the SMMU driver xen/arm: Introduce a generic way to describe device xen/iommu: Consolidate device assignment ops into a single

[Xen-devel] [PATCH v4 4/8] xen/arm: Describe devices supported by a driver with dt_device_match

2015-02-25 Thread Julien Grall
the exisiting compatible field and replace with a dt_match field Signed-off-by: Julien Grall julien.gr...@linaro.org Acked-by: Stefano Stabellini stefano.stabell...@eu.citrix.com Acked-by: Ian Campbell ian.campb...@citrix.com --- Changes in v4: - s/dt_match_node/dt_device_match/ in the commit

[Xen-devel] [PATCH v4 1/8] xen/iommu: arm: Remove temporary the SMMU driver

2015-02-25 Thread Julien Grall
The current SMMU driver has completly diverged. That makes me hard to maintain. Signed-off-by: Julien Grall julien.gr...@linaro.org Acked-by: Stefano Stabellini stefano.stabell...@eu.citrix.com Acked-by: Ian Campbell ian.campb...@citrix.com --- Currently none of the platform used on OSStest

[Xen-devel] [PATCH v4 7/8] xen/iommu: smmu: Advertise when the SMMU support coherent table walk

2015-02-25 Thread Julien Grall
coherent table walk. If not, we may need to enable this feature per-domain. Signed-off-by: Julien Grall julien.gr...@linaro.org --- I've just noticed that the support on the previous driver (i.e in Xen 4.5) may incorrectly expose this feature when all the SMMUs is not supporting coherent

[Xen-devel] [PATCH v4 8/8] DO NOT APPLY xen/iommu: smmu: Changes to support Midway SMMU

2015-02-25 Thread Julien Grall
Basic changes to support SMMUs which protect the network cards. The one for the SATA is more complicate to support because it requires to support stream ID matching. Signed-off-by: Julien Grall julien.gr...@linaro.org --- xen/drivers/passthrough/arm/smmu.c | 32

[Xen-devel] [PATCH v4 6/8] xen/iommu: smmu: Add Xen specific code to be able to use the driver

2015-02-25 Thread Julien Grall
when required the code which comes from Linux. If so a comment has been added with /* Xen: ... */ explaining why it's necessary. The support for PCI has been commented because it's not yet supported by Xen ARM and therefore won't compile. Signed-off-by: Julien Grall julien.gr...@linaro.org

Re: [Xen-devel] [PATCH v3 01/24] xen: Extend DOMCTL createdomain to support arch configuration

2015-02-20 Thread Julien Grall
Hi Ian, On 20/02/15 15:15, Ian Campbell wrote: On Tue, 2015-01-13 at 14:25 +, Julien Grall wrote: On ARM the virtual GIC may differ between each guest (emulated GIC version, number of SPIs...). Those informations are already known at the domain creation This information is already

Re: [Xen-devel] RFC: [PATCH 1/3] Enhance platform support for PCI

2015-02-20 Thread Julien Grall
for dom0. I think xen can enforce the ABI while creating device tree While it's true for Linux, you can't assume that another OS (such as FreeBSD) will parse the device tree in the same order. Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel

[Xen-devel] [PATCH] xen/xsm: Generate the permission in a spec-compliant way

2015-02-20 Thread Julien Grall
-by: Julien Grall julien.gr...@linaro.org --- https://bugs.launchpad.net/ubuntu/+source/mawk/+bug/1159933 This bug impacts every version of Xen built on Debian Jessie when XSM is enabled. It would be interesting to backport it. --- xen/xsm/flask/policy/mkaccess_vector.sh | 6 +++--- 1 file changed, 3

Re: [Xen-devel] [PATCH v3 18/24] xen/passthrough: iommu_deassign_device_dt: By default reassign device to nobody

2015-02-23 Thread Julien Grall
On 20/02/2015 17:04, Ian Campbell wrote: On Tue, 2015-01-13 at 14:25 +, Julien Grall wrote: Currently, when the device is deassigned from a domain, we directly reassign to DOM0. As the device may not have been correctly reset, this may lead to corruption or expose some part of DOM0

Re: [Xen-devel] [PATCH v5 2/4] xen/arm: Add support for DTBs with strange names of Hip04 GICv2

2015-02-24 Thread Julien Grall
been nice add a link to the discussion [1]. It would have avoid us to get back on the archives to find the reason of this patch. Regards, [1] http://lists.xen.org/archives/html/xen-devel/2014-11/msg00507.html -- Julien Grall ___ Xen-devel mailing list

Re: [Xen-devel] [PATCH v5 2/4] xen/arm: Add support for DTBs with strange names of Hip04 GICv2

2015-02-24 Thread Julien Grall
been nice add a link to the discussion [1]. It would have avoid us to get back on the archives to find the reason of this patch. Regards, [1] http://lists.xen.org/archives/html/xen-devel/2014-11/msg00507.html -- Julien Grall ___ Xen-devel mailing list

Re: [Xen-devel] [PATCH v3 06/13] xen/arm: Introduce a generic way to describe device

2015-02-24 Thread Julien Grall
Hi Ian, On 20/02/15 12:22, Ian Campbell wrote: On Fri, 2015-01-30 at 18:49 +, Julien Grall wrote: @@ -2,8 +2,34 @@ #define __ASM_ARM_DEVICE_H #include xen/init.h + +enum device_type +{ +DEV_DT, I suppose no #ifdef here because an empty enum is a bit silly? this code

Re: [Xen-devel] [PATCH] xen/arm: Handle translated addresses for hardware domains in GICv2

2015-02-24 Thread Julien Grall
On 24/02/15 16:01, Ian Campbell wrote: On Mon, 2015-02-23 at 16:30 +, Julien Grall wrote: I'm wondering if it's worth to backport this patch to Xen 4.5? Maybe, I doubt 4.5 is being used on platforms which suffer from this issue, but it's pretty un-intrusive I suppose. There is no reason

Re: [Xen-devel] [PATCH] xen/arm: Handle translated addresses for hardware domains in GICv2

2015-02-24 Thread Julien Grall
done here. Agreed. Just it doesn't make sense to backport new GICv3 patches as we know that without the previous series any Linux = 3.19 won't boot at all. Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http

Re: [Xen-devel] [PATCH v6 3/5] xen/arm: Make gic-v2 code handle hip04-d01 platform

2015-02-26 Thread Julien Grall
- unstable.git;a=commit;h=7ba87910e557b06c589c3c0fbc6757fa664d029e Regards, -- Julien Grall Please quote the correct part when you answer. Is this patch in staging? Not yet. But we should avoid to introduce more mess in the headers. Regards, -- Julien Grall

Re: [Xen-devel] RFC: [PATCH 1/3] Enhance platform support for PCI

2015-02-26 Thread Julien Grall
irq routing to guest. The IRQ injection should not consider a 1:1 mapping between pIRQ and vIRQ. I have a patch which allow virq != pirq: https://patches.linaro.org/43012/ Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org

Re: [Xen-devel] [PATCH v6 1/5] xen/arm: Duplicate gic-v2.c file to support hip04 platform version

2015-02-26 Thread Julien Grall
miss at least one change in the copied GICv2 driver. Regards, -- Julien Grall I think you are referring to changes in staging, right ? Yes. If you already start to diverge from staging that's not good. We may have some patches reaching upstream before this series. You have to keep up-date

Re: [Xen-devel] [RFC] When to use domain creation flag or HVM param?

2015-02-26 Thread Julien Grall
this will help to sort out a good approach for both ARM and x86. Regards, [1] http://lists.xen.org/archives/html/xen-devel/2014-11/msg00522.html [2] http://lists.xen.org/archives/html/xen-devel/2015-01/msg01184.html -- Julien Grall ___ Xen-devel mailing list

Re: [Xen-devel] [PATCH v6 1/5] xen/arm: Duplicate gic-v2.c file to support hip04 platform version

2015-02-26 Thread Julien Grall
driver. Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] RFC: [PATCH 1/3] Enhance platform support for PCI

2015-02-26 Thread Julien Grall
that all falls out from the proper tracking of the vdev_id to pdev_id and from vits to pits for a given domain and the management/tracking of the struct pci_dev. I think this is the right way to go. Though I haven't read the ITS spec closely. Regards, -- Julien Grall

Re: [Xen-devel] [PATCH v6 3/5] xen/arm: Make gic-v2 code handle hip04-d01 platform

2015-02-26 Thread Julien Grall
the commit [1] in your branch. [1] http://xenbits.xen.org/gitweb/?p=people/julieng/xen-unstable.git;a=commit;h=7ba87910e557b06c589c3c0fbc6757fa664d029e Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen

<    2   3   4   5   6   7   8   9   10   11   >