Re: [Xen-devel] [PATCH] ARM: xen: only set pm function ptrs for Xen guests

2013-08-28 Thread Julien Grall
tart; > - > - return 0; > -} > -subsys_initcall(xen_pm_init); > - > static irqreturn_t xen_arm_callback(int irq, void *arg) > { > xen_hvm_evtchn_do_upcall(); > -- Julien Grall -- To unsubscribe from this list: send the line "unsubscribe linux-ker

Re: [PATCH v2] ARM: xen: only set pm function ptrs for Xen guests

2013-08-28 Thread Julien Grall
; + return -ENODEV; > + You should use the macro xen_domain() to check if we are running in a Xen guest. Cheers, -- Julien Grall -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org Mor

Re: [Xen-devel] [PATCH] xen/hvc-console: Make it work with HVM guests.

2013-10-06 Thread Julien Grall
On 09/30/2013 03:45 PM, Konrad Rzeszutek Wilk wrote: On Fri, Sep 27, 2013 at 10:49:37PM +0100, Julien Grall wrote: On 09/27/2013 10:25 PM, Konrad Rzeszutek Wilk wrote: @@ -641,7 +641,20 @@ struct console xenboot_console = { void xen_raw_console_write(const char *str

Re: [Xen-devel] [PATCH v5 02/13] arm: introduce a global dma_ops pointer

2013-09-02 Thread Julien Grall
for the Arndale. I have a compilation error because dma_ops is already used in samsung sound driver (sound/soc/samsung/dma.c). This small fix allow me to built this serie for the Arndale. Do I need to send it separately? ======= commit

Re: [Xen-devel] [PATCH] xen/hvc-console: Make it work with HVM guests.

2013-09-27 Thread Julien Grall
outb(str[i], 0xe9); + } } xen_cpuid_base and outb(0xe9) is x86 specific and won't compile on ARM. You need to add ifdef around. -- Julien Grall -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org Mo

[PATCH] arm: choose debug/uncompress.h include when uncompress debug is disabled

2013-07-15 Thread Julien Grall
Even if uncompress debug is disabled, some board will continue to print information during uncompress step. By using debug/uncompress.h include, all debug output will be disabled. This is usefull in Xen environment for DOM0 because the UART is stolen by Xen. Signed-off-by: Julien Grall

[PATCH] xen/arm: disable cpuidle when linux is running as dom0

2013-07-15 Thread Julien Grall
board instead of dom0. Signed-off-by: Julien Grall --- arch/arm/xen/enlighten.c |7 +++ 1 file changed, 7 insertions(+) diff --git a/arch/arm/xen/enlighten.c b/arch/arm/xen/enlighten.c index 49839d8..a98999f 100644 --- a/arch/arm/xen/enlighten.c +++ b/arch/arm/xen/enlighten.c @@ -24,6

[PATCH] xen/arm: enable PV control for ARM

2013-07-15 Thread Julien Grall
Enable power management from the toolstack for ARM guest. Signed-off-by: Julien Grall --- drivers/xen/Makefile |3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/xen/Makefile b/drivers/xen/Makefile index 2bf461a..a5f12bd 100644 --- a/drivers/xen/Makefile +++ b

[PATCH] xen/control: protect functions with CONFIG_HIBERNATE_CALLBACKS to avoid warning

2013-07-15 Thread Julien Grall
s/xen/manage.c:59:13: warning: 'xen_post_suspend' defined but not used [-Wunused-function] Signed-off-by: Julien Grall --- drivers/xen/manage.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/xen/manage.c b/drivers/xen/manage.c index 412b96c..7680276 100644 --

Re: [Xen-devel] [PATCH] xen/control: protect functions with CONFIG_HIBERNATE_CALLBACKS to avoid warning

2013-07-15 Thread Julien Grall
Forgot the different cc. On 07/15/2013 04:47 PM, Julien Grall wrote: > On 07/15/2013 04:27 PM, Konrad Rzeszutek Wilk wrote: >> On Mon, Jul 15, 2013 at 03:24:35PM +0100, Julien Grall wrote: >>> If CONFIG_HIBERNATE_CALLBACKS is not set gcc will issue warnings: >>>

Re: [Xen-devel] [PATCH] xen/arm: disable cpuidle when linux is running as dom0

2013-07-15 Thread Julien Grall
On 07/15/2013 04:25 PM, Konrad Rzeszutek Wilk wrote: > On Mon, Jul 15, 2013 at 03:21:41PM +0100, Julien Grall wrote: >> When linux is running as dom0, Xen doesn't show the physical cpu but a >> virtual CPU. >> On some ARM SOC (for instance the exynos 5250), linux register

[PATCH] xen/arm: missing put_cpu in xen_percpu_init

2013-07-29 Thread Julien Grall
because xen_percpu_init uses get_cpu but doesn't have the corresponding put_cpu. Signed-off-by: Julien Grall --- arch/arm/xen/enlighten.c |1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/xen/enlighten.c b/arch/arm/xen/enlighten.c index f71c37e..dc9f284 100644 --- a/arch/ar

Re: [PATCH] arm: choose debug/uncompress.h include when uncompress debug is disabled

2013-07-18 Thread Julien Grall
On 17 July 2013 14:25, Stefano Stabellini wrote: > On Mon, 15 Jul 2013, Julien Grall wrote: >> Even if uncompress debug is disabled, some board will continue to print >> information during uncompress step. > > Are you talking about DEBUG_UNCOMPRESS? > Should I read

Re: [PATCH] arm: choose debug/uncompress.h include when uncompress debug is disabled

2013-07-18 Thread Julien Grall
On 07/17/2013 04:11 PM, Russell King - ARM Linux wrote: > On Wed, Jul 17, 2013 at 02:25:38PM +0100, Stefano Stabellini wrote: >> On Mon, 15 Jul 2013, Julien Grall wrote: >>> Even if uncompress debug is disabled, some board will continue to print >>> information during

Re: [PATCH] xen/arm: enable PV control for ARM

2013-07-18 Thread Julien Grall
On 17 July 2013 14:40, Stefano Stabellini wrote: > On Mon, 15 Jul 2013, Julien Grall wrote: >> Enable power management from the toolstack for ARM guest. >> >> Signed-off-by: Julien Grall > > Considering that now we support both ARM and ARM64, could you please > a

Re: [Xen-devel] [PATCH] xen/arm: disable cpuidle when linux is running as dom0

2013-07-18 Thread Julien Grall
On 17 July 2013 14:28, Stefano Stabellini wrote: > On Mon, 15 Jul 2013, Konrad Rzeszutek Wilk wrote: >> On Mon, Jul 15, 2013 at 03:21:41PM +0100, Julien Grall wrote: >> > When linux is running as dom0, Xen doesn't show the physical cpu but a >> > virtual CPU. &g

[PATCH v2] xen/arm: disable cpuidle and cpufreq when linux is running as dom0

2013-07-19 Thread Julien Grall
t the whole board instead of only dom0. Signed-off-by: Julien Grall --- Changes in v2: - Disable cpufreq --- arch/arm/xen/enlighten.c |8 1 file changed, 8 insertions(+) diff --git a/arch/arm/xen/enlighten.c b/arch/arm/xen/enlighten.c index 49839d8..af82792 100644 --- a

[PATCH v2] xen/arm: enable PV control for ARM

2013-07-19 Thread Julien Grall
Enable power management from the toolstack for ARM guest. Signed-off-by: Julien Grall --- Changes in v2: - Don't compile xen/cpu_hotplug.o with ARM64 --- drivers/xen/Makefile |5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/xen/Makefile b/dr

Re: [PATCH v2] xen/arm: enable PV control for ARM

2013-07-21 Thread Julien Grall
On 21 July 2013 15:54, Stefano Stabellini wrote: > On Fri, 19 Jul 2013, Julien Grall wrote: >> Enable power management from the toolstack for ARM guest. >> >> Signed-off-by: Julien Grall >> >> --- >> Changes in v2: >> - Don't compi

[PATCH v3] xen/arm: enable PV control for ARM

2013-07-22 Thread Julien Grall
Enable power management from the toolstack for ARM guest. Signed-off-by: Julien Grall --- Changes in v3: - Fix condition to compile cpu_hotplug.o Changes in v2: - Don't compile xen/cpu_hotplug.o with ARM64 --- drivers/xen/Makefile |5 ++--- 1 file chang

[PATCH] xen/arm64: Don't compile cpu hotplug

2013-07-22 Thread Julien Grall
On ARM64, when CONFIG_XEN=y, the compilation will fail because CPU hotplug is not yet supported with XEN. For now, disable it. Signed-off-by: Julien Grall --- drivers/xen/Makefile |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/xen/Makefile b/drivers/xen/Makefile

[PATCH v4] xen/arm: enable PV control for ARM

2013-07-22 Thread Julien Grall
Enable power management from the toolstack for ARM guest. Signed-off-by: Julien Grall --- Changes in v4: - Divide the patch in 2 distinct parts Changes in v3: - Fix condition to compile cpu_hotplug.o Changes in v2: - Don't compile xen/cpu_hotplug.o with

Re: [PATCH v4] xen/arm: enable PV control for ARM

2013-07-23 Thread Julien Grall
On 07/23/2013 01:32 AM, Konrad Rzeszutek Wilk wrote: > Julien Grall wrote: >> Enable power management from the toolstack for ARM guest. >> >> Signed-off-by: Julien Grall >> >> --- >>Changes in v4: >>- Divide the patch in 2 distinct parts &

[PATCH v5] xen/arm: enable PV control for ARM

2013-07-23 Thread Julien Grall
Enable lifecyle management (reboot, shutdown...) from the toolstack for ARM guest. Signed-off-by: Julien Grall --- Changes in v5: - Rework commit message Changes in v4: - Divide the patch in 2 distinct parts Changes in v3: - Fix condition to compile

Re: [PATCH] xen: clear IRQ_NOAUTOEN and IRQ_NOREQUEST when a VIRQ is bound

2013-04-30 Thread Julien Grall
On 04/30/2013 04:02 PM, Stefano Stabellini wrote: > On Mon, 29 Apr 2013, Julien Grall wrote: >> Reset the IRQ_NOAUTOEN and IRQ_NOREQUEST flags that are enabled by >> default on ARM. If IRQ_NOAUTOEN is set, __setup_irq doesn't call >> irq_startup, that is responsible

[PATCH V2] xen: clear IRQ_NOAUTOEN and IRQ_NOREQUEST

2013-04-30 Thread Julien Grall
o_irq with commit a8636c0 but was missing on all others bind_*_to_irq. Move the clear in xen_irq_info_common_init. On x86, IRQ_NOAUTOEN and IRQ_NOREQUEST are cleared by default, so this commit doesn't impact this architecture. Signed-off-by: Julien Grall --- Changes since v1: - Specify t

[PATCH] xen: clear IRQ_NOAUTOEN and IRQ_NOREQUEST when a VIRQ is bound

2013-04-29 Thread Julien Grall
o_irq with commit a8636c0 but was missing in bind_virq_to_irq. Signed-off-by: Julien Grall --- drivers/xen/events.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/xen/events.c b/drivers/xen/events.c index d8cc812..b0ad226 100644 --- a/drivers/xen/events.c +++ b/drivers/xen/eve

Re: [PATCH linux v3 3/9] xen: introduce xen_vcpu_id mapping

2016-09-07 Thread Julien Grall
s. You can get the MPIDR from both the device tree and ACPI. The firmware table is parsed at boot time and the value is stored in cpu_logical_map(). Regards, -- Julien Grall

Re: [PATCH linux v3 3/9] xen: introduce xen_vcpu_id mapping

2016-09-07 Thread Julien Grall
Hi Vitaly, On 07/09/2016 12:23, Vitaly Kuznetsov wrote: BTW, were you able to try the patch I suggested? In my opinion it would be preferable to fix the immediate SMP issue now and play with MPIDR info later. Not yet sorry. I will see if I can try to today or tomorrow. Cheers, -- Julien

Re: [PATCH linux v3 3/9] xen: introduce xen_vcpu_id mapping

2016-09-02 Thread Julien Grall
for PVHVM guest gets its id from CPUID. With secondary CPUs it is a bit more trickier. Currently, we initialize IPI vectors before these CPUs boot so we can't use CPUID. Use ACPI ids from MADT instead. Signed-off-by: Vitaly Kuznetsov --- Changes since v2: - Use uint32_t for xen_vcpu_id m

[PATCH] xen/events: events_fifo: Don't use {get,put}_cpu() in xen_evtchn_fifo_init()

2017-08-17 Thread Julien Grall
. Signed-off-by: Julien Grall Reported-by: Andre Przywara Fixes: 1fe565517b57 ("xen/events: use the FIFO-based ABI if available") --- drivers/xen/events/events_fifo.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/xen/events/events_fifo.c b/drivers/

[PATCH] xen/events: events_fifo: Don't use {get,put}_cpu() in xen_evtchn_fifo_init()

2017-08-17 Thread Julien Grall
. Signed-off-by: Julien Grall Reported-by: Andre Przywara Fixes: 1fe565517b57 ("xen/events: use the FIFO-based ABI if available") --- drivers/xen/events/events_fifo.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/xen/events/events_fifo.c b/drivers/

Re: [PATCH] xen/events: events_fifo: Don't use {get,put}_cpu() in xen_evtchn_fifo_init()

2017-08-18 Thread Julien Grall
Hi Boris, On 17/08/17 18:36, Boris Ostrovsky wrote: On 08/17/2017 12:14 PM, Julien Grall wrote: When booting Linux as Xen guest with CONFIG_DEBUG_ATOMIC, the following splat appears: [0.002323] Mountpoint-cache hash table entries: 1024 (order: 1, 8192 bytes) [0.019717] ASID allocator

Re: [Xen-devel] [PATCH 2/2] block/xen-blkfront: Handle non-indirect grant with 64KB pages

2015-10-05 Thread Julien Grall
On 05/10/15 17:01, Roger Pau Monné wrote: > El 11/09/15 a les 21.32, Julien Grall ha escrit: >> The minimal size of request in the block framework is always PAGE_SIZE. >> It means that when 64KB guest is support, the request will at least be >> 64KB. >> >> Althoug

Re: [Xen-devel] [PATCH 2/2] block/xen-blkfront: Handle non-indirect grant with 64KB pages

2015-10-06 Thread Julien Grall
Hi Roger, On 06/10/2015 10:39, Roger Pau Monné wrote: El 05/10/15 a les 19.05, Julien Grall ha escrit: On 05/10/15 17:01, Roger Pau Monné wrote: El 11/09/15 a les 21.32, Julien Grall ha escrit: ring_req->u.rw.nr_segments = num_grant; + if (unlik

[PATCH] xen/balloon: Use the correct sizeof when declaring frame_list

2015-10-07 Thread Julien Grall
"xen_pfn_t" in commit 965c0aaafe3e75d4e65cd4ec862915869bde3abd "xen: balloon: use correct type for frame_list". Signed-off-by: Julien Grall Cc: Konrad Rzeszutek Wilk Cc: Boris Ostrovsky Cc: David Vrabel --- drivers/xen/balloon.c | 2 +- 1 file changed, 1 insert

Re: [Xen-devel] [PATCH v3 04/20] xen/grant: Introduce helpers to split a page into grant

2015-08-28 Thread Julien Grall
Hi David, On 20/08/15 10:51, David Vrabel wrote: > On 07/08/15 17:46, Julien Grall wrote: >> Currently, a grant is always based on the Xen page granularity (i.e >> 4KB). When Linux is using a different page granularity, a single page >> will be split between multiple grants.

Re: [Xen-devel] [PATCH v3 11/20] tty/hvc: xen: Use xen page definition

2015-08-28 Thread Julien Grall
Hi David, On 20/08/15 10:55, David Vrabel wrote: > On 07/08/15 17:46, Julien Grall wrote: >> The console ring is always based on the page granularity of Xen. > [...] >> --- a/drivers/tty/hvc/hvc_xen.c >> +++ b/drivers/tty/hvc/hvc_xen.c >> @@ -230,7 +230,7 @@ static

Re: [Xen-devel] [PATCH v3 12/20] xen/balloon: Don't rely on the page granularity is the same for Xen and Linux

2015-08-28 Thread Julien Grall
On 20/08/15 10:59, David Vrabel wrote: > On 07/08/15 17:46, Julien Grall wrote: >> For ARM64 guests, Linux is able to support either 64K or 4K page >> granularity. Although, the hypercall interface is always based on 4K >> page granularity. >> >> With 64K page g

Re: [PATCH v3 15/20] block/xen-blkfront: Make it running on 64KB page granularity

2015-08-28 Thread Julien Grall
On 20/08/15 09:10, Roger Pau Monné wrote: > Hello, Hi, > I have some comments regarding the commit message, IMHO it would be good > that a native English speaker reviews it too. > > El 07/08/15 a les 18.46, Julien Grall ha escrit: >> The PV block protocol is using 4KB page

Re: [PATCH v3 19/20] xen/privcmd: Add support for Linux 64KB page granularity

2015-09-01 Thread Julien Grall
multiple xen_pfn, so we want to get the next xen_pfn for the next iteration. Regards, -- Julien Grall -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordom

Re: [Xen-devel] [PATCH 2/2] block/xen-blkfront: Handle non-indirect grant with 64KB pages

2015-10-12 Thread Julien Grall
On 06/10/15 11:06, Roger Pau Monné wrote: > El 06/10/15 a les 11.58, Julien Grall ha escrit: >> Hi Roger, >> >> On 06/10/2015 10:39, Roger Pau Monné wrote: >>> El 05/10/15 a les 19.05, Julien Grall ha escrit: >>>> On 05/10/15 17:01, Roger Pau Monné wr

[PATCH 3/3] xenbus: Support multiple grants ring with 64KB

2015-10-13 Thread Julien Grall
always be mapped on the first 4KB of each Linux page which make the final ring not contiguous in the memory. This can be fixed by mapping multiple grant in a same Linux page. Signed-off-by: Julien Grall --- Cc: Konrad Rzeszutek Wilk Cc: Boris Ostrovsky Cc: David Vrabel Cc: Stefano Stabellini

[PATCH 0/3] xen/xenbus: Support multiple grants ring with 64KB page

2015-10-13 Thread Julien Grall
Hi all, The support of multiple grants ring was left aside for 64KB page. This series aims to fix it. It's based on xentip/for-linus-4.4. Sincelerely yours, Cc: Boris Ostrovsky Cc: David Vrabel Cc: Konrad Rzeszutek Wilk Cc: "Roger Pau Monné" Cc: Stefano Stabellini Julien G

[PATCH 1/3] xen/xenbus: Rename *RING_PAGE* to *RING_GRANT*

2015-10-13 Thread Julien Grall
Linux may use a different page size than the size of grant. So make clear that the order is actually in number of grant. Signed-off-by: Julien Grall --- Cc: Konrad Rzeszutek Wilk Cc: "Roger Pau Monné" Cc: Boris Ostrovsky Cc: David Vrabel Cc: Stefano Stabellini --- drivers

[PATCH 2/3] xen/grant-table: Add an helper to iterate over a specific number of grants

2015-10-13 Thread Julien Grall
a new helper which take an array of Linux page and a number of grant and will figure out the address of each grant. Signed-off-by: Julien Grall --- Cc: Konrad Rzeszutek Wilk Cc: Boris Ostrovsky Cc: David Vrabel Cc: Stefano Stabellini --- drivers/xen/grant-table.c | 22 ++ in

Re: [Xen-devel] [PATCH] xen-blkback: free requests on disconnection

2015-09-07 Thread Julien Grall
On 07/09/15 07:07, Bob Liu wrote: > Hi Julien, Hi Bob, > On 09/04/2015 09:51 PM, Julien Grall wrote: >> Hi Roger, >> >> On 04/09/15 11:08, Roger Pau Monne wrote: >>> Request allocation has been moved to connect_ring, which is called every >>> time

[PATCH v4 02/20] arm/xen: Drop pte_mfn and mfn_pte

2015-09-07 Thread Julien Grall
They are not used in common code expect in one place in balloon.c which is only compiled when Linux is using PV MMU. It's not the case on ARM. Rather than worrying how to handle the 64KB case, drop them. Signed-off-by: Julien Grall Reviewed-by: Stefano Stabellini --- Cc: Russell

[PATCH v4 08/20] block/xen-blkfront: split get_grant in 2

2015-09-07 Thread Julien Grall
to copy data from persistent grant or indirect grant. Avoid to set it for other use case as it will have no meaning given the page will be split in multiple grant. Provide 2 functions, to setup indirect grant, the other for bio page. Signed-off-by: Julien Grall Acked-by: Roger Pau Monné --- Cc

[PATCH v4 03/20] xen: Add Xen specific page definition

2015-09-07 Thread Julien Grall
page definition. They have exactly the same name but prefixed with XEN_/xen_ prefix. Also modify xen_page_to_gfn to use new Xen page definition. Signed-off-by: Julien Grall Reviewed-by: Stefano Stabellini --- Cc: Konrad Rzeszutek Wilk Cc: Boris Ostrovsky Cc: David Vrabel Changes

[PATCH v4 01/20] net/xen-netback: xenvif_gop_frag_copy: move GSO check out of the loop

2015-09-07 Thread Julien Grall
The skb doesn't change within the function. Therefore it's only necessary to check if we need GSO once at the beginning. Signed-off-by: Julien Grall Acked-by: Wei Liu --- Cc: Ian Campbell Cc: net...@vger.kernel.org Changes in v4: - Add Wei's acked

[PATCH v4 04/20] xen/grant: Introduce helpers to split a page into grant

2015-09-07 Thread Julien Grall
k the compilation. Furthermore, only definition in interface/grant_table.h is required. Signed-off-by: Julien Grall Reviewed-by: David Vrabel Reviewed-by: Stefano Stabellini --- Cc: Konrad Rzeszutek Wilk Cc: Boris Ostrovsky Cc: Thomas Gleixner Cc: Ingo Molnar Cc: "H. Peter Anv

[PATCH v4 05/20] xen/grant: Add helper gnttab_page_grant_foreign_access_ref_one

2015-09-07 Thread Julien Grall
which result to page_mfn not being defined when necessary. Signed-off-by: Julien Grall Reviewed-by: David Vrabel Reviewed-by: Stefano Stabellini --- Cc: Konrad Rzeszutek Wilk Cc: Boris Ostrovsky Changes in v3: - Rename gnttab_page_grant_foreign_access_ref into

[PATCH v4 09/20] xen/biomerge: Don't allow biovec's to be merged when Linux is not using 4KB pages

2015-09-07 Thread Julien Grall
-off-by: Julien Grall Reviewed-by: Stefano Stabellini --- Cc: Konrad Rzeszutek Wilk Cc: Boris Ostrovsky Cc: David Vrabel There is some ideas to check whether two biovec could be merged (see [1]) but it's not critical and can be consider as a performance improvement. Ch

[PATCH v4 00/20] xen/arm64: Add support for 64KB page in Linux

2015-09-07 Thread Julien Grall
k maintainers (Ian or Wei) Julien Grall (20): A net/xen-netback: xenvif_gop_frag_copy: move GSO check out of the loop A arm/xen: Drop pte_mfn and mfn_pte A M L xen: Add Xen specific page definition A M xen/grant: Introduce helpers to split a page into grant A

[PATCH v4 07/20] block/xen-blkfront: Store a page rather a pfn in the grant structure

2015-09-07 Thread Julien Grall
All the usage of the field pfn are done using the same idiom: pfn_to_page(grant->pfn) This will return always the same page. Store directly the page in the grant to clean up the code. Signed-off-by: Julien Grall Acked-by: Roger Pau Monné Reviewed-by: Stefano Stabellini --- Cc: Kon

[PATCH v4 06/20] block/xen-blkfront: Split blkif_queue_request in 2

2015-09-07 Thread Julien Grall
execution path, separate the function in 2. This will also remove one level of tabulation. Signed-off-by: Julien Grall Reviewed-by: Roger Pau Monné --- Cc: Konrad Rzeszutek Wilk Cc: Boris Ostrovsky Cc: David Vrabel Roger, if you really want if can drop the else clause in

[PATCH v4 19/20] xen/privcmd: Add support for Linux 64KB page granularity

2015-09-07 Thread Julien Grall
will have to map multiple Xen PFN in a single Linux page. Note that this solution works on page granularity which is a multiple of 4KB. Signed-off-by: Julien Grall Reviewed-by: David Vrabel --- Cc: Konrad Rzeszutek Wilk Cc: Boris Ostrovsky I kept the hypercall arguments in remap_data to

[PATCH v4 11/20] tty/hvc: xen: Use xen page definition

2015-09-07 Thread Julien Grall
The console ring is always based on the page granularity of Xen. Signed-off-by: Julien Grall Reviewed-by: Stefano Stabellini --- Cc: Greg Kroah-Hartman Cc: Jiri Slaby Cc: David Vrabel Cc: Boris Ostrovsky Cc: linuxppc-...@lists.ozlabs.org Changes in v4: - The ring is always 4K

[PATCH v4 13/20] xen/events: fifo: Make it running on 64KB granularity

2015-09-07 Thread Julien Grall
chunk). That would require more care when we fail to expand the event channel. Signed-off-by: Julien Grall Reviewed-by: David Vrabel Reviewed-by: Stefano Stabellini --- Cc: Konrad Rzeszutek Wilk Cc: Boris Ostrovsky Note I haven't updated the suggestion to reduce the memory

[PATCH v4 10/20] xen/xenbus: Use Xen page definition

2015-09-07 Thread Julien Grall
All the ring (xenstore, and PV rings) are always based on the page granularity of Xen. Signed-off-by: Julien Grall Reviewed-by: David Vrabel Reviewed-by: Stefano Stabellini --- Cc: Konrad Rzeszutek Wilk Cc: Boris Ostrovsky Changes in v3: - Fix errors reported by checkpatch.pl

[PATCH v4 14/20] xen/grant-table: Make it running on 64KB granularity

2015-09-07 Thread Julien Grall
multiple grant. It will require some care with the {Set,Clear}ForeignPage macro. Note that no changes has been made in the x86 code because both Linux and Xen will only use 4KB page granularity. Signed-off-by: Julien Grall Reviewed-by: David Vrabel Reviewed-by: Stefano Stabellini --- Cc: Stefano

[PATCH v4 16/20] block/xen-blkback: Make it running on 64KB page granularity

2015-09-07 Thread Julien Grall
lying on the grant table code. Note that the grant table code is allocating a Linux page per grant which will result to waste 6OKB for every grant when Linux is using 64KB page granularity. This could be improved by sharing the page between multiple grants. Signed-off-by: Julien Grall Acked-by: &

[PATCH v4 12/20] xen/balloon: Don't rely on the page granularity is the same for Xen and Linux

2015-09-07 Thread Julien Grall
the extent_order field to directly allocate/free chunk of the Linux page size. Note that PVMMU is only used for PV guest (which is x86) and the page granularity is always 4KB. Some BUILD_BUG_ON has been added to ensure that because the code has not been modified. Signed-off-by: Julien Grall --- Cc

[PATCH v4 18/20] net/xen-netback: Make it running on 64KB page granularity

2015-09-07 Thread Julien Grall
ng on the grant table code. Signed-off-by: Julien Grall --- Cc: Ian Campbell Cc: Wei Liu Cc: net...@vger.kernel.org Improvement such as support of 64KB grant is not taken into consideration in this patch because we have the requirement to run a Linux using 64KB pages on a non-modified Xen.

[PATCH v4 17/20] net/xen-netfront: Make it running on 64KB page granularity

2015-09-07 Thread Julien Grall
n the grant table code. Note that we allocate a Linux page for each rx skb but only the first 4KB is used. We may improve the memory usage by extending the size of the rx skb. Signed-off-by: Julien Grall Reviewed-by: David Vrabel --- Cc: Konrad Rzeszutek Wilk Cc: Boris Ostrovsky Cc

[PATCH v4 15/20] block/xen-blkfront: Make it running on 64KB page granularity

2015-09-07 Thread Julien Grall
64KB page granularity). Furthermore, in the case of persistent grants we allocate one Linux page per grant although only the first 4KB of the page will be effectively in use. This could be improved by sharing the page with multiple grants. Signed-off-by: Julien Grall Acked-by: Roger Pau Monné

[PATCH v4 20/20] arm/xen: Add support for 64KB page granularity

2015-09-07 Thread Julien Grall
page even though only the first 4KB is used. I don't think this is really important for now as it helps to have the pointer 4KB aligned (XENMEM_add_to_physmap is taking a Xen PFN). Signed-off-by: Julien Grall Reviewed-by: Stefano Stabellini --- Cc: Russell King Stefano, I've dr

Re: [Xen-devel] [PATCH v4 18/20] net/xen-netback: Make it running on 64KB page granularity

2015-09-08 Thread Julien Grall
n the latest linus' master. > On Mon, Sep 07, 2015 at 04:33:56PM +0100, Julien Grall wrote: >> The PV network protocol is using 4KB page granularity. The goal of this >> patch is to allow a Linux using 64KB page granularity working as a >> network backend on a non-modified

Re: [PATCH v4 3/4] irqchip: GIC: Convert to EOImode == 1

2015-09-09 Thread Julien Grall
static_key_slow_dec(&supports_deactivate); > + > + /* >* Initialize zero GIC instance (no multi-GIC support). Also, set GIC >* as default IRQ domain to allow for GSI registration and GSI to IRQ >* number translation (see acpi_register_gsi() and

Re: [PATCH] xen: check return value of xenbus_printf

2015-10-15 Thread Julien Grall
but the backend will supply relative coordinates. > > > I cannot understand If the frontend is not able to write the node "request-abs-pointer" in the xenstore, the backend will always supply relative coordinates. Although, as abs = 1, the frontend will be configured to hand

[PATCH] device property: Don't overwrite addr when failing in device_get_mac_address

2015-09-03 Thread Julien Grall
evice_ functions for determining mac/phy" Signed-off-by: Julien Grall Cc: Jeremy Linton Cc: David S. Miller --- Cc: Greg Kroah-Hartman Cc: net...@vger.kernel.org --- drivers/base/property.c | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/drivers/base/

Re: [Xen-devel] [PATCH] xen-blkback: free requests on disconnection

2015-09-04 Thread Julien Grall
n request allocation and freeing. > > Reported-by: Julien Grall > Signed-off-by: Roger Pau Monné > Cc: Julien Grall > Cc: Konrad Rzeszutek Wilk > Cc: Boris Ostrovsky > Cc: David Vrabel > Cc: xen-de...@lists.xenproject.org The patch is fixing my problem when usin

Re: [Xen-devel] [PATCH 2/2] block/xen-blkfront: Handle non-indirect grant with 64KB pages

2015-10-19 Thread Julien Grall
at: Note that I'm planning to update the commit message to summarize my previous mail. > Acked-by: Roger Pau Monné Thank you, I will try to resend a new version today. Regards, -- Julien Grall -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in th

[PATCH v2 0/2] block/xen-blkfront: Support non-indirect grant with 64KB page granularity

2015-10-19 Thread Julien Grall
szutek Wilk Cc: "Roger Pau Monné" Cc: Boris Ostrovsky Cc: David Vrabel Julien Grall (2): block/xen-blkfront: Introduce blkif_ring_get_request block/xen-blkfront: Handle non-indirect grant with 64KB pages drivers/block/xen-blkfront.c | 230 ++-

[PATCH v2 1/2] block/xen-blkfront: Introduce blkif_ring_get_request

2015-10-19 Thread Julien Grall
The code to get a request is always the same. Therefore we can factorize it in a single function. Signed-off-by: Julien Grall Acked-by: Roger Pau Monné --- Cc: Konrad Rzeszutek Wilk Cc: Boris Ostrovsky Cc: David Vrabel Changes in v2: - Add Royger's acked-by --- drivers/

[PATCH v2 2/2] block/xen-blkfront: Handle non-indirect grant with 64KB pages

2015-10-19 Thread Julien Grall
e_max_* helpers haven't been updated. The block code will set the mimimum size supported and we may be able to support directly any change in the block framework that lower down the minimal size of a request. [1] http://lists.xen.org/archives/html/xen-devel/2015-08/msg02200.html Signed-o

Re: [PATCH] xen: check return value of xenbus_printf

2015-10-19 Thread Julien Grall
rguments should be aligned to the first parameter. I.e: xenbus_printf(XBT_NIL, dev->nodename, "request-abs-pointer", "1"); See an example in xenkbd_backend_changed. With that fixed: Reviewed-by: Julien Grall > + if (ret) { > + pr_warning(

[PATCH] mm: hotplug: Don't release twice the resource on error

2015-10-23 Thread Julien Grall
caller is allocating the resource, let him handle the release. This has been introduced by commit b75351f "mm: memory hotplug with an existing resource". Signed-off-by: Julien Grall --- Cc: David Vrabel Cc: linux...@kvack.org The patch who introduced this issue is in xentip/for

Re: [Xen-devel] [PATCH 0/2] block/xen-blkfront: Support non-indirect with 64KB page granularity

2015-10-02 Thread Julien Grall
Hi, Ping, any comment on this series? Regards, On 11/09/15 20:31, Julien Grall wrote: > Hi all, > > This is a follow-up on the previous discussion [1] related to guest using 64KB > page granularity not booting with backend using non-indirect grant. > > This has been success

Re: [PATCH v5 12/22] xen/balloon: Don't rely on the page granularity is the same for Xen and Linux

2015-10-02 Thread Julien Grall
Hi David, On 02/10/15 15:09, David Vrabel wrote: > On 30/09/15 11:45, Julien Grall wrote: >> For ARM64 guests, Linux is able to support either 64K or 4K page >> granularity. Although, the hypercall interface is always based on 4K >> page granularity. >> >> With 64

Re: [PATCH v5 12/22] xen/balloon: Don't rely on the page granularity is the same for Xen and Linux

2015-10-02 Thread Julien Grall
On 02/10/15 15:31, Julien Grall wrote: > Hi David, > > On 02/10/15 15:09, David Vrabel wrote: >> On 30/09/15 11:45, Julien Grall wrote: >>> For ARM64 guests, Linux is able to support either 64K or 4K page >>> granularity. Although, the hypercall interfa

Re: [PATCH v4] xen/privcmd: add IOCTL_PRIVCMD_MMAP_RESOURCE

2018-04-11 Thread Julien Grall
s is going to work well if the hypervisor is using a different granularity for the page. Imagine Xen is using 4K but the kernel 64K. You would end up to have the resource not mapped contiguously in the memory. Cheers, -- Julien Grall

Re: [Xen-devel] [RFC, v2, 1/9] hyper_dmabuf: initial upload of hyper_dmabuf drv core framework

2018-04-10 Thread Julien Grall
uest? Cheers, -- Julien Grall

Re: [PATCH v2 1/8] xen/events: reset affinity of 2-level event when tearing it down

2021-02-14 Thread Julien Grall
zeroed out). Cc: sta...@vger.kernel.org Reported-by: Julien Grall Signed-off-by: Juergen Gross Reviewed-by: Julien Grall Cheers, --- V2: - reset affinity when tearing down the event (Julien Grall) --- drivers/xen/events/events_2l.c | 15 +++ drivers/xen/events

Re: [PATCH v2 3/8] xen/events: avoid handling the same event on two cpus at the same time

2021-02-14 Thread Julien Grall
t data and call the handler only if this flag isn't set. Reported-by: Julien Grall Signed-off-by: Juergen Gross --- V2: - new patch --- drivers/xen/events/events_base.c | 19 +++ 1 file changed, 15 insertions(+), 4 deletions(-) diff --git a/drivers/xen/events/events_base.

Re: [Xen-devel] [PATCH v3 2/2] x86, arm64, platform, xen, kconfig: add xen defconfig helper

2014-12-09 Thread Julien Grall
don't have PVMMU support on ARM. Shouldn't you move this config in architecture specific code? Regards -- Julien Grall -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at ht

Re: [Xen-devel] [PATCH v3 2/2] x86, arm64, platform, xen, kconfig: add xen defconfig helper

2014-12-09 Thread Julien Grall
On 09/12/14 20:22, Luis R. Rodriguez wrote: > On Tue, Dec 9, 2014 at 1:06 AM, Julien Grall wrote: >> Hello Luis, >> >> On 08/12/2014 23:05, Luis R. Rodriguez wrote: >>> >>> diff --git a/kernel/configs/xen.config b/kernel/configs/xen.config >>>

Re: [Xen-devel] [RFC v3 2/2] x86/xen: allow privcmd hypercalls to be preempted

2015-01-22 Thread Julien Grall
On 22/01/15 18:56, Luis R. Rodriguez wrote: > On Thu, Jan 22, 2015 at 01:10:49PM +0000, Julien Grall wrote: >> Hi Luis, >> >> On 22/01/15 02:17, Luis R. Rodriguez wrote: >>> diff --git a/drivers/xen/events/events_base.c >>> b/drivers/xen/events/events_b

Re: [Xen-devel] [PATCH v2 2/2] x86, arm, platform, xen, kconfig: add xen defconfig helper

2015-01-13 Thread Julien Grall
IG_XEN_XENBUS_FRONTEND=y >>> +CONFIG_XEN_GNTDEV=m >>> +CONFIG_XEN_GRANT_DEV_ALLOC=m >>> +CONFIG_SWIOTLB_XEN=y >>> +CONFIG_XEN_PCIDEV_BACKEND=m >>> +CONFIG_XEN_PRIVCMD=m >>> +CONFIG_XEN_ACPI_PROCESSOR=m >> >> The common fragment config look

Re: [PATCH v10 00/21] Introduce ACPI for ARM64 based on ACPI 5.1

2015-03-22 Thread Julien Grall
+= xen-acpi-pad.o dom0-$(CONFIG_X86) += pcpu.o obj-$(CONFIG_XEN_DOM0)+= $(dom0-y) [..] AFAIK, There is already a kernel patch exists to fix this issue. I think Julien or Parth is a right person to ask. Hence I am CCed Julien Grall too. The ACPI support for Xen is not r

Re: [PATCH v10 00/21] Introduce ACPI for ARM64 based on ACPI 5.1

2015-03-22 Thread Julien Grall
On 22/03/2015 21:49, Rafael J. Wysocki wrote: On Sunday, March 22, 2015 09:05:21 PM Julien Grall wrote: Hello, On 21/03/2015 12:09, Naresh Bhat wrote: From 268dcdafa34a690e2f99c0784ca33a6d2352ecf5 Mon Sep 17 00:00:00 2001 From: Hanjun Guo mailto:hanjun@linaro.org>>

[PATCH v4 0/3] net/xen: Clean up

2015-06-16 Thread Julien Grall
Hi, The first 2 patches was originally part of the the Xen 64KB series [1]. Although, I think they can go, assuming everything will be acked/reviewed, without waiting the rest of the 64KB series. The third patch as been added in this version. Regards, Julien Grall (3): net/xen-netfront

[PATCH v4 2/3] net/xen-netback: Remove unused code in xenvif_rx_action

2015-06-16 Thread Julien Grall
The variables old_req_cons and ring_slots_used are assigned but never used since commit 1650d5455bd2dc6b5ee134bd6fc1a3236c266b5b "xen-netback: always fully coalesce guest Rx packets". Signed-off-by: Julien Grall Acked-by: Wei Liu Cc: Ian Campbell Cc: net...@vger.kernel.org ---

[PATCH v4 1/3] net/xen-netfront: Correct printf format in xennet_get_responses

2015-06-16 Thread Julien Grall
rx->status is an int16_t, print it using %d rather than %u in order to have a meaningful value when the field is negative. Also use %u rather than %x for rx->offset. Signed-off-by: Julien Grall Reviewed-by: David Vrabel Cc: Konrad Rzeszutek Wilk Cc: Boris Ostrovsky Cc: net...@vger.kern

[PATCH v4 3/3] net/xen-netback: Don't mix hexa and decimal with 0x in the printf format

2015-06-16 Thread Julien Grall
Append 0x to all %x in order to avoid while reading when there is other decimal value in the log. Also replace some of the hexadecimal print to decimal to uniformize the format with netfront. Signed-off-by: Julien Grall Cc: Wei Liu Cc: Ian Campbell Cc: net...@vger.kernel.org --- Changes

Re: [PATCH v4 3/3] net/xen-netback: Don't mix hexa and decimal with 0x in the printf format

2015-06-17 Thread Julien Grall
Hi Ian, On 17/06/2015 10:25, Ian Campbell wrote: On Tue, 2015-06-16 at 20:10 +0100, Julien Grall wrote: Append 0x to all %x in order to avoid while reading when there is other decimal value in the log. Also replace some of the hexadecimal print to decimal to uniformize the format with

[PATCH v5 0/3] net/xen: Clean up

2015-06-17 Thread Julien Grall
Cc: net...@vger.kernel.org [1] http://lkml.org/lkml/2015/5/14/533 Julien Grall (3): net/xen-netfront: Correct printf format in xennet_get_responses net/xen-netback: Remove unused code in xenvif_rx_action net/xen-netback: Don't mix hexa and decimal with 0x in the printf format dr

[PATCH v5 3/3] net/xen-netback: Don't mix hexa and decimal with 0x in the printf format

2015-06-17 Thread Julien Grall
Prepend 0x to all %x in order to avoid confusion while reading when there is other decimal value in the log. Also replace some of the hexadecimal print to decimal to uniformize the format with netfront. Signed-off-by: Julien Grall Acked-by: Ian Campbell Cc: Wei Liu Cc: net...@vger.kernel.org

[PATCH v5 2/3] net/xen-netback: Remove unused code in xenvif_rx_action

2015-06-17 Thread Julien Grall
The variables old_req_cons and ring_slots_used are assigned but never used since commit 1650d5455bd2dc6b5ee134bd6fc1a3236c266b5b "xen-netback: always fully coalesce guest Rx packets". Signed-off-by: Julien Grall Acked-by: Wei Liu Cc: Ian Campbell Cc: net...@vger.kernel.org ---

  1   2   3   4   5   6   7   8   >