Re: [Xen-devel] [PATCH 2/2 v2] xen: Fix 16550 UART console for HP Moonshot (Aarch64) platform

Hi, On Thursday 16 November 2017 03:26 PM, George Dunlap wrote: On Nov 15, 2017, at 9:20 PM, Konrad Rzeszutek Wilk <konrad.w...@oracle.com> wrote: On Thu, Nov 09, 2017 at 03:49:24PM +0530, Bhupinder Thakur wrote: The console was not working on HP Moonshot (HPE Proliant Aarch64) b

Re: [Xen-devel] [PATCH 1/2 v2] xen: Add support for initializing 16550 UART using ACPI

Hi, On Thursday 09 November 2017 05:01 PM, Roger Pau Monné wrote: On Thu, Nov 09, 2017 at 03:49:23PM +0530, Bhupinder Thakur wrote: Currently, Xen supports only DT based initialization of 16550 UART. This patch adds support for initializing 16550 UART using ACPI SPCR table. This patch also

Re: [Xen-devel] [PATCH 1/2 v2] xen: Add support for initializing 16550 UART using ACPI

Hi Julien, On Tuesday 14 November 2017 12:21 AM, Julien Grall wrote: Hi Bhupinder, On 11/09/2017 10:19 AM, Bhupinder Thakur wrote: Currently, Xen supports only DT based initialization of 16550 UART. This patch adds support for initializing 16550 UART using ACPI SPCR table. This patch also

Re: [Xen-devel] [PATCH 1/4 v3 for-4.10] libxl: Fix the bug introduced in commit "libxl: use correct type modifier for vuart_gfn"

Hi, On 14 Nov 2017 3:35 pm, "Wei Liu" wrote: > On Mon, Nov 13, 2017 at 03:56:23PM +, Julien Grall wrote: > > Hi Wei, > > > > Sorry I missed that e-mail. > > > > On 10/31/2017 10:07 AM, Wei Liu wrote: > > > Change the tag to for-4.10. > > > > > > Julien, this is needed

Re: [Xen-devel] Bringing up OSS test framework on moonshot(aarch64) systems

Hi Ian, On Wednesday 08 November 2017 05:09 PM, Ian Jackson wrote: Bhupinder Thakur writes ("Bringing up OSS test framework on moonshot(aarch64) systems"): While going through [1], I have some queries/doubts on the configuration. H NetNameservers 10.80.248.2 10.80.16.28 1

[Xen-devel] [PATCH 2/2 v2] xen: Fix 16550 UART console for HP Moonshot (Aarch64) platform

ntel.com> Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> --- CC: Andrew Cooper <andrew.coop...@citrix.com> CC: George Dunlap <george.dun...@eu.citrix.com> CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Jan Beulich <jbeul...@suse.com> CC: Konr

[Xen-devel] [PATCH 1/2 v2] xen: Add support for initializing 16550 UART using ACPI

Currently, Xen supports only DT based initialization of 16550 UART. This patch adds support for initializing 16550 UART using ACPI SPCR table. This patch also makes the uart initialization code common between DT and ACPI based initialization. Signed-off-by: Bhupinder Thakur <bhupinder.

[Xen-devel] [PATCH 0/2 v2] xen: ACPI/SPCR based initialization of 8250 UART

: Stefano Stabellini <sstabell...@kernel.org> CC: Tim Deegan <t...@xen.org> CC: Wei Liu <wei.l...@citrix.com> CC: Julien Grall <julien.gr...@arm.com> Bhupinder Thakur (2): xen: Add support for initializing 16550 UART using ACPI xen: Fix 16550 UART console for HP Moonshot

Re: [Xen-devel] [PATCH 1/2] xen: Add support for initializing 16550 UART using ACPI

Hi Julien, On 2 November 2017 at 17:45, Julien Grall <julien.gr...@linaro.org> wrote: > Hi Bhupinder, > > Please write a cover letter even if it is small when your send a series with > multiple patches. > > > On 02/11/17 10:13, Bhupinder Thakur wrote: >> >>

[Xen-devel] Bringing up OSS test framework on moonshot(aarch64) systems

Hi, I am trying to bring up OSS test framework on a couple of moonshot systems which are accessible to me remotely. While going through [1], I have some queries/doubts on the configuration. 1. The following configuration: DnsDomain uk.xensource.com NetNameservers 10.80.248.2 10.80.16.28

[Xen-devel] [PATCH 2/2] xen: Fix 16550 UART console for HP Moonshot (Aarch64) platform

ntel.com> Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> --- CC: Andrew Cooper <andrew.coop...@citrix.com> CC: George Dunlap <george.dun...@eu.citrix.com> CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Jan Beulich <jbeul...@suse.com> CC: Konr

[Xen-devel] [PATCH 1/2] xen: Add support for initializing 16550 UART using ACPI

Currently, Xen supports only DT based initialization of 16550 UART. This patch adds support for initializing 16550 UART using ACPI SPCR table. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> --- CC: Andrew Cooper <andrew.coop...@citrix.com> CC: George Dunlap

[Xen-devel] [PATCH 1/4 v3] libxl: Fix the bug introduced in commit "libxl: use correct type modifier for vuart_gfn"

format specifier "PRIu_xen_pfn" which formats the value as a decimal value. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Wei Liu <wei.l...@citrix.com> CC: Stef

[Xen-devel] [PATCH 2/4 v3] libxl: Change the type of console_mfn to xen_pfn_t

is misleading as it is actually a gfn. This patch also modifies the name to console_gfn. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> --- CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Wei Liu <wei.l...@citrix.com> CC: Stefano Stabellini <sstabell...@kernel.org> CC:

[Xen-devel] [PATCH 3/4 v3] xenconsole: Change the type of ring_ref to xen_pfn_t in console_create_ring

-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Wei Liu <wei.l...@citrix.com> CC: Stefano Stabellini <sstabell...@kernel.org> CC: Julien Grall <julien.gr...@arm.com&

[Xen-devel] [PATCH 4/4 v3] xenconsole: Define and use a macro XEN_INVALID_PFN instead of -1

xenconsole will use a new macro XEN_INVALID_PFN instead of -1 for initializing ring-ref. Since the type of ring_ref is changed to xen_pfn_t (which is an unsigned value) assigning -1 appeared to be confusing. For clarity, XEN_INVALID_PFN is introduced. Signed-off-by: Bhupinder Thakur

Re: [Xen-devel] [PATCH 2/5 v2] libxl: Change the type of console_mfn to xen_pfn_t

Hi, On 26 October 2017 at 16:47, Andrew Cooper <andrew.coop...@citrix.com> wrote: > On 26/10/17 12:13, Wei Liu wrote: >> On Wed, Oct 25, 2017 at 02:57:05PM +0530, Bhupinder Thakur wrote: >>> Currently the type of console mfn is unsigned long in libxl. This may be >>&

Re: [Xen-devel] [PATCH 5/5 v2] xenconsole: Define and use a macro XEN_INVALID_PFN instead of -1

On 27 October 2017 at 12:34, Bhupinder Thakur <bhupinder.tha...@linaro.org> wrote: > Hi Wei, > > On 26 October 2017 at 16:56, Wei Liu <wei.l...@citrix.com> wrote: >> On Wed, Oct 25, 2017 at 02:57:08PM +0530, Bhupinder Thakur wrote: >>> xenconsole will use a new

Re: [Xen-devel] [PATCH 5/5 v2] xenconsole: Define and use a macro XEN_INVALID_PFN instead of -1

Hi Wei, On 26 October 2017 at 16:56, Wei Liu <wei.l...@citrix.com> wrote: > On Wed, Oct 25, 2017 at 02:57:08PM +0530, Bhupinder Thakur wrote: >> xenconsole will use a new macro XEN_INVALID_PFN instead of -1 for >> initializing ring-ref. > > Can you please paste in t

[Xen-devel] [PATCH 2/5 v2] libxl: Change the type of console_mfn to xen_pfn_t

Currently the type of console mfn is unsigned long in libxl. This may be an issue for 32-bit toolstack running on 64-bit Xen, where the pfn are 64 bit. To ensure that console_mfn can hold any valid 64-bit pfn, the type of console_mfn is changed to xen_pfn_t. Signed-off-by: Bhupinder Thakur

[Xen-devel] [PATCH 3/5 v2] libxc: Fix the data type of mfn parameter passed to xc_map_foreign_range()

igned long" to xen_pfn_t. Also the parameter name "mfn" is changed to "pfn" which is a more accurate indication of what this parameter represents. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> --- CC: Ian Jackson <ian.jack...@eu.citrix.com> CC:

[Xen-devel] [PATCH 5/5 v2] xenconsole: Define and use a macro XEN_INVALID_PFN instead of -1

xenconsole will use a new macro XEN_INVALID_PFN instead of -1 for initializing ring-ref. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> --- CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Wei Liu <wei.l...@citrix.com> CC: Stefano Stabellini <sstabell...@k

[Xen-devel] [PATCH 1/5 v2] libxl: Fix the bug introduced in commit "libxl: use correct type modifier for vuart_gfn"

format specifier "PRIu_xen_pfn" which formats the value as a decimal value. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Wei Liu <wei.l...@citrix.com> CC: Stef

[Xen-devel] [PATCH 4/5 v2] xenconsole: Change the type of ring_ref to xen_pfn_t in console_create_ring

-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> --- CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Wei Liu <wei.l...@citrix.com> CC: Stefano Stabellini <sstabell...@kernel.org> CC: Julien Grall <julien.gr...@arm.com> This patch is as per the review of commit

Re: [Xen-devel] [PATCH RFC] ARM: vPL011: use receive timeout interrupt

Hi, On 23 October 2017 at 21:31, Andre Przywara <andre.przyw...@linaro.org> wrote: > Hi, > > On 18/10/17 17:32, Bhupinder Thakur wrote: >> Hi Andre, >> >> I verified this patch on qualcomm platform. It is working fine. >> >> On 18 October 2017 at

Re: [Xen-devel] [PATCH RFC] ARM: vPL011: use receive timeout interrupt

Hi Andre, On 24 October 2017 at 16:57, Andre Przywara <andre.przyw...@linaro.org> wrote: > Hi, > > On 24/10/17 12:00, Julien Grall wrote: >> Hi, >> >> On 23/10/2017 17:01, Andre Przywara wrote: >>> Hi, >>> >>> On 18/10/17 17:32,

Re: [Xen-devel] [PATCH RFC] ARM: vPL011: use receive timeout interrupt

Hi Andre, I verified this patch on qualcomm platform. It is working fine. On 18 October 2017 at 19:11, Andre Przywara wrote: > Instead of asserting the receive interrupt (RXI) on the first character > in the FIFO, lets (ab)use the receive timeout interrupt (RTI) for that

Re: [Xen-devel] [PATCH 27/27 v12] arm/xen: vpl011: Correct the logic for asserting/de-asserting SBSA UART TX interrupt

On 18 October 2017 at 15:56, Andre Przywara <andre.przyw...@arm.com> wrote: > Hi, > > On 13/10/17 11:40, Bhupinder Thakur wrote: >> This patch fixes the issue observed when pl011 patches were tested on >> the junos hardware by Andre/Julien. It was observed that when l

Re: [Xen-devel] [PATCH 26/27 v12] arm/xen: vpl011: Fix the slow early console SBSA UART output

1/2]. And please have a cover letter giving > a brief overview what this series fixes. > Should I resend the patch series with a cover letter? I will also add a reported-by tag. > On 13/10/17 11:40, Bhupinder Thakur wrote: >> The early console output uses pl011_early_write() t

[Xen-devel] xenconsole: Define and use a macro INVALID_XEN_PFN instead of -1

xenconsole will use a new macro INVALID_XEN_PFN instead of -1 for initializing ring-ref. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> --- CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Wei Liu <wei.l...@citrix.com> CC: Andrew Cooper <andrew.coop...@citrix.c

[Xen-devel] libxl: Change the type of console_mfn to xen_pfn_t

Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> --- CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Wei Liu <wei.l...@citrix.com> CC: Stefano Stabellini <sstabell...@kernel.org> CC: Julien Grall <julien.gr...@arm.com> tools/libxl/libxl_console.c |

[Xen-devel] libxl: Fix the bug introduced in commit "libxl: use correct type modifier for vuart_gfn"

format specifier "PRIu_xen_pfn" which formats the value as a decimal value. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Wei Liu <wei.l...@citrix.com> CC: Stef

[Xen-devel] xenconsole: Change the type of ring_ref to xen_pfn_t in console_create_ring

-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> --- CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Wei Liu <wei.l...@citrix.com> CC: Stefano Stabellini <sstabell...@kernel.org> CC: Julien Grall <julien.gr...@arm.com> tools/console/daemon/io.c | 16 ++

[Xen-devel] libxc: Fix the data type of mfn parameter passed to xc_map_foreign_range()

igned long" to xen_pfn_t. Also the parameter name "mfn" is changed to "pfn" which is a more accurate indication of what this parameter represents. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> --- CC: Ian Jackson <ian.jack...@eu.citrix.com> CC:

Re: [Xen-devel] libxl/xenconsole: vpl011: Fix hex to dec conversion of vuart_gfn in libxl__device_vuart_add

On 16 October 2017 at 15:53, Jan Beulich wrote: On 16.10.17 at 11:02, wrote: >> static int console_create_ring(struct console *con) >> { >> - int err, remote_port, ring_ref, rc; >> + int err, remote_port, rc; >> + xen_pfn_t

Re: [Xen-devel] libxl: vpl011: Fix hex to dec conversion of vuart_gfn in libxl__device_vuart_add

On 13 October 2017 at 20:36, Jan Beulich wrote: On 13.10.17 at 16:35, wrote: >> Hi Jan, >> >> On 13/10/17 15:03, Jan Beulich wrote: >> On 13.10.17 at 15:03, wrote: On 13/10/17 13:32, Jan Beulich wrote:

[Xen-devel] libxl/xenconsole: vpl011: Fix hex to dec conversion of vuart_gfn in libxl__device_vuart_add

ng "PRIu_xen_pfn" which formats the value as a decimal value. Also corrected the type of ring_ref to xen_pfn_t in xenconsole. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> --- This patch was verified for arm64 and arm32 toolstack compilation. CC: Ian Jackson <ian

[Xen-devel] libxl: vpl011: Fix hex to dec conversion of vuart_gfn in libxl__device_vuart_add

ng "PRIu_xen_pfn" which formats the value as a decimal value. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> --- CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Wei Liu <wei.l...@citrix.com> CC: Stefano Stabellini <sstabell...@kernel.org> CC: Julien G

[Xen-devel] [PATCH 27/27 v12] arm/xen: vpl011: Correct the logic for asserting/de-asserting SBSA UART TX interrupt

. [1] http://infocenter.arm.com/help/topic/com.arm.doc.ddi0183f/DDI0183.pdf Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> --- CC: Julien Grall <julien.gr...@arm.com> CC: Andre Przywara <andre.przyw...@arm.com> CC: Stefano Stabellini <sstabell...@kernel.org> C

[Xen-devel] [PATCH 26/27 v12] arm/xen: vpl011: Fix the slow early console SBSA UART output

. This will ensure that pl011_early_write() is not delayed unduly to write the data. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> --- CC: Julien Grall <julien.gr...@arm.com> CC: Andre Przywara <andre.przyw...@arm.com> CC: Stefano Stabellini <sstabell...@kerne

Re: [Xen-devel] ARM32 - build-issues with xen/arm: vpl011: Add a new vuart node in the xenstore

On 13 October 2017 at 00:34, Andrew Cooper <andrew.coop...@citrix.com> wrote: > On 12/10/17 19:54, Bhupinder Thakur wrote: >> On 5 October 2017 at 15:07, Wei Liu <wei.l...@citrix.com> wrote: >>> On Wed, Oct 04, 2017 at 09:58:32PM -0400, Konrad Rzeszutek Wilk wrote:

Re: [Xen-devel] ARM32 - build-issues with xen/arm: vpl011: Add a new vuart node in the xenstore

On 5 October 2017 at 15:07, Wei Liu wrote: > On Wed, Oct 04, 2017 at 09:58:32PM -0400, Konrad Rzeszutek Wilk wrote: >> I get this when compiling under ARM32 (Ubuntu 15.04, >> gcc (Ubuntu/Linaro 4.9.2-10ubuntu13) 4.9.2): >> >> libxl_console.c: In function

Re: [Xen-devel] [PATCH 27/27 v11] xen/arm: vpl011: Correct the logic for asserting/de-asserting SBSA UART TX interrupt

On 9 October 2017 at 22:48, Andre Przywara <andre.przyw...@arm.com> wrote: > Hi, > > can you please re-break the commit message to fit into 72 characters? > git show looks rather ugly as it is now. ok. > > On 27/09/17 07:13, Bhupinder Thakur wrote: >> This patch fixes

Re: [Xen-devel] [PATCH 27/27 v10] xen/arm: vpl011: Correct the logic for asserting/de-asserting SBSA UART TX interrupt

On 11 October 2017 at 15:38, Dave Martin <dave.mar...@arm.com> wrote: > On Wed, Oct 11, 2017 at 01:28:44PM +0530, Bhupinder Thakur wrote: >> Hi Dave, >> >> On 26 September 2017 at 20:08, Dave Martin <dave.mar...@arm.com> wrote: >> > On Fri, Sep 22, 2017 a

Re: [Xen-devel] [PATCH 27/27 v10] xen/arm: vpl011: Correct the logic for asserting/de-asserting SBSA UART TX interrupt

Hi Dave, On 26 September 2017 at 20:08, Dave Martin <dave.mar...@arm.com> wrote: > On Fri, Sep 22, 2017 at 01:53:26PM +0530, Bhupinder Thakur wrote: >> This patch fixes the issue observed when pl011 patches were tested on >> the junos hardware by Andre/Julien. It was obs

Re: [Xen-devel] [PATCH v2] xen, tools: console.h shouldn't require string.h by default

ould define the macro. > > Partially revert af8d9356417cb617b635c5ace782388ebfe86e3a. > > Signed-off-by: Wei Liu <wei.l...@citrix.com> > --- > Cc: Julien Grall <julien.gr...@arm.com> > Cc: Bhupinder Thakur <bhupinder.tha...@linaro.org> > Cc: Andrew Coo

Re: [Xen-devel] ARM32 - build-issues with xen/arm: vpl011: Add a new vuart node in the xenstore

Hi Wei, On 5 October 2017 at 15:07, Wei Liu wrote: > On Wed, Oct 04, 2017 at 09:58:32PM -0400, Konrad Rzeszutek Wilk wrote: >> I get this when compiling under ARM32 (Ubuntu 15.04, >> gcc (Ubuntu/Linaro 4.9.2-10ubuntu13) 4.9.2): >> >> libxl_console.c: In function

Re: [Xen-devel] [xen-unstable-smoke test] 114005: regressions - trouble: blocked/fail

Hi, On 4 October 2017 at 17:56, Julien Grall wrote: > > > On 04/10/17 12:47, osstest service owner wrote: >> flight 114005 xen-unstable-smoke real [real] >> http://logs.test-lab.xenproject.org/osstest/logs/114005/ >> >> Regressions :-( >> >> Tests which did not succeed and

[Xen-devel] [PATCH v3] xen/arm: Fix the issue in cmp_mmio_handler used in find_mmio_handler

earch for mmio handler lookup". Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> --- CC: Stefano Stabellini <sstabell...@kernel.org> CC: Julien Grall <julien.gr...@arm.com> This patch may have to be back ported to 4.8 also. xen/arch/arm/io.c | 2 +- 1 fil

[Xen-devel] [PATCH v2] xen/arm: Fix the issue in cmp_mmio_handler used in find_mmio_handler

to be start+size. This bug was introduced when the mmio region search was switched from linear search to binary search in the following commit: 8047e09 "xen/arm: io: Use binary search for mmio handler lookup". This change may have to be back-ported to 4.8 also. Signed-off-by: Bhupin

[Xen-devel] [PATCH 26/27 v11] xen/arm: vpl011: Fix the slow early console SBSA UART output

. This will ensure that pl011_early_write() is not delayed unduly to write the data. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> --- CC: Julien Grall <julien.gr...@arm.com> CC: Andre Przywara <andre.przyw...@arm.com> CC: Stefano Stabellini <sstabell...@kerne

[Xen-devel] [PATCH 21/27 v11] xen/arm: vpl011: Add support for multiple consoles in xenconsole

will not do anything for that console type. This patch is in preparation to support a new vuart console. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Wei Liu <wei.l...@ci

[Xen-devel] [PATCH 24/27 v11] xen/arm: vpl011: Add a pl011 uart DT node in the guest device tree

e firmware Currently the baud rate of 115200 has been selected as a default value, which is one of the valid baud rate settings. Higher baud rate was selected since an emulated pl011 can support any valid baud rate without any limitation of the hardware. Signed-off-by: Bhupinder Thakur <bhupinder.tha.

[Xen-devel] [PATCH 11/27 v11] xen/arm: vpl011: Add a new console_init function in xenconsole

This patch introduces a new console_init function. This function initializes the console structure. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Wei Liu <wei.l...@ci

[Xen-devel] [PATCH 25/27 v11] xen/arm: vpl011: Update documentation for vuart console support

1. Update documentation for a new vuart option added. 2. Update documentation about SPI irq reserved for vuart. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Acked-by: Wei Liu <wei.l...@citrix.com> -

[Xen-devel] [PATCH 23/27 v11] xen/arm: vpl011: Add a new vuart console type to xenconsole client

Add a new console type VUART to connect to guest's emualated vuart console. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian Jackson <ian.jack...@eu.ci

[Xen-devel] [PATCH 27/27 v11] xen/arm: vpl011: Correct the logic for asserting/de-asserting SBSA UART TX interrupt

. [1] http://infocenter.arm.com/help/topic/com.arm.doc.ddi0183f/DDI0183.pdf Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> --- CC: Julien Grall <julien.gr...@arm.com> CC: Andre Przywara <andre.przyw...@arm.com> CC: Stefano Stabellini <sstabell...@kernel.org>

[Xen-devel] [PATCH 12/27 v11] xen/arm: vpl011: Add a new buffer_available function in xenconsole

This patch introduces a new buffer_available function to check if more data is allowed to be buffered. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian

[Xen-devel] [PATCH 05/27 v11] xen/arm: vpl011: Rearrange xen header includes in alphabetical order in domctl.c

Rearrange xen header includes in alphabetical order in domctl.c. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Reviewed-by: Julien Grall <julien.gr...@arm.com> --- CC: Stefano Stabellini <sstabell...@k

[Xen-devel] [PATCH 15/27 v11] xen/arm: vpl011: Add a new console_evtchn_unmask function in xenconsole

channels which had been rate limited for a specified duration. The other for loop was run to add the event channel fd and the tty fd to the poll list. These two for loops were merged so that the these operations can be done in one iteration instead of two iterations. Signed-off-by: Bhupinder

[Xen-devel] [PATCH 22/27 v11] xen/arm: vpl011: Add support for vuart console in xenconsole

ony for ARM. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Wei Liu <wei.l...@citrix.com> CC: Julien Grall <julien.gr...@arm.com> CC: Stefano Stabellini <

[Xen-devel] [PATCH 01/27 v11] xen/arm: vpl011: Define common ring buffer helper functions in console.h

defines DEFINE_XEN_FLEX_RING. In console/daemon/io.c, string.h had to be included before io/console.h because ring.h uses string functions. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Acked-by: Wei Liu <wei.

[Xen-devel] [PATCH 10/27 v11] xen/arm: vpl011: Modify xenconsole functions to take console structure as input

"console_" to indicate that these are console specific functions. This patch is in preparation to support multiple consoles to support vuart console. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Acked-by: Stefano Stabellini <sstabell...@kernel.org> Acked-by: Wei

[Xen-devel] [PATCH 08/27 v11] xen/arm: vpl011: Modify xenconsole to define and use a new console structure

/to console tty. This patch is in preparation to support multiple consoles to support vuart console. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian

[Xen-devel] [PATCH 09/27 v11] xen/arm: vpl011: Rename the console structure field conspath to xspath

The console->conspath name is changed to console->xspath as it is clear from the name that it is referring to xenstore path. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Acked-by: Wei Liu <wei.l...@ci

[Xen-devel] [PATCH 13/27 v11] xen/arm: vpl011: Add a new maybe_add_console_evtchn_fd function in xenconsole

This patch introduces a new maybe_add_console_evtchn_fd function. This function adds the console event channel FD to list of polled FDs. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian Jackson <ian.jack...@eu.citr

[Xen-devel] [PATCH 16/27 v11] xen/arm: vpl011: Add a new handle_console_ring function in xenconsole

as there is no functional change. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Wei Liu <wei.l...@citrix.

[Xen-devel] [PATCH 20/27 v11] xen/arm: vpl011: Add a new console_close_evtchn function in xenconsole

This patch introduces a console_close_evtchn function. This function closes the console event channel. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian

[Xen-devel] [PATCH 03/27 v11] xen/arm: vpl011: Allocate a new GFN in the toolstack for vuart

Allocate a new gfn to be used as a ring buffer between xenconsole and Xen for sending/receiving pl011 console data. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Acked-by: Stefano Stabellini <sstabell...@kernel.org> Acked-by: Wei Liu <wei.l...@citrix.com> -

[Xen-devel] [PATCH 04/27 v11] xen/arm: vpl011: Add support for vuart in libxl

in the guest configuration file: vuart = "sbsa_uart" Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Acked-by: Stefano Stabellini <sstabell...@kernel.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian Jackson <ian.jack...@eu.citrix.com> CC

[Xen-devel] [PATCH 17/27 v11] xen/arm: vpl011: Add a new handle_console_tty function in xenconsole

This patch introduces a new handle_console_tty function. This function performs read/write from/to console tty. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Acked-by: Wei Liu <wei.l...@citrix.com> -

[Xen-devel] [PATCH 06/27 v11] xen/arm: vpl011: Add a new domctl API to initialize vpl011

data on the guest domain's behalf. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Wei Liu <wei.l...@ci

[Xen-devel] [PATCH 02/27 v11] xen/arm: vpl011: Add SBSA UART emulation in Xen

is covered in Appendix B of https://static.docs.arm.com/den0029/a/Server_Base_System_Architecture_v3_1_ARM_DEN_0029A.pdf Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Acked-by: Julien Grall <julien.gr...@arm.com> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org&g

[Xen-devel] [PATCH 18/27 v11] xen/arm: vpl011: Add a new console_cleanup function in xenconsole

This patch introduces a new console_cleanup function. This function frees up the console resources. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian

[Xen-devel] [PATCH 00/27 v11] SBSA UART emulation support in Xen

@citrix.com> CC: Julien Grall <julien.gr...@arm.com> CC: Andre Przywara <andre.przyw...@arm.com> Bhupinder Thakur (27): xen/arm: vpl011: Define common ring buffer helper functions in console.h xen/arm: vpl011: Add SBSA UART emulation in Xen xen/arm: vpl011: Allocate a new G

[Xen-devel] [PATCH 14/27 v11] xen/arm: vpl011: Add a new maybe_add_console_tty_fd function in xenconsole

This patch introduces a new maybe_add_console_tty_fd function. This function adds the tty fd to the list of polled fds. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Acked-by: Wei Liu <wei.l...@citrix.c

[Xen-devel] [PATCH 07/27 v11] xen/arm: vpl011: Add a new vuart node in the xenstore

/receiving vuart data. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Wei Liu <wei.l...@citrix.com> CC

[Xen-devel] [PATCH 19/27 v11] xen/arm: vpl011: Add a new console_open_log function in xenconsole

This patch introduces a console_open_log console_cleanup function. This function opens the console log file. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian

[Xen-devel] [PATCH] xen/arm: Fix the issue in cmp_mmio_handler used in find_mmio_handler

to be start+size. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> --- CC: Stefano Stabellini <sstabell...@kernel.org> CC: Julien Grall <julien.gr...@arm.com> xen/arch/arm/io.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xen/arch/arm/io.c b/xen/a

Re: [Xen-devel] [PATCH 01/27 v10] xen/arm: vpl011: Define common ring buffer helper functions in console.h

Hi Jan, On 26 September 2017 at 12:45, Jan Beulich wrote: On 26.09.17 at 01:08, wrote: >> Yes, after including the __STRICT_ANSI__ check the headers.chk check >> passes. But I had to include string header file (after suggestion from >>

Re: [Xen-devel] [PATCH 01/27 v10] xen/arm: vpl011: Define common ring buffer helper functions in console.h

:49, Jan Beulich <jbeul...@suse.com> wrote: >>>> On 23.09.17 at 00:44, <sstabell...@kernel.org> wrote: >> On Fri, 22 Sep 2017, Bhupinder Thakur wrote: >>> DEFINE_XEN_FLEX_RING(xencons) defines common helper functions such as >>> xencons_queued() to te

Re: [Xen-devel] [PATCH 01/27 v10] xen/arm: vpl011: Define common ring buffer helper functions in console.h

Hi, On 23 September 2017 at 18:55, Julien Grall <julien.gr...@arm.com> wrote: > Hi, > > On 09/22/2017 11:44 PM, Stefano Stabellini wrote: >> >> Adding Jan >> >> On Fri, 22 Sep 2017, Bhupinder Thakur wrote: >>> >>> DEFI

[Xen-devel] [PATCH 27/27 v10] xen/arm: vpl011: Correct the logic for asserting/de-asserting SBSA UART TX interrupt

. [1] http://infocenter.arm.com/help/topic/com.arm.doc.ddi0183f/DDI0183.pdf Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> --- CC: Julien Grall <julien.gr...@arm.com> CC: Andre Przywara <andre.przyw...@arm.com> CC: Stefano Stabellini <sstabell...@kernel.org>

[Xen-devel] [PATCH 25/27 v10] xen/arm: vpl011: Update documentation for vuart console support

1. Update documentation for a new vuart option added. 2. Update documentation about SPI irq reserved for vuart. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Acked-by: Wei Liu <wei.l...@citrix.com> -

[Xen-devel] [PATCH 26/27 v10] xen/arm: vpl011: Fix the slow early console SBSA UART output

. This will ensure that pl011_early_write() is not delayed unduly to write the data. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> --- CC: Julien Grall <julien.gr...@arm.com> CC: Andre Przywara <andre.przyw...@arm.com> CC: Stefano Stabellini <sstabell...@kerne

[Xen-devel] [PATCH 23/27 v10] xen/arm: vpl011: Add a new vuart console type to xenconsole client

Add a new console type VUART to connect to guest's emualated vuart console. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian Jackson <ian.jack...@eu.ci

[Xen-devel] [PATCH 12/27 v10] xen/arm: vpl011: Add a new buffer_available function in xenconsole

This patch introduces a new buffer_available function to check if more data is allowed to be buffered. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian

[Xen-devel] [PATCH 17/27 v10] xen/arm: vpl011: Add a new handle_console_tty function in xenconsole

This patch introduces a new handle_console_tty function. This function performs read/write from/to console tty. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Acked-by: Wei Liu <wei.l...@citrix.com> -

[Xen-devel] [PATCH 24/27 v10] xen/arm: vpl011: Add a pl011 uart DT node in the guest device tree

e firmware Currently the baud rate of 115200 has been selected as a default value, which is one of the valid baud rate settings. Higher baud rate was selected since an emulated pl011 can support any valid baud rate without any limitation of the hardware. Signed-off-by: Bhupinder Thakur <bhupinder.tha.

[Xen-devel] [PATCH 07/27 v10] xen/arm: vpl011: Add a new vuart node in the xenstore

/receiving vuart data. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Wei Liu <wei.l...@citrix.com> CC

[Xen-devel] [PATCH 21/27 v10] xen/arm: vpl011: Add support for multiple consoles in xenconsole

will not do anything for that console type. This patch is in preparation to support a new vuart console. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Wei Liu <wei.l...@ci

[Xen-devel] [PATCH 14/27 v10] xen/arm: vpl011: Add a new maybe_add_console_tty_fd function in xenconsole

This patch introduces a new maybe_add_console_tty_fd function. This function adds the tty fd to the list of polled fds. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Acked-by: Wei Liu <wei.l...@citrix.c

[Xen-devel] [PATCH 16/27 v10] xen/arm: vpl011: Add a new handle_console_ring function in xenconsole

as there is no functional change. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Wei Liu <wei.l...@citrix.

[Xen-devel] [PATCH 05/27 v10] xen/arm: vpl011: Rearrange xen header includes in alphabetical order in domctl.c

Rearrange xen header includes in alphabetical order in domctl.c. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Reviewed-by: Julien Grall <julien.gr...@arm.com> --- CC: Stefano Stabellini <sstabell...@k

[Xen-devel] [PATCH 22/27 v10] xen/arm: vpl011: Add support for vuart console in xenconsole

ony for ARM. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Wei Liu <wei.l...@citrix.com> CC: Julien Grall <julien.gr...@arm.com> CC: Stefano Stabellini <

[Xen-devel] [PATCH 11/27 v10] xen/arm: vpl011: Add a new console_init function in xenconsole

This patch introduces a new console_init function. This function initializes the console structure. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Wei Liu <wei.l...@ci

[Xen-devel] [PATCH 18/27 v10] xen/arm: vpl011: Add a new console_cleanup function in xenconsole

This patch introduces a new console_cleanup function. This function frees up the console resources. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian

[Xen-devel] [PATCH 19/27 v10] xen/arm: vpl011: Add a new console_open_log function in xenconsole

This patch introduces a console_open_log console_cleanup function. This function opens the console log file. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian

[Xen-devel] [PATCH 06/27 v10] xen/arm: vpl011: Add a new domctl API to initialize vpl011

data on the guest domain's behalf. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> --- CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Wei Liu <wei.l...@citrix.com> CC: Andrew Cooper <andrew.coop...@citrix.com> CC: George Dunlap <george.dun...@eu.citrix.

[Xen-devel] [PATCH 08/27 v10] xen/arm: vpl011: Modify xenconsole to define and use a new console structure

/to console tty. This patch is in preparation to support multiple consoles to support vuart console. Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org> Reviewed-by: Stefano Stabellini <sstabell...@kernel.org> Acked-by: Wei Liu <wei.l...@citrix.com> --- CC: Ian

  1   2   3   4   5   >