[Xen-devel] [PATCH v5 1/2] libs/gnttab: introduce grant copy interface

2016-08-15 Thread Paulina Szubarczyk
In a linux part an ioctl(gntdev, IOCTL_GNTDEV_GRANT_COPY, ..) system call is invoked. In mini-os the operation is yet not implemented. For the OSs that does not implement gnttab the call of the grant copy operation causes abort. Signed-off-by: Paulina Szubarczyk Reviewed-by: David Vrabel --- Cha

[Xen-devel] [PATCH v5 2/2] qdisk - hw/block/xen_disk: grant copy implementation

2016-08-15 Thread Paulina Szubarczyk
Copy data operated on during request from/to local buffers to/from the grant references. Before grant copy operation local buffers must be allocated what is done by calling ioreq_init_copy_buffers. For the 'read' operation, first, the qemu device invokes the read operation on local buffers and on

[Xen-devel] [PATCH v4 0/2] qemu-qdisk: Implementation of grant copy operation.

2016-08-15 Thread Paulina Szubarczyk
Hi, It is a proposition for implementation of grant copy operation in qemu-qdisk and interface in libxc/libs. Changes since v3: Interface: - revert to cast from xengnttab_grant_copy_segment_t to ioctl_gntdev_grant_copy. - added compile-time check to compare the libs xengnttab_grant_copy_seg

Re: [Xen-devel] [PATCH 3/3] Significant changes to decision making; some new roles and minor changes

2016-08-15 Thread Tim Deegan
Hi, At 14:55 + on 15 Aug (1471272946), Lars Kurth wrote: > But I see your point. The text should really have said something like... > - > In situations where the entire Xen Project community becomes paralysed, > the project leaderships team or project lead should work with the > community

[Xen-devel] [qemu-mainline baseline-only test] 67537: tolerable FAIL

2016-08-15 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 67537 qemu-mainline real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/67537/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-qemuu-nested-intel 16 debian-hvm-insta

Re: [Xen-devel] [RFC v3 00/13] linux: generalize sections, ranges and linker tables

2016-08-15 Thread Alan Cox
> This is the module tag ... it says what licence the module is under, > not the licence for the module combined with the kernel, which is > always GPLv2 because the stricter licence rules. Because if I build a BSD licensed module against the kernel, give you the binaries and refuse to give you t

[Xen-devel] [xen-4.5-testing test] 100496: regressions - FAIL

2016-08-15 Thread osstest service owner
flight 100496 xen-4.5-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/100496/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qcow2 9 debian-di-installfail REGR. vs. 100338 test-amd64-i386

[Xen-devel] [PATCH] arm64: head: Fill image size

2016-08-15 Thread Peng Fan
When booting xen from U-Boot, U-Boot will use the image size info. Because this information is lacked in XEN image,U-Boot assume the image size is 16MB to memmove, which will cost lots time on simulation platform. Signed-off-by: Peng Fan Cc: Stefano Stabellini Cc: Julien Grall --- xen/arch/arm

Re: [Xen-devel] [PATCH v2 23/23] libxc/xc_dom_core: Copy ACPI tables to guest space

2016-08-15 Thread Shannon Zhao
On 2016/8/15 20:49, Boris Ostrovsky wrote: > On 08/15/2016 03:48 AM, Shannon Zhao wrote: >> Hi Boris >> >> On 2016/8/5 5:06, Boris Ostrovsky wrote: >>> Load ACPI modules into guest space >>> >>> Signed-off-by: Boris Ostrovsky >>> --- >>> v2: >>> * New patch, loosely based on Shannon's ARM patch

[Xen-devel] [xen-4.6-testing test] 100495: regressions - FAIL

2016-08-15 Thread osstest service owner
flight 100495 xen-4.6-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/100495/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl-xsm 6 xen-boot fail REGR. vs. 100352 Regressions whi

Re: [Xen-devel] [RFC 05/22] xen/arm: traps: Move MMIO emulation code in a separate helper

2016-08-15 Thread Stefano Stabellini
On Thu, 28 Jul 2016, Julien Grall wrote: > Currently, a stage-2 fault translation will likely access an emulated > region. All the checks are pre-sanitity check for MMIO emulation. > > A follow-up patch will handle a new case that could lead to a stage-2 > translation. To improve the clarity of th

Re: [Xen-devel] [RFC 04/22] xen/arm: p2m: Use typesafe gfn in p2m_mem_access_radix_set

2016-08-15 Thread Stefano Stabellini
On Thu, 28 Jul 2016, Julien Grall wrote: > p2m_mem_access_radix_set is expecting a gfn in a parameter. Rename the > parameter 'pfn' to 'gfn' to match its content and use the typesafe gfn > to avoid possible misusage. > > Also rename the parameter to gfn to match its content. > > Signed-off-by: Ju

Re: [Xen-devel] [RFC 03/22] xen/arm: p2m: Rename parameter in p2m_{remove, write}_pte...

2016-08-15 Thread Stefano Stabellini
On Thu, 28 Jul 2016, Julien Grall wrote: > to make clear of the usage. I.e it is used to inform whether Xen needs > to clean the entry after writing in the page table. > > Signed-off-by: Julien Grall Acked-by: Stefano Stabellini > xen/arch/arm/p2m.c | 8 > 1 file changed, 4 insertio

Re: [Xen-devel] [RFC 02/22] xen/arm: p2m: Store in p2m_domain whether we need to clean the entry

2016-08-15 Thread Stefano Stabellini
On Thu, 28 Jul 2016, Julien Grall wrote: > Each entry in the page table has to table clean when the IOMMU does not What does it mean to "table clean" ? > support coherent walk. Rather than querying every time the page table is > updated, it is possible to do it only once when the p2m is initiali

Re: [Xen-devel] [RFC 01/22] xen/arm: do_trap_instr_abort_guest: Move the IPA computation out of the switch

2016-08-15 Thread Stefano Stabellini
On Thu, 28 Jul 2016, Julien Grall wrote: > A follow-up patch will add more case to the switch that will require the > IPA. So move the computation out of the switch. > > Signed-off-by: Julien Grall > --- > xen/arch/arm/traps.c | 36 ++-- > 1 file changed, 18 inser

Re: [Xen-devel] [RFC v3 00/13] linux: generalize sections, ranges and linker tables

2016-08-15 Thread James Bottomley
On Tue, 2016-08-09 at 21:51 -0700, Andy Lutomirski wrote: > On Aug 9, 2016 7:09 PM, "James Bottomley" < > james.bottom...@hansenpartnership.com> wrote: > > > > On Tue, 2016-08-09 at 15:24 +0100, One Thousand Gnomes wrote: > > > > table development go under copyleft-next, Rusty recently asked > >

Re: [Xen-devel] [RFC v3 00/13] linux: generalize sections, ranges and linker tables

2016-08-15 Thread James Bottomley
On Mon, 2016-08-15 at 21:15 +0100, Alan Cox wrote: > > This is the module tag ... it says what licence the module is > > under, not the licence for the module combined with the kernel, > > which is always GPLv2 because the stricter licence rules. > > Because if I build a BSD licensed module agai

[Xen-devel] [xen-unstable test] 100492: regressions - FAIL

2016-08-15 Thread osstest service owner
flight 100492 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/100492/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemuu-winxpsp3 9 windows-install fail REGR. vs. 100488 Regressions which

Re: [Xen-devel] [RFC v3 00/13] linux: generalize sections, ranges and linker tables

2016-08-15 Thread Steven Rostedt
On Mon, 15 Aug 2016 21:15:06 +0100 Alan Cox wrote: > > This is the module tag ... it says what licence the module is under, > > not the licence for the module combined with the kernel, which is > > always GPLv2 because the stricter licence rules. > > Because if I build a BSD licensed module ag

Re: [Xen-devel] [PULL 0/2] xen-20160812-tag-2

2016-08-15 Thread Peter Maydell
On 13 August 2016 at 00:42, Stefano Stabellini wrote: > The following changes since commit 28b874429ba16e71e0caa46453f3a3e31efb3c51: > > Merge remote-tracking branch > 'remotes/amit-migration/tags/migration-for-2.7-7' into staging (2016-08-11 > 17:53:35 +0100) > > are available in the git repo

[Xen-devel] [xen-4.7-testing test] 100491: regressions - FAIL

2016-08-15 Thread osstest service owner
flight 100491 xen-4.7-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/100491/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-credit2 19 guest-start/debian.repeat fail REGR. vs. 99972 Regressions whi

Re: [Xen-devel] [V4 PATCH 2/2] mips/panic: Replace smp_send_stop() with kdump friendly version in panic path

2016-08-15 Thread Corey Minyard
On 08/15/2016 12:06 PM, Corey Minyard wrote: On 08/15/2016 06:35 AM, 河合英宏 / KAWAI,HIDEHIRO wrote: Hi Corey, From: Corey Minyard [mailto:cminy...@mvista.com] Sent: Friday, August 12, 2016 10:56 PM I'll try to test this, but I have one comment inline... Thank you very much! On 08/11/2016 10:1

Re: [Xen-devel] [PATCH 6/7] arm64: xen: Enable user access before a privcmd hvc call

2016-08-15 Thread Stefano Stabellini
On Mon, 15 Aug 2016, Julien Grall wrote: > Hi Catalin, > > I have CCed Stefano who is maintaining the Xen ARM code in Linux. > > On 12/08/2016 17:27, Catalin Marinas wrote: > > Privcmd calls are issued by the userspace. The kernel needs to enable > > access to TTBR0_EL1 as the hypervisor would is

Re: [Xen-devel] [Qemu-devel] [RFC for 2.8 0/3] Drop support for 64 bit guests on 32 bit hosts

2016-08-15 Thread Stefano Stabellini
On Wed, 10 Aug 2016, Gerd Hoffmann wrote: > On Di, 2016-08-09 at 16:55 +0100, Alex Bennée wrote: > > Hi, > > > > I'm proposing for the 2.8 cycle we officially drop supporting 64 bit > > guests on 32 bit hosts. For most of the KVM targets it doesn't make > > any sense anyway and for TCG it makes th

Re: [Xen-devel] [V4 PATCH 2/2] mips/panic: Replace smp_send_stop() with kdump friendly version in panic path

2016-08-15 Thread Corey Minyard
On 08/15/2016 06:35 AM, 河合英宏 / KAWAI,HIDEHIRO wrote: Hi Corey, From: Corey Minyard [mailto:cminy...@mvista.com] Sent: Friday, August 12, 2016 10:56 PM I'll try to test this, but I have one comment inline... Thank you very much! On 08/11/2016 10:17 PM, Dave Young wrote: On 08/10/16 at 05:09p

Re: [Xen-devel] [win-pv-devel] rtc timeoffset not being set on TZ changes?

2016-08-15 Thread Nathan March
Hi Paul, Nope, I’m using qemu provided by xen-runtime in the centos packages (https://cbs.centos.org/koji/packageinfo?packageID=88). root 1462 0.2 0.3 406856 21016 ?SLsl Aug13 9:37 /usr/lib64/xen/bin/qemu-system-i386 -xen-domid 31 -chardev socket,id=libxl-cmd,path=/var/r

Re: [Xen-devel] [PATCH v2 23/25] arm/altp2m: Extend libxl to activate altp2m on ARM.

2016-08-15 Thread Sergej Proskurin
Hi Wei, On 08/11/2016 06:00 PM, Wei Liu wrote: > Sorry for the late reply. > No worries, it's all good :) Thanks for your reply. [...] ("smbios_firmware", string), ("acpi_firmware",string),

Re: [Xen-devel] [PATCH 2/2] xen/events: Convert to hotplug state machine

2016-08-15 Thread Boris Ostrovsky
On 08/15/2016 11:06 AM, David Vrabel wrote: > On 15/08/16 15:46, Boris Ostrovsky wrote: >> From: Sebastian Andrzej Siewior >> >> Install the callbacks via the state machine. > [...] >> +static int xen_evtchn_cpu_dead(unsigned int cpu) >> +{ >> +__evtchn_fifo_handle_events(cpu, true); >> +r

Re: [Xen-devel] [PATCH v1] Livepatch ARM 64 implementation

2016-08-15 Thread Konrad Rzeszutek Wilk
On Mon, Aug 15, 2016 at 04:52:58PM +0200, Julien Grall wrote: > > > On 15/08/2016 01:07, Konrad Rzeszutek Wilk wrote: > > Hey! > > Hi Konrad, > > > This is the first (non RFC) posting of the enablement of Livepatch under > > ARM64. > > > > The patches are based on: [PATCH v3] Livepatch fixes

[Xen-devel] [PATCH] tools: delete gtraceview and gtracestat

2016-08-15 Thread Wei Liu
There has not been any substantial update to them since 2011. My quick check shows that they don't work. Just delete them. It would be easy to resurrect them from git log should people still need them. Signed-off-by: Wei Liu --- Cc: Ian Jackson --- .gitignore |2 - .hgignore

Re: [Xen-devel] [PATCH v1 6/9] livepatch: Initial ARM64 support.

2016-08-15 Thread Andrew Cooper
On 15/08/16 16:25, Julien Grall wrote: > > > On 15/08/2016 17:17, Konrad Rzeszutek Wilk wrote: >> On Mon, Aug 15, 2016 at 04:57:26PM +0200, Julien Grall wrote: >>> Hi Jan and Konrad, >>> >>> On 15/08/2016 16:23, Jan Beulich wrote: >>> On 15.08.16 at 16:09, wrote: > On Mon, Aug 15, 2016 at

Re: [Xen-devel] [PATCH v1 6/9] livepatch: Initial ARM64 support.

2016-08-15 Thread Julien Grall
On 15/08/2016 17:17, Konrad Rzeszutek Wilk wrote: On Mon, Aug 15, 2016 at 04:57:26PM +0200, Julien Grall wrote: Hi Jan and Konrad, On 15/08/2016 16:23, Jan Beulich wrote: On 15.08.16 at 16:09, wrote: On Mon, Aug 15, 2016 at 02:21:48AM -0600, Jan Beulich wrote: On 15.08.16 at 01:07, wrote

Re: [Xen-devel] Device model operation hypercall (DMOP, re qemu depriv)

2016-08-15 Thread Jan Beulich
>>> On 15.08.16 at 16:50, wrote: > On 09/08/16 11:29, Jan Beulich wrote: > On 08.08.16 at 15:46, wrote: >>> Jan Beulich writes ("Re: Device model operation hypercall (DMOP, re qemu >>> depriv)"): On 05.08.16 at 18:28, wrote: > That is, a bug of class 2 would allow the unprivileged

Re: [Xen-devel] Device model operation hypercall (DMOP, re qemu depriv)

2016-08-15 Thread Jan Beulich
>>> On 15.08.16 at 16:57, wrote: > On 15/08/16 12:20, Jan Beulich wrote: > On 15.08.16 at 12:47, wrote: >>> On 15/08/16 11:19, Jan Beulich wrote: Well, none of the options considered so far are really nice or readily available. I think the easiest to use for both the caller and

Re: [Xen-devel] [PATCH v3 6/9] livepatch: Add parsing for the symbol+0x/

2016-08-15 Thread Konrad Rzeszutek Wilk
On Mon, Aug 15, 2016 at 09:12:21AM -0600, Jan Beulich wrote: > >>> On 15.08.16 at 16:35, wrote: > > On Mon, Aug 15, 2016 at 04:53:48AM -0600, Jan Beulich wrote: > >> >>> On 14.08.16 at 23:52, wrote: > >> > in case we want to patch at specific offsets inside > >> > a function. (for example if we w

Re: [Xen-devel] [PATCH v1 6/9] livepatch: Initial ARM64 support.

2016-08-15 Thread Julien Grall
Hi Konrad, On 15/08/2016 01:07, Konrad Rzeszutek Wilk wrote: [...] diff --git a/xen/arch/arm/arm32/Makefile b/xen/arch/arm/arm32/Makefile index b20db64..5966de0 100644 --- a/xen/arch/arm/arm32/Makefile +++ b/xen/arch/arm/arm32/Makefile @@ -4,8 +4,8 @@ obj-$(EARLY_PRINTK) += debug.o obj-y += d

Re: [Xen-devel] [PATCH v3 9/9] livepach: Add .livepatch.hooks functions and test-case

2016-08-15 Thread Jan Beulich
>>> On 15.08.16 at 16:46, wrote: > On Mon, Aug 15, 2016 at 05:15:28AM -0600, Jan Beulich wrote: >> >>> On 14.08.16 at 23:52, wrote: >> > @@ -72,7 +73,11 @@ struct payload { >> > struct livepatch_build_id dep; /* >> > ELFNOTE_DESC(.livepatch.depends). */ >> > void *bss;

Re: [Xen-devel] [PATCH v1 6/9] livepatch: Initial ARM64 support.

2016-08-15 Thread Konrad Rzeszutek Wilk
On Mon, Aug 15, 2016 at 04:57:26PM +0200, Julien Grall wrote: > Hi Jan and Konrad, > > On 15/08/2016 16:23, Jan Beulich wrote: > > > > > On 15.08.16 at 16:09, wrote: > > > On Mon, Aug 15, 2016 at 02:21:48AM -0600, Jan Beulich wrote: > > > > > > > On 15.08.16 at 01:07, wrote: > > > > > @@ -711,9

Re: [Xen-devel] [PATCH v3 6/9] livepatch: Add parsing for the symbol+0x/

2016-08-15 Thread Jan Beulich
>>> On 15.08.16 at 16:35, wrote: > On Mon, Aug 15, 2016 at 04:53:48AM -0600, Jan Beulich wrote: >> >>> On 14.08.16 at 23:52, wrote: >> > in case we want to patch at specific offsets inside >> > a function. (for example if we want to do NOP patching). >> > >> > We also assume that the 'len' is on

Re: [Xen-devel] [RFC 00/22] xen/arm: Rework the P2M code to follow break-before-make sequence

2016-08-15 Thread Edgar E. Iglesias
On Thu, Jul 28, 2016 at 03:51:23PM +0100, Julien Grall wrote: > Hello all, > > The ARM architecture mandates the use of a break-before-make sequence when > changing translation entries if the page table is shared between multiple > CPUs whenever a valid entry is replaced by another valid entry (se

Re: [Xen-devel] [PATCH v3 1/9] livepatch: Clear .bss when payload is reverted

2016-08-15 Thread Jan Beulich
>>> On 15.08.16 at 16:29, wrote: > On Mon, Aug 15, 2016 at 04:27:38AM -0600, Jan Beulich wrote: >> >>> On 14.08.16 at 23:52, wrote: >> > @@ -1034,6 +1047,9 @@ static int revert_payload(struct payload *data) >> > list_del_rcu(&data->applied_list); >> > unregister_virtual_region(&data->re

Re: [Xen-devel] [PATCH 2/4] x86emul: drop RIP-relative special case for TEST

2016-08-15 Thread Jan Beulich
>>> On 15.08.16 at 16:25, wrote: > On 15/08/16 09:34, Jan Beulich wrote: >> @@ -1851,11 +1911,6 @@ x86_emulate( >> ((op_bytes == 8) ? 4 : op_bytes); >> else if ( (d & SrcMask) == SrcImmByte ) >> ea.mem.off += 1; >> -els

Re: [Xen-devel] [PATCH 2/2] xen/events: Convert to hotplug state machine

2016-08-15 Thread David Vrabel
On 15/08/16 15:46, Boris Ostrovsky wrote: > From: Sebastian Andrzej Siewior > > Install the callbacks via the state machine. [...] > +static int xen_evtchn_cpu_dead(unsigned int cpu) > +{ > + __evtchn_fifo_handle_events(cpu, true); > + return 0; > +} I'm not familiar with the new state m

Re: [Xen-devel] [Very RFC PATCH 3/3] livepatch: Initial ARM32/64 support.

2016-08-15 Thread Konrad Rzeszutek Wilk
On Mon, Aug 15, 2016 at 03:27:08PM +0200, Julien Grall wrote: > Hi Konrad, > > On 12/08/2016 22:50, Konrad Rzeszutek Wilk wrote: > > On Fri, Aug 12, 2016 at 05:00:47PM +0200, Julien Grall wrote: > > > > diff --git a/xen/include/asm-arm/current.h > > > > b/xen/include/asm-arm/current.h > > > > ind

[Xen-devel] [PATCH] xenbus: don't look up transaction IDs for ordinary writes

2016-08-15 Thread Jan Beulich
This should really only be done for XS_TRANSACTION_END messages, or else at least some of the xenstore-* tools don't work anymore. Fixes: 0beef634b8 ("xenbus: don't BUG() on user mode induced condition") Reported-by: Richard Schütz Cc: Signed-off-by: Jan Beulich Tested-by: Richard Schütz ---

Re: [Xen-devel] Device model operation hypercall (DMOP, re qemu depriv)

2016-08-15 Thread George Dunlap
On 15/08/16 12:20, Jan Beulich wrote: On 15.08.16 at 12:47, wrote: >> On 15/08/16 11:19, Jan Beulich wrote: >>> Well, none of the options considered so far are really nice or >>> readily available. I think the easiest to use for both the caller and >>> the implementation of the hypercall woul

Re: [Xen-devel] [PATCH v1 6/9] livepatch: Initial ARM64 support.

2016-08-15 Thread Julien Grall
Hi Jan and Konrad, On 15/08/2016 16:23, Jan Beulich wrote: On 15.08.16 at 16:09, wrote: On Mon, Aug 15, 2016 at 02:21:48AM -0600, Jan Beulich wrote: On 15.08.16 at 01:07, wrote: @@ -711,9 +711,15 @@ static int prepare_payload(struct payload *payload, return -EINVAL;

Re: [Xen-devel] [PATCH 3/3] Significant changes to decision making; some new roles and minor changes

2016-08-15 Thread Lars Kurth
Hi Tim, >At 00:13 +0100 on 12 Aug (1470960826), Lars Kurth wrote: >> +### Conflict Resolution {#conflict} >> + >> +Sub-projects and teams hosted on Xenproject.org are not democracies >>but >> +meritocracies. In situations where there is disagreement on issues >>related to >> +the day-to-day runn

[Xen-devel] [XTF PATCH v3] xtf-runner: support two modes for getting output

2016-08-15 Thread Wei Liu
We need two modes for getting output: 1. Use console directly with newer (>=4.8) Xen 2. Use log files for older Xen This patch implements both. The default behaviour is to use the console mode. Signed-off-by: Wei Liu --- v3: 1. use os module for opening log file 2. poll up to 1 second for final

Re: [Xen-devel] [PATCH v1] Livepatch ARM 64 implementation

2016-08-15 Thread Julien Grall
On 15/08/2016 01:07, Konrad Rzeszutek Wilk wrote: Hey! Hi Konrad, This is the first (non RFC) posting of the enablement of Livepatch under ARM64. The patches are based on: [PATCH v3] Livepatch fixes and features for v4.8. (https://lists.xen.org/archives/html/xen-devel/2016-08/msg01825.html

Re: [Xen-devel] Device model operation hypercall (DMOP, re qemu depriv)

2016-08-15 Thread David Vrabel
On 09/08/16 11:29, Jan Beulich wrote: On 08.08.16 at 15:46, wrote: >> Jan Beulich writes ("Re: Device model operation hypercall (DMOP, re qemu >> depriv)"): >>> On 05.08.16 at 18:28, wrote: That is, a bug of class 2 would allow the unprivileged qemu process in dom0 to cause damage

Re: [Xen-devel] [PATCH v3 9/9] livepach: Add .livepatch.hooks functions and test-case

2016-08-15 Thread Konrad Rzeszutek Wilk
On Mon, Aug 15, 2016 at 05:15:28AM -0600, Jan Beulich wrote: > >>> On 14.08.16 at 23:52, wrote: > > v4..v11: Defered for v4.8 > > v12: s/xsplice/livepatch/ > > v13: Clarify the comments about spin_debug_enable > > (Side note: v13 here vs v3 in the subject.) > > > Rename one of the hooks to

[Xen-devel] [PATCH 1/2] xen/x86: Convert to hotplug state machine

2016-08-15 Thread Boris Ostrovsky
Switch to new CPU hotplug infrastructure. Signed-off-by: Boris Ostrovsky Suggested-by: Sebastian Andrzej Siewior --- arch/x86/xen/enlighten.c | 115 +--- include/linux/cpuhotplug.h |2 + 2 files changed, 67 insertions(+), 50 deletions(-) diff --gi

Re: [Xen-devel] [PATCH v3 7/9] livepatch: NOP if func->new_[addr, size] is zero.

2016-08-15 Thread Konrad Rzeszutek Wilk
On Mon, Aug 15, 2016 at 04:59:52AM -0600, Jan Beulich wrote: > >>> On 14.08.16 at 23:52, wrote: > > The NOP functionality will NOP any of the code at > > the 'old_addr' or at 'name' if the 'new_addr' and 'new_size' > > are both zero. The purpose of this is to NOP out calls, such as: > > > > e9 <

[Xen-devel] [PATCH 2/2] xen/events: Convert to hotplug state machine

2016-08-15 Thread Boris Ostrovsky
From: Sebastian Andrzej Siewior Install the callbacks via the state machine. Signed-off-by: Sebastian Andrzej Siewior Signed-off-by: Boris Ostrovsky --- Changes from Sebastian's original version: * Renamed hotplug state from CPUHP_XEN_EV_PREPEARE to CPUHP_XEN_EVTCHN_PREPARE * Dropped suggestio

[Xen-devel] [PATCH 0/2] Convert to new CPU hotplug framework

2016-08-15 Thread Boris Ostrovsky
New CPU hotplug framework was introduced recently. These patches convert Xen CPU hotplug code to this infrastructure. The patches (patch 1 specifically) will apply on top of https://lists.xenproject.org/archives/html/xen-devel/2016-08/msg00562.html (I will be out of the office starting tomorrow

Re: [Xen-devel] [PATCH v3 6/9] livepatch: Add parsing for the symbol+0x/

2016-08-15 Thread Konrad Rzeszutek Wilk
On Mon, Aug 15, 2016 at 04:53:48AM -0600, Jan Beulich wrote: > >>> On 14.08.16 at 23:52, wrote: > > in case we want to patch at specific offsets inside > > a function. (for example if we want to do NOP patching). > > > > We also assume that the 'len' is only the size of > > an isns that would be

Re: [Xen-devel] [PATCH v3 4/9] livepatch: Sync cache of build-id before using it first time.

2016-08-15 Thread Konrad Rzeszutek Wilk
On Mon, Aug 15, 2016 at 11:46:13AM +0100, Andrew Cooper wrote: > On 15/08/16 11:38, Jan Beulich wrote: > On 14.08.16 at 23:52, wrote: > >> We don't print at bootup time the build-id. The reason is > >> that xen_build_init and livepatch_init are both __initcall > >> type routines. This meant t

Re: [Xen-devel] [PATCH v3 1/9] livepatch: Clear .bss when payload is reverted

2016-08-15 Thread Konrad Rzeszutek Wilk
On Mon, Aug 15, 2016 at 04:27:38AM -0600, Jan Beulich wrote: > >>> On 14.08.16 at 23:52, wrote: > > @@ -374,7 +376,18 @@ static int move_payload(struct payload *payload, > > struct livepatch_elf *elf) > > elf->name, elf->sec[i].name, > > elf->sec[i].load_addr); > >

Re: [Xen-devel] Device model operation hypercall (DMOP, re qemu depriv)

2016-08-15 Thread Jan Beulich
>>> On 15.08.16 at 14:07, wrote: > Jan Beulich writes ("Re: Device model operation hypercall (DMOP, re qemu > depriv)"): >> On 15.08.16 at 12:47, wrote: >> > What about including in the "fixed" part of the hypercall a virtual >> > address range that all pointers must be in? That wouldn't even r

Re: [Xen-devel] [PATCH 2/4] x86emul: drop RIP-relative special case for TEST

2016-08-15 Thread Andrew Cooper
On 15/08/16 09:34, Jan Beulich wrote: > @@ -1851,11 +1911,6 @@ x86_emulate( > ((op_bytes == 8) ? 4 : op_bytes); > else if ( (d & SrcMask) == SrcImmByte ) > ea.mem.off += 1; > -else if ( !ext && ((b & 0xfe) == 0xf6) && >

Re: [Xen-devel] [PATCH v1 6/9] livepatch: Initial ARM64 support.

2016-08-15 Thread Jan Beulich
>>> On 15.08.16 at 16:09, wrote: > On Mon, Aug 15, 2016 at 02:21:48AM -0600, Jan Beulich wrote: >> >>> On 15.08.16 at 01:07, wrote: >> > @@ -711,9 +711,15 @@ static int prepare_payload(struct payload *payload, >> > return -EINVAL; >> > } >> > } >> > +#ifndef

Re: [Xen-devel] [PATCH 2/2] x86/EFI: use less crude way of generating the build ID

2016-08-15 Thread Konrad Rzeszutek Wilk
On Mon, Aug 15, 2016 at 01:58:47AM -0600, Jan Beulich wrote: > >>> On 15.08.16 at 01:42, wrote: > >> --- a/xen/arch/x86/efi/Makefile > >> +++ b/xen/arch/x86/efi/Makefile > >> @@ -9,6 +9,9 @@ efi := $(if $(efi),$(shell $(CC) $(filte > >> efi := $(if $(efi),$(shell $(LD) -mi386pep --subsystem=10 -o

Re: [Xen-devel] [PATCH v1 6/9] livepatch: Initial ARM64 support.

2016-08-15 Thread Konrad Rzeszutek Wilk
On Mon, Aug 15, 2016 at 02:21:48AM -0600, Jan Beulich wrote: > >>> On 15.08.16 at 01:07, wrote: > > --- a/xen/common/Kconfig > > +++ b/xen/common/Kconfig > > @@ -222,7 +222,7 @@ endmenu > > config LIVEPATCH > > bool "Live patching support (TECH PREVIEW)" > > default n > > - depends on X

Re: [Xen-devel] [PATCH 1/4] x86emul: remove dead code

2016-08-15 Thread Andrew Cooper
On 15/08/16 09:34, Jan Beulich wrote: > As of commit 989cdfa9b4 ("x86emul: don't special case fetching unsigned > 8-bit immediates") the conditional being removed has been always false. > > Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper ___ Xen-

Re: [Xen-devel] [PATCH] xentop: Adds options for tabs-separators, and including the domain ID in the output.

2016-08-15 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH] xentop: Adds options for tabs-separators, and including the domain ID in the output."): > On Sat, Aug 13, 2016 at 05:51:21PM +, Stefan Wieser wrote: > > This change adds two options to xentop: > > > > -T adds a tabulator (\t) character after each fiel

Re: [Xen-devel] [Very RFC PATCH 3/3] livepatch: Initial ARM32/64 support.

2016-08-15 Thread Julien Grall
Hi Konrad, On 12/08/2016 22:50, Konrad Rzeszutek Wilk wrote: On Fri, Aug 12, 2016 at 05:00:47PM +0200, Julien Grall wrote: diff --git a/xen/include/asm-arm/current.h b/xen/include/asm-arm/current.h index 65c0cdf..f4fcfd6 100644 --- a/xen/include/asm-arm/current.h +++ b/xen/include/asm-arm/curre

Re: [Xen-devel] [PATCH] acpi: Re-license ACPI builder files from GPLv2 to LGPLv2.1

2016-08-15 Thread Boris Ostrovsky
On 08/01/2016 09:56 AM, Boris Ostrovsky wrote: > Simon, Keir, Simon, ping? > > > In case you didn't realize --- this needs your ACK. > > Jan, now that I checked the logs more carefully, you are the only one > from Suse/Novell who touched these files so your ACK is needed as well. > > Thanks. >

Re: [Xen-devel] [PATCH v2 23/23] libxc/xc_dom_core: Copy ACPI tables to guest space

2016-08-15 Thread Boris Ostrovsky
On 08/15/2016 03:48 AM, Shannon Zhao wrote: > Hi Boris > > On 2016/8/5 5:06, Boris Ostrovsky wrote: >> Load ACPI modules into guest space >> >> Signed-off-by: Boris Ostrovsky >> --- >> v2: >> * New patch, loosely based on Shannon's ARM patch >> >> tools/libxc/xc_dom_core.c | 92 >> ++

Re: [Xen-devel] [PATCH v2 00/23] Make ACPI builder available to components other than hvmloader

2016-08-15 Thread Boris Ostrovsky
On 08/15/2016 02:37 AM, Shannon Zhao wrote: > > On 2016/8/5 5:06, Boris Ostrovsky wrote: >> The goal here is to build ACPI tables for PVHv2/HVMlite guests while reusing >> existing >> hvmloader's ACPI builder code. The builder is provided as a library in >> tools/libacpi. >> >> This version is bu

[Xen-devel] [xen-unstable-smoke test] 100493: tolerable all pass - PUSHED

2016-08-15 Thread osstest service owner
flight 100493 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/100493/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-libvirt 12 migrate-support-checkfail never pass test-armhf-armhf-xl 1

Re: [Xen-devel] [PATCH] xentop: Adds options for tabs-separators, and including the domain ID in the output.

2016-08-15 Thread Wei Liu
On Mon, Aug 15, 2016 at 01:09:32PM +0100, Wei Liu wrote: > On Sat, Aug 13, 2016 at 05:51:21PM +, Stefan Wieser wrote: > > This change adds two options to xentop: > > > > -T adds a tabulator (\t) character after each field, to allow > > easier and more robust parsin

Re: [Xen-devel] [PATCH 0/4] Clarify License information : m4, stubdom, tools/blktap2, xen/crypto

2016-08-15 Thread Wei Liu
Cc Ian's correct email address On Fri, Aug 12, 2016 at 06:32:32PM +0100, Lars Kurth wrote: > This series contains some of the easier license clean-up cases, which can be > fixed > by adding COPYING files or README.source files. The series specially contains: > > xen/crypto and xen/include/crypto

Re: [Xen-devel] [PATCH] xentop: Adds options for tabs-separators, and including the domain ID in the output.

2016-08-15 Thread Wei Liu
On Sat, Aug 13, 2016 at 05:51:21PM +, Stefan Wieser wrote: > This change adds two options to xentop: > > -T adds a tabulator (\t) character after each field, to allow > easier and more robust parsing. This affects batch mode only. > -I includes a colum

Re: [Xen-devel] Device model operation hypercall (DMOP, re qemu depriv)

2016-08-15 Thread Ian Jackson
Jan Beulich writes ("Re: Device model operation hypercall (DMOP, re qemu depriv)"): > On 15.08.16 at 12:47, wrote: > > What about including in the "fixed" part of the hypercall a virtual > > address range that all pointers must be in? That wouldn't even require > > a user/kernel flag actually; a

Re: [Xen-devel] [OSSTEST PATCH] ts-coverity-upload: Double coverity upload timeout

2016-08-15 Thread Andrew Cooper
On 15/08/16 11:46, Ian Jackson wrote: > From 3600s to 7200s. It failed in flight 100483 in the Mass colo, > with a timeout, having uploaded 95% (151Mby) in 3600s. > > Ideally we would like to release the build host, but there is no > infrastructure for doing that right now. > > Signed-off-by: Ian

Re: [Xen-devel] [V4 PATCH 2/2] mips/panic: Replace smp_send_stop() with kdump friendly version in panic path

2016-08-15 Thread 河合英宏 / KAWAI,HIDEHIRO
Hi Corey, > From: Corey Minyard [mailto:cminy...@mvista.com] > Sent: Friday, August 12, 2016 10:56 PM > I'll try to test this, but I have one comment inline... Thank you very much! > On 08/11/2016 10:17 PM, Dave Young wrote: > > On 08/10/16 at 05:09pm, Hidehiro Kawai wrote: [snip] > >> diff --gi

Re: [Xen-devel] [V4 PATCH 1/2] x86/panic: Replace smp_send_stop() with kdump friendly version in panic path

2016-08-15 Thread 河合英宏 / KAWAI,HIDEHIRO
Hi Dave, Thank you for the review. > From: Dave Young [mailto:dyo...@redhat.com] > Sent: Friday, August 12, 2016 12:17 PM > > Thanks for the update. > On 08/10/16 at 05:09pm, Hidehiro Kawai wrote: > > Daniel Walker reported problems which happens when > > crash_kexec_post_notifiers kernel option

Re: [Xen-devel] Device model operation hypercall (DMOP, re qemu depriv)

2016-08-15 Thread Jan Beulich
>>> On 15.08.16 at 12:47, wrote: > On 15/08/16 11:19, Jan Beulich wrote: >> Well, none of the options considered so far are really nice or >> readily available. I think the easiest to use for both the caller and >> the implementation of the hypercall would be the auxiliary >> hypercall for a kerne

Re: [Xen-devel] [PATCH v3 9/9] livepach: Add .livepatch.hooks functions and test-case

2016-08-15 Thread Jan Beulich
>>> On 14.08.16 at 23:52, wrote: > v4..v11: Defered for v4.8 > v12: s/xsplice/livepatch/ > v13: Clarify the comments about spin_debug_enable (Side note: v13 here vs v3 in the subject.) > Rename one of the hooks to lower-case (Z->z) to guarantee it being > called last. Does lower case

Re: [Xen-devel] [PATCH v3 8/9] symbols: Generate an xen-sym.map

2016-08-15 Thread Jan Beulich
>>> On 14.08.16 at 23:52, wrote: > --- a/xen/arch/x86/Makefile > +++ b/xen/arch/x86/Makefile > @@ -136,8 +136,12 @@ $(TARGET)-syms: prelink.o xen.lds > $(BASEDIR)/common/symbols-dummy.o > | $(BASEDIR)/tools/symbols $(all_symbols) --sysv --sort > --warn-dup \ > >$(@D)/

Re: [Xen-devel] [PATCH v3 7/9] livepatch: NOP if func->new_[addr, size] is zero.

2016-08-15 Thread Jan Beulich
>>> On 14.08.16 at 23:52, wrote: > The NOP functionality will NOP any of the code at > the 'old_addr' or at 'name' if the 'new_addr' and 'new_size' > are both zero. The purpose of this is to NOP out calls, such as: > > e9 <4-bytes-offset> Except that E9 is JMP; CALL is E8. > (5 byte insn), or

Re: [Xen-devel] [PATCH 3/3] Significant changes to decision making; some new roles and minor changes

2016-08-15 Thread Tim Deegan
Hi, At 00:13 +0100 on 12 Aug (1470960826), Lars Kurth wrote: > +### Conflict Resolution {#conflict} > + > +Sub-projects and teams hosted on Xenproject.org are not democracies but > +meritocracies. In situations where there is disagreement on issues related > to > +the day-to-day running of the

Re: [Xen-devel] [XTF PATCH v2] xtf-runner: support two modes for getting output

2016-08-15 Thread Wei Liu
On Fri, Aug 12, 2016 at 10:51:37AM +0100, Ian Jackson wrote: > Wei Liu writes ("Re: [XTF PATCH v2] xtf-runner: support two modes for getting > output"): > > On Thu, Aug 11, 2016 at 07:29:00PM +0100, Ian Jackson wrote: > > > We don't care when xenconsoled closes the logfile. We care about when > >

Re: [Xen-devel] [PATCH v3 6/9] livepatch: Add parsing for the symbol+0x/

2016-08-15 Thread Jan Beulich
>>> On 14.08.16 at 23:52, wrote: > in case we want to patch at specific offsets inside > a function. (for example if we want to do NOP patching). > > We also assume that the 'len' is only the size of > an isns that would be for a call opcode (so 5 bytes > on x86, and 4 on ARM 32/64). Which makes

[Xen-devel] [RFC PATCH] tools: remove blktap2 related code and documentation

2016-08-15 Thread Wei Liu
Blktap2 is effectively dead code for a few years. Notable changes in this patch: 0. Unhook blktap2 from build system 1. Now libxl no longer supports TAP ask backend, appropriate assertions are added and some code paths now return ERROR_FAIL 2. Tap is no longer a supported backend in doc 3. Rem

Re: [Xen-devel] [PATCH v2 6/7] libxenstat: honour XEN_RUN_DIR

2016-08-15 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH v2 6/7] libxenstat: honour XEN_RUN_DIR"): > On Wed, Jul 20, 2016 at 03:40:56PM +0100, Ian Jackson wrote: > > Wei Liu writes ("[PATCH v2 6/7] libxenstat: honour XEN_RUN_DIR"): > > > Backport candidate. > > > > Thanks. Can you let me know when it's applied to staging ? >

Re: [Xen-devel] Device model operation hypercall (DMOP, re qemu depriv)

2016-08-15 Thread George Dunlap
On 15/08/16 11:19, Jan Beulich wrote: On 15.08.16 at 11:39, wrote: >> On 12/08/16 12:50, Jan Beulich wrote: >> On 12.08.16 at 11:44, wrote: On 09/08/16 12:30, Jan Beulich wrote: On 09.08.16 at 12:48, wrote: >> Jan Beulich writes ("Re: Device model operation hypercall (

Re: [Xen-devel] [PATCH v3 4/9] livepatch: Sync cache of build-id before using it first time.

2016-08-15 Thread Andrew Cooper
On 15/08/16 11:38, Jan Beulich wrote: On 14.08.16 at 23:52, wrote: >> We don't print at bootup time the build-id. The reason is >> that xen_build_init and livepatch_init are both __initcall >> type routines. This meant that when livepatch_init called >> xen_build_id, it would return -ENODATA

[Xen-devel] [OSSTEST PATCH] ts-coverity-upload: Double coverity upload timeout

2016-08-15 Thread Ian Jackson
From 3600s to 7200s. It failed in flight 100483 in the Mass colo, with a timeout, having uploaded 95% (151Mby) in 3600s. Ideally we would like to release the build host, but there is no infrastructure for doing that right now. Signed-off-by: Ian Jackson --- ts-coverity-upload | 2 +- 1 file ch

Re: [Xen-devel] [PATCH v3 5/9] livepatch: Move code from prepare_payload to own routine

2016-08-15 Thread Jan Beulich
>>> On 14.08.16 at 23:52, wrote: > --- a/xen/common/livepatch.c > +++ b/xen/common/livepatch.c > @@ -232,6 +232,29 @@ static const char *livepatch_symbols_lookup(unsigned > long addr, > return n; > } > > +static int lookup_symbol(struct livepatch_func *f, struct livepatch_elf *elf) The l

Re: [Xen-devel] [PATCH v3 4/9] livepatch: Sync cache of build-id before using it first time.

2016-08-15 Thread Jan Beulich
>>> On 14.08.16 at 23:52, wrote: > We don't print at bootup time the build-id. The reason is > that xen_build_init and livepatch_init are both __initcall > type routines. This meant that when livepatch_init called > xen_build_id, it would return -ENODATA as build_id_len was > not setup yet (b/c xe

Re: [Xen-devel] [PATCH v3 3/9] version/livepatch: Move xen_build_id_check to version.h

2016-08-15 Thread Jan Beulich
>>> On 14.08.16 at 23:52, wrote: > It makes more sense for it to be there. However that > means the version.h has now a dependency on > as the Elf_Note is a macro. > > The elfstructs.h has a dependency on types.h as well so > we need that. We cannot put that #include > in elfstructs.h as that f

Re: [Xen-devel] [PATCH v3 2/9] livepatch: Deal with payloads without any .text

2016-08-15 Thread Jan Beulich
>>> On 14.08.16 at 23:52, wrote: > It is possible. Especially if the only thing they do is > NOP functions - in which case there is only .livepatch.funcs > sections. > > Signed-off-by: Konrad Rzeszutek Wilk Reviewed-by: Jan Beulich ___ Xen-devel ma

Re: [Xen-devel] [PATCH v3 1/9] livepatch: Clear .bss when payload is reverted

2016-08-15 Thread Jan Beulich
>>> On 14.08.16 at 23:52, wrote: > @@ -374,7 +376,18 @@ static int move_payload(struct payload *payload, struct > livepatch_elf *elf) > elf->name, elf->sec[i].name, elf->sec[i].load_addr); > } > else > -memset(elf->sec[i].load_add

Re: [Xen-devel] [RFC 00/22] xen/arm: Rework the P2M code to follow break-before-make sequence

2016-08-15 Thread Julien Grall
Hi Stefano, Do you have any comments on this series? Cheers, On 28/07/2016 16:51, Julien Grall wrote: Hello all, The ARM architecture mandates the use of a break-before-make sequence when changing translation entries if the page table is shared between multiple CPUs whenever a valid entry is

Re: [Xen-devel] [PATCH v2] Remove ambiguities in the COPYING file; add CONTRIBUTING file

2016-08-15 Thread Andrew Cooper
On 12/08/16 10:37, Lars Kurth wrote: > COPYING file: > The motivation of this change is to make it easier for new > contributors to conduct a license and patent review, WITHOUT > changing any licenses. > - Remove references to BSD-style licenses as we have more > common license exceptions and rep

Re: [Xen-devel] Device model operation hypercall (DMOP, re qemu depriv)

2016-08-15 Thread Jan Beulich
>>> On 15.08.16 at 11:39, wrote: > On 12/08/16 12:50, Jan Beulich wrote: > On 12.08.16 at 11:44, wrote: >>> On 09/08/16 12:30, Jan Beulich wrote: >>> On 09.08.16 at 12:48, wrote: > Jan Beulich writes ("Re: Device model operation hypercall (DMOP, re qemu > depriv)"): >> Actua

Re: [Xen-devel] [RFC 18/22] xen/arm: p2m: Introduce p2m_set_entry and __p2m_set_entry

2016-08-15 Thread Sergej Proskurin
Hi Julien, [...] > +static int p2m_set_entry(struct p2m_domain *p2m, > + gfn_t sgfn, > + unsigned long todo, > + mfn_t smfn, > + p2m_type_t t, > + p2m_access_t a) > +{ > +in

Re: [Xen-devel] [PATCH 2/2] make use of .startof.() and .sizeof.() assembler expressions

2016-08-15 Thread Jan Beulich
>>> On 15.08.16 at 11:43, wrote: > On 12/08/16 15:48, Jan Beulich wrote: >> Section start symbols frequently obscure the actual symbol name living >> at the start of the section. Eliminate them where they can be replaced >> by linker resolved .startof.* symbols. (Section end symbols may have >> th

  1   2   >