Re: [Xen-devel] [PATCH 2/3] docs: add DIRECTORY_PART specification do xenstore protocol doc

2017-05-08 Thread Juergen Gross
On 08/05/17 13:53, Ian Jackson wrote: > Juergen Gross writes ("Re: [PATCH 2/3] docs: add DIRECTORY_PART specification > do xenstore protocol doc"): >> On 08/05/17 12:09, Ian Jackson wrote: >>> The "generation count" is not defined anywhere else in this pr

Re: [Xen-devel] [PATCH 1/3] docs: specify endianess of xenstore protocol header

2017-05-08 Thread Juergen Gross
On 08/05/17 12:24, Ian Jackson wrote: > Juergen Gross writes ("Re: [PATCH 1/3] docs: specify endianess of xenstore > protocol header"): >> On 08/05/17 12:07, Ian Jackson wrote: >>> Yes, but this is not correct. On a big-endian cpu, they would be in >>>

Re: [Xen-devel] [PATCH 2/3] docs: add DIRECTORY_PART specification do xenstore protocol doc

2017-05-08 Thread Juergen Gross
On 08/05/17 12:09, Ian Jackson wrote: > Juergen Gross writes ("[PATCH 2/3] docs: add DIRECTORY_PART specification do > xenstore protocol doc"): >> DIRECTORY_PART was missing in docs/misc/xenstore.txt. Add it. > ... >> +DIRECTORY_PART | &

Re: [Xen-devel] [PATCH 3/3] docs: document CONTROL command of xenstore protocol

2017-05-08 Thread Juergen Gross
On 08/05/17 12:13, Ian Jackson wrote: > Juergen Gross writes ("[PATCH 3/3] docs: document CONTROL command of xenstore > protocol"): >> The CONTROL command (former DEBUG command) isn't specified in the >> xenstore protocol doc. Add it. > ... >&g

Re: [Xen-devel] [PATCH 1/3] docs: specify endianess of xenstore protocol header

2017-05-08 Thread Juergen Gross
On 08/05/17 12:07, Ian Jackson wrote: > Juergen Gross writes ("[PATCH 1/3] docs: specify endianess of xenstore > protocol header"): >> The endianess of the xenstore protocol header should be specified. > ... >> -followed by xsd_sockmsg.len bytes of payload. >&g

Re: [Xen-devel] null domains after xl destroy

2017-05-08 Thread Juergen Gross
On 04/05/17 00:17, Glenn Enright wrote: > On 04/05/17 04:58, Steven Haigh wrote: >> On 04/05/17 01:53, Juergen Gross wrote: >>> On 03/05/17 12:45, Steven Haigh wrote: >>>> Just wanted to give this a little nudge now people seem to be back on >>>> de

[Xen-devel] [PATCH 1/3] docs: specify endianess of xenstore protocol header

2017-05-08 Thread Juergen Gross
The endianess of the xenstore protocol header should be specified. Signed-off-by: Juergen Gross <jgr...@suse.com> --- docs/misc/xenstore.txt | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/docs/misc/xenstore.txt b/docs/misc/xenstore.txt index ae1b6a8c6e..5051340227

[Xen-devel] [PATCH 0/3] docs: add some missing xenstore documentation

2017-05-08 Thread Juergen Gross
There were some bits missing in docs/misc/xenstore.txt, so add them. We might want to include this in 4.9, but I'm not feeling really strong about this. Resending with correct email address of Julien. Juergen Gross (3): docs: specify endianess of xenstore protocol header docs: add

[Xen-devel] [PATCH 3/3] docs: document CONTROL command of xenstore protocol

2017-05-08 Thread Juergen Gross
The CONTROL command (former DEBUG command) isn't specified in the xenstore protocol doc. Add it. Signed-off-by: Juergen Gross <jgr...@suse.com> --- docs/misc/xenstore.txt | 29 +++-- 1 file changed, 23 insertions(+), 6 deletions(-) diff --git a/docs/misc/xenstore

[Xen-devel] [PATCH 2/3] docs: add DIRECTORY_PART specification do xenstore protocol doc

2017-05-08 Thread Juergen Gross
DIRECTORY_PART was missing in docs/misc/xenstore.txt. Add it. Signed-off-by: Juergen Gross <jgr...@suse.com> --- docs/misc/xenstore.txt | 9 + 1 file changed, 9 insertions(+) diff --git a/docs/misc/xenstore.txt b/docs/misc/xenstore.txt index 5051340227..fd38d781e2 100644 --- a/doc

[Xen-devel] [PATCH 2/3] docs: add DIRECTORY_PART specification do xenstore protocol doc

2017-05-08 Thread Juergen Gross
DIRECTORY_PART was missing in docs/misc/xenstore.txt. Add it. Signed-off-by: Juergen Gross <jgr...@suse.com> --- docs/misc/xenstore.txt | 9 + 1 file changed, 9 insertions(+) diff --git a/docs/misc/xenstore.txt b/docs/misc/xenstore.txt index 5051340227..fd38d781e2 100644 --- a/doc

[Xen-devel] [PATCH 0/3] docs: add some missing xenstore documentation

2017-05-08 Thread Juergen Gross
There were some bits missing in docs/misc/xenstore.txt, so add them. We might want to include this in 4.9, but I'm not feeling really strong about this. Juergen Gross (3): docs: specify endianess of xenstore protocol header docs: add DIRECTORY_PART specification do xenstore protocol doc

[Xen-devel] [PATCH 3/3] docs: document CONTROL command of xenstore protocol

2017-05-08 Thread Juergen Gross
The CONTROL command (former DEBUG command) isn't specified in the xenstore protocol doc. Add it. Signed-off-by: Juergen Gross <jgr...@suse.com> --- docs/misc/xenstore.txt | 29 +++-- 1 file changed, 23 insertions(+), 6 deletions(-) diff --git a/docs/misc/xenstore

[Xen-devel] [PATCH 1/3] docs: specify endianess of xenstore protocol header

2017-05-08 Thread Juergen Gross
The endianess of the xenstore protocol header should be specified. Signed-off-by: Juergen Gross <jgr...@suse.com> --- docs/misc/xenstore.txt | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/docs/misc/xenstore.txt b/docs/misc/xenstore.txt index ae1b6a8c6e..5051340227

[Xen-devel] [stable-4.11: PATCH] xen: revert commits 72a9b186292 and da72ff5bfcb0

2017-05-08 Thread Juergen Gross
oject.org Cc: linux-ker...@vger.kernel.org Cc: linux-...@vger.kernel.org Cc: Anthony Liguori <aligu...@amazon.com> Cc: KarimAllah Ahmed <karah...@amazon.de> Signed-off-by: Juergen Gross <jgr...@suse.com> --- Please apply to stable-4.11.y --- arch/x86/include/asm/xen/events.h | 11

[Xen-devel] [stable-4.9: PATCH] xen: revert commit 72a9b186292

2017-05-07 Thread Juergen Gross
<paul.gortma...@windriver.com> Cc: Ross Lagerwall <ross.lagerw...@citrix.com> Cc: xen-de...@lists.xenproject.org Cc: linux-ker...@vger.kernel.org Cc: linux-...@vger.kernel.org Cc: Anthony Liguori <aligu...@amazon.com> Cc: KarimAllah Ahmed <karah...@amazon.de> Signed-o

[Xen-devel] [stable-4.10: PATCH] xen: revert commits 72a9b186292 and da72ff5bfcb0

2017-05-07 Thread Juergen Gross
oject.org Cc: linux-ker...@vger.kernel.org Cc: linux-...@vger.kernel.org Cc: Anthony Liguori <aligu...@amazon.com> Cc: KarimAllah Ahmed <karah...@amazon.de> Signed-off-by: Juergen Gross <jgr...@suse.com> --- Please apply to stable-4.10.y --- arch/x86/include

Re: [Xen-devel] [PATCH] xen/x86: Do not call xen_init_time_ops() until shared_info is initialized

2017-05-05 Thread Juergen Gross
On 03/05/17 22:20, Boris Ostrovsky wrote: > Routines that are set by xen_init_time_ops() use shared_info's > pvclock_vcpu_time_info area. This area is not properly available until > shared_info is mapped in xen_setup_shared_info(). > > This became especially problematic due to commit dd759d93f4dd

[Xen-devel] [stable-4.10: PATCH] xen: revert commits 72a9b186292 and da72ff5bfcb0

2017-05-05 Thread Juergen Gross
x-ker...@vger.kernel.org Cc: linux-...@vger.kernel.org Cc: Anthony Liguori <aligu...@amazon.com> Cc: KarimAllah Ahmed <karah...@amazon.de> Signed-off-by: Juergen Gross <jgr...@suse.com> --- Please apply to stable-4.10.y and stable-4.11.y --- arch/x86/include

[Xen-devel] [stable-4.9: PATCH] xen: revert commit 72a9b186292

2017-05-05 Thread Juergen Gross
;paul.gortma...@windriver.com> Cc: Ross Lagerwall <ross.lagerw...@citrix.com> Cc: xen-de...@lists.xenproject.org Cc: linux-ker...@vger.kernel.org Cc: linux-...@vger.kernel.org Cc: Anthony Liguori <aligu...@amazon.com> Cc: KarimAllah Ahmed <karah...@amazon.de> Signed-off-by: Ju

Re: [Xen-devel] [PATCH v3] x86/amd: don't set X86_BUG_SYSRET_SS_ATTRS when running under Xen

2017-05-04 Thread Juergen Gross
Any comments? Juergen On 27/04/17 07:01, Juergen Gross wrote: > When running as Xen pv guest X86_BUG_SYSRET_SS_ATTRS must not be set > on AMD cpus. > > This bug/feature bit is kind of special as it will be used very early > when switching threads. Setting the bit and clearing

Re: [Xen-devel] [PATCH] xen/x86: Do not call xen_init_time_ops() until shared_info is initialized

2017-05-04 Thread Juergen Gross
it dd759d93f4dd ("x86/timers: > Add simple udelay calibration") where we end up reading tsc_to_system_mul > from xen_dummy_shared_info (i.e. getting zero value) and then trying > to divide by it in pvclock_tsc_khz(). > > Signed-off-by: Boris Ostrovsky <boris.ostrov...@o

[Xen-devel] [GIT PULL] xen: fixes and features for 4.12

2017-05-03 Thread Juergen Gross
Stabellini adding a new 9pfs xen frontend driver - a series by Juergen Gross to simplify Xen's cpuid handling by making use of cpu capabilities - a series by Oleksandr Andrushchenko adding(modifying some headers for new Xen paravirtualized devices - a series by Julien Grall for EFI reset_system

Re: [Xen-devel] null domains after xl destroy

2017-05-03 Thread Juergen Gross
debug output to make sure we see what is happening. Juergen commit 246aaf60bd934b7571944b98a31078d519d637c6 Author: Juergen Gross <jgr...@suse.com> Date: Wed May 3 15:57:18 2017 +0200 xen/blkback: don't free be structure too early The be structure must nor be freed

Re: [Xen-devel] null domains after xl destroy

2017-05-03 Thread Juergen Gross
already said: this is a complete mess. Working on a patch now... BTW: Glenn, the debug patch isn't important any longer. It was just meant to locate the problem which is now known. Juergen > > On 01/05/17 10:55, Glenn Enright wrote: >> On 19/04/17 22:09, Juergen Gross wrote: >>> On 19/0

Re: [Xen-devel] [PATCH] xen: Move xen_have_vector_callback definition to enlighten.c

2017-05-02 Thread Juergen Gross
On 02/05/17 19:23, Boris Ostrovsky wrote: > Commit 84d582d236dc ("xen: Revert commits da72ff5bfcb0 and > 72a9b186292d") defined xen_have_vector_callback in enlighten_hvm.c. > Since guest-type-neutral code refers to this variable this causes > build failures when CONFIG_XEN_PVHVM is not defined. >

Re: [Xen-devel] [PATCH] xen: Move xen_have_vector_callback definition to enlighten.c

2017-05-02 Thread Juergen Gross
N_PVHVM is not defined. > > Moving xen_have_vector_callback definition to enlighten.c resolves > this issue. > > Signed-off-by: Boris Ostrovsky <boris.ostrov...@oracle.com> > Reported-by: Randy Dunlap <rdun...@infradead.org> Reviewed-by

Re: [Xen-devel] [PATCH 0/3 v2] xen: Implement EFI reset_system callback

2017-05-02 Thread Juergen Gross
ris.ostrov...@oracle.com> > Cc: Juergen Gross <jgr...@suse.com> Series rebased and pushed to xen/tip for-linus-4.12b Juergen ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH 0/3 v2] xen: Implement EFI reset_system callback

2017-05-02 Thread Juergen Gross
it will not be possible to reboot/power off ARM64 DOM0 >> when using ACPI. >> >> Cheers, >> >> Cc: Boris Ostrovsky <boris.ostrov...@oracle.com> >> Cc: Juergen Gross <jgr...@suse.com> >> >> Julien Grall (3): >> xen: Export xen_reboo

Re: [Xen-devel] [PATCH] xen: Revert commits da72ff5bfcb0 and 72a9b186292d

2017-04-28 Thread Juergen Gross
On 24/04/17 21:04, Boris Ostrovsky wrote: > Recent discussion (http://marc.info/?l=xen-devel=149192184523741) > established that commit 72a9b186292d ("xen: Remove event channel > notification through Xen PCI platform device") (and thus commit > da72ff5bfcb0 ("partially revert "xen: Remove event

Re: [Xen-devel] [PATCH] xen/x86: Call xen_smp_intr_init_pv() on BSP

2017-04-28 Thread Juergen Gross
On 26/04/17 15:42, Boris Ostrovsky wrote: > Recent code rework that split handling ov PV, HVM and PVH guests into > separate files missed calling xen_smp_intr_init_pv() on CPU0. > > Add this call. > > Signed-off-by: Boris Ostrovsky > Reported-by: Sander Eikelenboom

Re: [Xen-devel] [PATCH 1/3 v2] xen: Export xen_reboot

2017-04-28 Thread Juergen Gross
On 25/04/17 06:02, Juergen Gross wrote: > On 24/04/17 20:21, Boris Ostrovsky wrote: >> On 04/24/2017 01:58 PM, Julien Grall wrote: >>> The helper xen_reboot will be called by the EFI code in a later patch. >>> >>> Note that the ARM version does not yet exist an

Re: [Xen-devel] [PATCH] xen/x86: Call xen_smp_intr_init_pv() on BSP

2017-04-27 Thread Juergen Gross
rted-by: Sander Eikelenboom <li...@eikelenboom.it> Reviewed-by: Juergen Gross <jgr...@suse.com> Thanks, Juergen ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

[Xen-devel] [PATCH v3] x86/amd: don't set X86_BUG_SYSRET_SS_ATTRS when running under Xen

2017-04-26 Thread Juergen Gross
has enlarged a little bit by using setup_clear_cpu_cap() instead of the hypervisor's set_cpu_features callback. It seems this larger window now makes it rather easy to hit the problem. The proper solution is to never set the bit in case of Xen. Signed-off-by: Juergen Gross <jgr...@suse.

Re: [Xen-devel] [PATCH] x86/amd: don't set X86_BUG_SYSRET_SS_ATTRS if forced to zero

2017-04-26 Thread Juergen Gross
On 27/04/17 00:04, Borislav Petkov wrote: > On Wed, Apr 26, 2017 at 08:24:12PM +0200, Juergen Gross wrote: >> I'm not feeling strong about it. So if you want to test for >> X86_FEATURE_XENPV to avoid setting X86_BUG_SYSRET_SS_ATTRS I'm fine >> with it. >> >&

[Xen-devel] [PATCH v2] x86/amd: don't set X86_BUG_SYSRET_SS_ATTRS when running under Xen

2017-04-26 Thread Juergen Gross
has enlarged a little bit by using setup_clear_cpu_cap() instead of the hypervisor's set_cpu_features callback. It seems this larger window now makes it rather easy to hit the problem. The proper solution is to never set the bit in case of Xen. Signed-off-by: Juergen Gross <jgr...@suse.

Re: [Xen-devel] [PATCH] x86/amd: don't set X86_BUG_SYSRET_SS_ATTRS if forced to zero

2017-04-26 Thread Juergen Gross
On 26/04/17 08:35, Borislav Petkov wrote: > On Wed, Apr 26, 2017 at 06:45:42AM +0200, Juergen Gross wrote: >> The really clean solution would be to add this test to set_cpu_bug() > > No, the really clean solution is to set it once and not play toggle > games. > >>

Re: [Xen-devel] David Vrabel mail bouncing

2017-04-25 Thread Juergen Gross
On 26/04/17 07:03, Stephen Rothwell wrote: > Hi all, > > David's citrix.com email address is bouncing. Is there a new one I should > use for the xen-tip tree contact, or should I just remove him? > Just remove him, please. He is no longer maintainer of the Xen tree. Juergen

Re: [Xen-devel] [PATCH] x86/amd: don't set X86_BUG_SYSRET_SS_ATTRS if forced to zero

2017-04-25 Thread Juergen Gross
On 25/04/17 21:18, Borislav Petkov wrote: > On Tue, Apr 25, 2017 at 08:34:34PM +0200, Juergen Gross wrote: >> And what happens when there is a scheduling event right here? >> __switch_to() will see X86_BUG_SYSRET_SS_ATTRS set and take a wrong >> path. > > So the whole

Re: [Xen-devel] [PATCH] x86/amd: don't set X86_BUG_SYSRET_SS_ATTRS if forced to zero

2017-04-25 Thread Juergen Gross
On 25/04/17 20:24, Borislav Petkov wrote: > On Tue, Apr 25, 2017 at 08:00:14PM +0200, Juergen Gross wrote: >> When running as Xen pv guest X86_BUG_SYSRET_SS_ATTRS must not be set >> on AMD cpus. Xen will disable this via setup_clear_cpu_cap(), so test >> cpu_caps_cleared

[Xen-devel] [PATCH] x86/amd: don't set X86_BUG_SYSRET_SS_ATTRS if forced to zero

2017-04-25 Thread Juergen Gross
solution is to never set the bit in case of Xen. Signed-off-by: Juergen Gross <jgr...@suse.com> --- arch/x86/kernel/cpu/amd.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/arch/x86/kernel/cpu/amd.c b/arch/x86/kernel/cpu/amd.c index c36140d788fe..f659b6f534b7

Re: [Xen-devel] Resend: Linux 4.11-rc7: kernel BUG at drivers/xen/events/events_base.c:1221

2017-04-25 Thread Juergen Gross
On 25/04/17 15:12, Sander Eikelenboom wrote: > On 25/04/17 14:49, Juergen Gross wrote: >> On 25/04/17 12:33, Sander Eikelenboom wrote: >>> On 25/04/17 09:01, Juergen Gross wrote: >>>> On 25/04/17 08:57, Sander Eikelenboom wrote: >>>>> On 25/04/17 08:4

Re: [Xen-devel] Resend: Linux 4.11-rc7: kernel BUG at drivers/xen/events/events_base.c:1221

2017-04-25 Thread Juergen Gross
On 25/04/17 12:33, Sander Eikelenboom wrote: > On 25/04/17 09:01, Juergen Gross wrote: >> On 25/04/17 08:57, Sander Eikelenboom wrote: >>> On 25/04/17 08:42, Juergen Gross wrote: >>>> On 25/04/17 08:35, Sander Eikelenboom wrote: >>>>> (XEN) [2017-04-2

Re: [Xen-devel] Resend: Linux 4.11-rc7: kernel BUG at drivers/xen/events/events_base.c:1221

2017-04-25 Thread Juergen Gross
On 25/04/17 13:28, Sander Eikelenboom wrote: > On 25/04/17 13:00, Juergen Gross wrote: >> On 25/04/17 12:33, Sander Eikelenboom wrote: >>> On 25/04/17 09:01, Juergen Gross wrote: >>>> On 25/04/17 08:57, Sander Eikelenboom wrote: >>>>> On 25/04/17 08:4

Re: [Xen-devel] Resend: Linux 4.11-rc7: kernel BUG at drivers/xen/events/events_base.c:1221

2017-04-25 Thread Juergen Gross
On 25/04/17 12:33, Sander Eikelenboom wrote: > On 25/04/17 09:01, Juergen Gross wrote: >> On 25/04/17 08:57, Sander Eikelenboom wrote: >>> On 25/04/17 08:42, Juergen Gross wrote: >>>> On 25/04/17 08:35, Sander Eikelenboom wrote: >>>>> (XEN) [2017-04-2

Re: [Xen-devel] Resend: Linux 4.11-rc7: kernel BUG at drivers/xen/events/events_base.c:1221

2017-04-25 Thread Juergen Gross
On 25/04/17 12:33, Sander Eikelenboom wrote: > On 25/04/17 09:01, Juergen Gross wrote: >> On 25/04/17 08:57, Sander Eikelenboom wrote: >>> On 25/04/17 08:42, Juergen Gross wrote: >>>> On 25/04/17 08:35, Sander Eikelenboom wrote: >>>>> (XEN) [2017-04-2

Re: [Xen-devel] [PATCH] xen: Revert commits da72ff5bfcb0 and 72a9b186292d

2017-04-25 Thread Juergen Gross
nce the only other mode > supported is INTx which. > > So based on this summary, I think before commit (72a9b186292) we were > in much better position from a user point of view. > > Signed-off-by: Boris Ostrovsky <boris.ostrov...@oracle.com> Reviewed-by: Juerge

Re: [Xen-devel] Resend: Linux 4.11-rc7: kernel BUG at drivers/xen/events/events_base.c:1221

2017-04-25 Thread Juergen Gross
On 25/04/17 08:57, Sander Eikelenboom wrote: > On 25/04/17 08:42, Juergen Gross wrote: >> On 25/04/17 08:35, Sander Eikelenboom wrote: >>> (XEN) [2017-04-24 21:20:53.203] d0v0 Unhandled invalid opcode fault/trap >>> [#6, ec=] >>> (XEN) [2017-04-24 21

Re: [Xen-devel] [PATCH v2] xen/pvh: Do not fill kernel's e820 map in init_pvh_bootparams()

2017-04-25 Thread Juergen Gross
On 21/04/17 17:13, Boris Ostrovsky wrote: > e820 map is updated with information from the zeropage (i.e. pvh_bootparams) > by default_machine_specific_memory_setup(). With the way things are done > now, we end up with a duplicated e820 map. > > Signed-off-by: Boris Ostrovsky

Re: [Xen-devel] [PATCH] xen/scsifront: use offset_in_page() macro

2017-04-25 Thread Juergen Gross
On 22/04/17 03:21, Geliang Tang wrote: > Use offset_in_page() macro instead of open-coding. > > Signed-off-by: Geliang Tang Pushed to xen/tip for-linus-4.12 Thanks, Juergen ___ Xen-devel mailing list Xen-devel@lists.xen.org

[Xen-devel] [PATCH] x86/xen: fix xsave capability setting

2017-04-25 Thread Juergen Gross
Commit 690b7f10b4f9f ("x86/xen: use capabilities instead of fake cpuid values for xsave") introduced a regression as it tried to make use of the fixup feature before it being available. Fall back to the old variant testing via cpuid(). Signed-off-by: Juergen Gross <jgr...@suse.com&g

Re: [Xen-devel] Resend: Linux 4.11-rc7: kernel BUG at drivers/xen/events/events_base.c:1221

2017-04-25 Thread Juergen Gross
On 25/04/17 08:35, Sander Eikelenboom wrote: > On 25/04/17 08:14, Juergen Gross wrote: >> On 24/04/17 22:15, Boris Ostrovsky wrote: >>> On 04/24/2017 12:10 PM, Sander Eikelenboom wrote: >>>> On 24/04/17 17:49, Boris Ostrovsky wrote: >>>>> On 04/24/2017

Re: [Xen-devel] Resend: Linux 4.11-rc7: kernel BUG at drivers/xen/events/events_base.c:1221

2017-04-25 Thread Juergen Gross
On 24/04/17 22:15, Boris Ostrovsky wrote: > On 04/24/2017 12:10 PM, Sander Eikelenboom wrote: >> On 24/04/17 17:49, Boris Ostrovsky wrote: >>> On 04/24/2017 10:20 AM, Sander Eikelenboom wrote: Hi Boris, Nope, not that i am aware of. >>> If you can keep console while running this,

Re: [Xen-devel] [PATCH] xen/scsifront: use offset_in_page() macro

2017-04-24 Thread Juergen Gross
On 25/04/17 00:15, Martin K. Petersen wrote: > > Juergen, > >> On 22/04/17 03:21, Geliang Tang wrote: >>> Use offset_in_page() macro instead of open-coding. >>> >>> Signed-off-by: Geliang Tang <geliangt...@gmail.com> >> >> Reviewed-

Re: [Xen-devel] [PATCH 1/3 v2] xen: Export xen_reboot

2017-04-24 Thread Juergen Gross
t; >> Signed-off-by: Julien Grall <julien.gr...@arm.com> > > I don't think these changes are worth a whole patch. They can be folded > into the third patch. No, the 2nd patch needs this, too. So: Reviewed-by: Juergen Gross <jgr...@suse.com> Thanks, Juergen __

Re: [Xen-devel] [PATCH v2] xen/pvh: Do not fill kernel's e820 map in init_pvh_bootparams()

2017-04-24 Thread Juergen Gross
y <boris.ostrov...@oracle.com> Reviewed-by: Juergen Gross <jgr...@suse.com> Juergen ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v3 09/11] x86/xen: use capabilities instead of fake cpuid values for xsave

2017-04-24 Thread Juergen Gross
On 21/04/17 16:51, Boris Ostrovsky wrote: > On 04/21/2017 10:45 AM, Andrew Cooper wrote: >> On 21/04/17 15:38, Juergen Gross wrote: >>> On 21/04/17 16:24, Boris Ostrovsky wrote: >>>>> +static bool __init xen_check_xsave(void) >>>>> { >>&

Re: [Xen-devel] [PATCH] xen/scsifront: use offset_in_page() macro

2017-04-23 Thread Juergen Gross
On 22/04/17 03:21, Geliang Tang wrote: > Use offset_in_page() macro instead of open-coding. > > Signed-off-by: Geliang Tang <geliangt...@gmail.com> Reviewed-by: Juergen Gross <jgr...@suse.com> Thanks, Juergen ___ Xen-devel m

Re: [Xen-devel] [PATCH v3 09/11] x86/xen: use capabilities instead of fake cpuid values for xsave

2017-04-21 Thread Juergen Gross
On 21/04/17 16:24, Boris Ostrovsky wrote: > >> +static bool __init xen_check_xsave(void) >> { >> -unsigned int ax, bx, cx, dx; >> -unsigned int xsave_mask; >> +unsigned int err, eax, edx; >> >> -ax = 1; >> -cx = 0; >> -cpuid(1, , , , ); >> +/* Test OSXSAVE

Re: [Xen-devel] [PATCH] xen/pvh: Do not fill kernel's e820 map in init_pvh_bootparams()

2017-04-21 Thread Juergen Gross
On 20/04/17 19:32, Boris Ostrovsky wrote: > e820 map is updated with information from the zeropage (i.e. > pvh_bootparams) by default_machine_specific_memory_setup(). > With the way things are done now, we end up with a duplicated > e820 map. > > Signed-off-by: Boris Ostrovsky

[Xen-devel] [PATCH] xen/blkback: fix disconnect while I/Os in flight

2017-04-21 Thread Juergen Gross
call. This will lead to inconsistencies in the refcnt handling. Cc: sta...@vger.kernel.org Signed-off-by: Juergen Gross <jgr...@suse.com> Tested-by: Steven Haigh <net...@crc.id.au> --- drivers/block/xen-blkback/common.h | 1 + drivers/block/xen-blkback/xenbus.c | 7 +-- 2 file

Re: [Xen-devel] null domains after xl destroy

2017-04-21 Thread Juergen Gross
On 21/04/17 10:42, Steven Haigh wrote: > On 20/04/17 02:22, Steven Haigh wrote: >> On 19/04/17 20:09, Juergen Gross wrote: >>> On 19/04/17 09:16, Roger Pau Monné wrote: >>>> On Wed, Apr 19, 2017 at 06:39:41AM +0200, Juergen Gross wrote: >>>>> On 19/04

Re: [Xen-devel] [PATCH] xen/pvh: Do not fill kernel's e820 map in init_pvh_bootparams()

2017-04-21 Thread Juergen Gross
vsky <boris.ostrov...@oracle.com> Reviewed-by: Juergen Gross <jgr...@suse.com> Thanks, Juergen > --- > This patch is against for-linus-4.12 branch. Since this is not > a critical issue I don't want to submit it to 4.11 at rc8 time > (plus it will require rebasing for-linus-4.

Re: [Xen-devel] [PATCH] xen/9pfs: select CONFIG_XEN_XENBUS_FRONTEND

2017-04-19 Thread Juergen Gross
On 20/04/17 01:10, Stefano Stabellini wrote: > Juergen, I have committed this patch to for-linus-4.12 and linux-next, I > hope that's OK. Sure. Juergen > > Og Wed, 19 Apr 2017, Stefano Stabellini wrote: >> On Wed, 19 Apr 2017, Arnd Bergmann wrote: >>> All Xen frontends need to select this

Re: [Xen-devel] [PATCH 1/2] xen/arm, arm64: fix xen_dma_ops after 815dd18 "Consolidate get_dma_ops..."

2017-04-19 Thread Juergen Gross
On 19/04/17 19:25, Stefano Stabellini wrote: > Hello Russell, > > Can I have your acked-by on the following fix (original patch is > 1492117462-19886-1-git-send-email-sstabell...@kernel.org)? Stefano, through which tree should this go? ARM or Xen or other? Juergen > > Thanks, > > Stefano >

Re: [Xen-devel] [PATCH] x86/vlapic: Don't reset APIC mode/ID when handling INIT signal

2017-04-19 Thread Juergen Gross
On 19/04/17 15:58, Juergen Gross wrote: > On 19/04/17 15:43, Boris Ostrovsky wrote: >> >> >> On 04/19/2017 09:21 AM, Jan Beulich wrote: >>>>>> On 19.04.17 at 06:56, <chao@intel.com> wrote: >>>> On Wed, Apr 19, 2017 at 02:48:57AM -0600

Re: [Xen-devel] [PATCH] x86/vlapic: Don't reset APIC mode/ID when handling INIT signal

2017-04-19 Thread Juergen Gross
On 19/04/17 15:43, Boris Ostrovsky wrote: > > > On 04/19/2017 09:21 AM, Jan Beulich wrote: > On 19.04.17 at 06:56, wrote: >>> On Wed, Apr 19, 2017 at 02:48:57AM -0600, Jan Beulich wrote: >>> On 18.04.17 at 23:51, wrote: > HVM guest can't

Re: [Xen-devel] [PATCH v3] xen, input: add xen-kbdfront module parameter for setting resolution

2017-04-19 Thread Juergen Gross
On 12/04/17 20:26, Juergen Gross wrote: > On 12/04/17 18:24, Dmitry Torokhov wrote: >> On Wed, Apr 12, 2017 at 06:04:30PM +0200, Juergen Gross wrote: >>> On 12/04/17 17:16, Dmitry Torokhov wrote: >>>> Hi Juergen, >>>> >>>> On Tue, Apr 11, 20

Re: [Xen-devel] null domains after xl destroy

2017-04-19 Thread Juergen Gross
On 19/04/17 09:16, Roger Pau Monné wrote: > On Wed, Apr 19, 2017 at 06:39:41AM +0200, Juergen Gross wrote: >> On 19/04/17 03:02, Glenn Enright wrote: >>> Thanks Juergen. I applied that, to our 4.9.23 dom0 kernel, which still >>> shows the issue. When replicating the lea

Re: [Xen-devel] null domains after xl destroy

2017-04-19 Thread Juergen Gross
On 19/04/17 09:16, Roger Pau Monné wrote: > On Wed, Apr 19, 2017 at 06:39:41AM +0200, Juergen Gross wrote: >> On 19/04/17 03:02, Glenn Enright wrote: >>> On 18/04/17 20:36, Juergen Gross wrote: >>>> On 12/04/17 00:45, Glenn Enright wrote: >>>>> On 12/0

Re: [Xen-devel] null domains after xl destroy

2017-04-18 Thread Juergen Gross
On 19/04/17 03:02, Glenn Enright wrote: > On 18/04/17 20:36, Juergen Gross wrote: >> On 12/04/17 00:45, Glenn Enright wrote: >>> On 12/04/17 10:23, Andrew Cooper wrote: >>>> On 11/04/2017 23:13, Glenn Enright wrote: >>>>> On 11/04/17 21:49, Dietmar H

Re: [Xen-devel] [PATCH] arm64: xen: Implement EFI reset_system callback

2017-04-18 Thread Juergen Gross
On 18/04/17 20:46, Stefano Stabellini wrote: > On Tue, 18 Apr 2017, Juergen Gross wrote: >> On 18/04/17 20:37, Stefano Stabellini wrote: >>> On Thu, 6 Apr 2017, Juergen Gross wrote: >>>> On 06/04/17 18:43, Daniel Kiper wrote: >>>>> On Thu, Apr 06,

Re: [Xen-devel] [PATCH] arm64: xen: Implement EFI reset_system callback

2017-04-18 Thread Juergen Gross
On 18/04/17 20:37, Stefano Stabellini wrote: > On Thu, 6 Apr 2017, Juergen Gross wrote: >> On 06/04/17 18:43, Daniel Kiper wrote: >>> On Thu, Apr 06, 2017 at 06:22:44PM +0200, Juergen Gross wrote: >>>> On 06/04/17 18:06, Daniel Kiper wrote: >>>>> Hi J

Re: [Xen-devel] [stable-4.10: PATCH] xen: revert commits 72a9b186292 and da72ff5bfcb0

2017-04-18 Thread Juergen Gross
On 18/04/17 15:52, Greg KH wrote: > On Thu, Apr 13, 2017 at 06:55:54PM +0200, Greg KH wrote: >> On Thu, Apr 13, 2017 at 06:28:33PM +0200, Juergen Gross wrote: >>> On 13/04/17 18:24, Greg KH wrote: >>>> On Thu, Apr 13, 2017 at 04:49:49PM +0200, Juergen Gross wrote: &

Re: [Xen-devel] [PATCH v3 09/11] x86/xen: use capabilities instead of fake cpuid values for xsave

2017-04-18 Thread Juergen Gross
On 18/04/17 12:02, Andrew Cooper wrote: > On 18/04/17 07:31, Juergen Gross wrote: >> @@ -281,22 +274,24 @@ static bool __init xen_check_mwait(void) >> return false; >> #endif >> } >> -static void __init xen_init_cpuid_mask(void) >> + >

Re: [Xen-devel] null domains after xl destroy

2017-04-18 Thread Juergen Gross
On 12/04/17 00:45, Glenn Enright wrote: > On 12/04/17 10:23, Andrew Cooper wrote: >> On 11/04/2017 23:13, Glenn Enright wrote: >>> On 11/04/17 21:49, Dietmar Hahn wrote: >>>> Am Dienstag, 11. April 2017, 20:03:14 schrieb Glenn Enright: >>>>> On 11/04/1

Re: [Xen-devel] [PATCH] xen/9pfs: initialize len to 0 to detect xenbus_read errors

2017-04-18 Thread Juergen Gross
On 13/04/17 19:57, Stefano Stabellini wrote: > In order to use "len" to check for xenbus_read errors properly, we need > to initialize len to 0 before passing it to xenbus_read. > > Signed-off-by: Stefano Stabellini > CC: dan.carpen...@oracle.com > CC: jgr...@suse.com > CC:

[Xen-devel] [PATCH v3 05/11] x86/xen: use capabilities instead of fake cpuid values for acc

2017-04-18 Thread Juergen Gross
. Signed-off-by: Juergen Gross <jgr...@suse.com> Reviewed-by: Boris Ostrovsky <boris.ostrov...@oracle.com> --- arch/x86/xen/enlighten_pv.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/arch/x86/xen/enlighten_pv.c b/arch/x86/xen/enlighten_pv.c index 37e174d9009b..

[Xen-devel] [PATCH v3 10/11] vmware: set cpu capabilities during platform initialization

2017-04-18 Thread Juergen Gross
quot; <h...@zytor.com> Cc: x...@kernel.org Cc: virtualizat...@lists.linux-foundation.org Signed-off-by: Juergen Gross <jgr...@suse.com> Reviewed-by: Boris Ostrovsky <boris.ostrov...@oracle.com> Acked-by: Alok Kataria <akata...@vmware.com> --- arch/x86/kernel/cpu/vmware.c | 39 +++

[Xen-devel] [PATCH v3 09/11] x86/xen: use capabilities instead of fake cpuid values for xsave

2017-04-18 Thread Juergen Gross
-by: Juergen Gross <jgr...@suse.com> --- arch/x86/xen/enlighten_pv.c | 46 +++-- 1 file changed, 24 insertions(+), 22 deletions(-) diff --git a/arch/x86/xen/enlighten_pv.c b/arch/x86/xen/enlighten_pv.c index 38dec28a8e6d..e6bf71d76e10 100644 --- a/arch/x

[Xen-devel] [PATCH v3 11/11] x86/cpu: remove hypervisor specific set_cpu_features

2017-04-18 Thread Juergen Gross
There is no user of x86_hyper->set_cpu_features() any more. Remove it. Cc: Thomas Gleixner <t...@linutronix.de> Cc: Ingo Molnar <mi...@redhat.com> Cc: "H. Peter Anvin" <h...@zytor.com> Cc: x...@kernel.org Signed-off-by: Juergen Gross <jgr...@suse.com> Revie

[Xen-devel] [PATCH v3 01/11] xen: set cpu capabilities from xen_start_kernel()

2017-04-18 Thread Juergen Gross
There is no need to set the same capabilities for each cpu individually. This can easily be done for all cpus when starting the kernel. Signed-off-by: Juergen Gross <jgr...@suse.com> Reviewed-by: Boris Ostrovsky <boris.ostrov...@oracle.com> --- arch/x86/xen/enlighten_pv.c | 14 +

[Xen-devel] [PATCH v3 00/11] x86: xen cpuid() cleanup

2017-04-18 Thread Juergen Gross
rg Cc: virtualizat...@lists.linux-foundation.org Juergen Gross (11): xen: set cpu capabilities from xen_start_kernel() x86/xen: don't indicate DCA support in pv domains x86/xen: use capabilities instead of fake cpuid values for aperf x86/xen: use capabilities instead of fake cpuid values for mtrr

[Xen-devel] [PATCH v3 06/11] x86/xen: use capabilities instead of fake cpuid values for acpi

2017-04-18 Thread Juergen Gross
for acpi instead. Signed-off-by: Juergen Gross <jgr...@suse.com> Reviewed-by: Boris Ostrovsky <boris.ostrov...@oracle.com> --- arch/x86/xen/enlighten_pv.c | 11 +++ 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/arch/x86/xen/enlighten_pv.c b/arch/x86/xen/enlighte

[Xen-devel] [PATCH v3 02/11] x86/xen: don't indicate DCA support in pv domains

2017-04-18 Thread Juergen Gross
Xen doesn't support DCA (direct cache access) for pv domains. Clear the corresponding capability indicator. Signed-off-by: Juergen Gross <jgr...@suse.com> Reviewed-by: Boris Ostrovsky <boris.ostrov...@oracle.com> --- arch/x86/xen/enlighten_pv.c | 1 + 1 file changed, 1 insertion(+)

[Xen-devel] [PATCH v3 03/11] x86/xen: use capabilities instead of fake cpuid values for aperf

2017-04-18 Thread Juergen Gross
-off-by: Juergen Gross <jgr...@suse.com> Reviewed-by: Boris Ostrovsky <boris.ostrov...@oracle.com> --- arch/x86/xen/enlighten_pv.c | 9 + 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/arch/x86/xen/enlighten_pv.c b/arch/x86/xen/enlighten_pv.c index bd69868909b4..

[Xen-devel] [PATCH v3 07/11] x86/xen: use capabilities instead of fake cpuid values for mwait

2017-04-18 Thread Juergen Gross
When running as pv domain xen_cpuid() is being used instead of native_cpuid(). In xen_cpuid() the mwait feature is indicated to be present or not by special casing the related cpuid leaf. Instead of delivering fake cpuid values use the cpu capability bit for mwait instead. Signed-off-by: Juergen

[Xen-devel] [PATCH v3 04/11] x86/xen: use capabilities instead of fake cpuid values for mtrr

2017-04-18 Thread Juergen Gross
When running as pv domain xen_cpuid() is being used instead of native_cpuid(). In xen_cpuid() the mtrr feature is indicated as not being present by special casing the related cpuid leaf. Instead of delivering fake cpuid values clear the cpu capability bit for mtrr instead. Signed-off-by: Juergen

[Xen-devel] [PATCH v3 08/11] x86/xen: use capabilities instead of fake cpuid values for x2apic

2017-04-18 Thread Juergen Gross
-by: Juergen Gross <jgr...@suse.com> Reviewed-by: Boris Ostrovsky <boris.ostrov...@oracle.com> --- arch/x86/xen/enlighten_pv.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/arch/x86/xen/enlighten_pv.c b/arch/x86/xen/enlighten_pv.c index d43382284ada..38dec28a8e6d 10064

Re: [Xen-devel] [Qemu-devel][PATCH] configure: introduce --enable-xen-fb-backend

2017-04-17 Thread Juergen Gross
On 14/04/17 19:52, Stefano Stabellini wrote: > On Fri, 14 Apr 2017, Juergen Gross wrote: >> On 14/04/17 08:06, Oleksandr Andrushchenko wrote: >>> On 04/14/2017 03:12 AM, Stefano Stabellini wrote: >>>> On Tue, 11 Apr 2017, Oleksandr Andrushchenko wrote: >&

Re: [Xen-devel] [Qemu-devel][PATCH] configure: introduce --enable-xen-fb-backend

2017-04-14 Thread Juergen Gross
On 14/04/17 08:06, Oleksandr Andrushchenko wrote: > On 04/14/2017 03:12 AM, Stefano Stabellini wrote: >> On Tue, 11 Apr 2017, Oleksandr Andrushchenko wrote: >>> From: Oleksandr Andrushchenko >>> >>> For some use cases when Xen framebuffer/input backend >>> is not

Re: [Xen-devel] [stable-4.10: PATCH] xen: revert commits 72a9b186292 and da72ff5bfcb0

2017-04-13 Thread Juergen Gross
On 13/04/17 18:24, Greg KH wrote: > On Thu, Apr 13, 2017 at 04:49:49PM +0200, Juergen Gross wrote: >> Revert commit 72a9b186292 ("xen: Remove event channel notification >> through Xen PCI platform device") as the original analysis was wrong >> that all the rem

[Xen-devel] [stable-4.10: PATCH] xen: revert commits 72a9b186292 and da72ff5bfcb0

2017-04-13 Thread Juergen Gross
lien Grall <julien.gr...@arm.com> Cc: Vitaly Kuznetsov <vkuzn...@redhat.com> Cc: Paul Gortmaker <paul.gortma...@windriver.com> Cc: Ross Lagerwall <ross.lagerw...@citrix.com> Cc: xen-de...@lists.xenproject.org Cc: linux-ker...@vger.kernel.org Cc: linux-...@vger.kernel.org Cc: Ant

[Xen-devel] [stable-4.9: PATCH] xen: revert commit 72a9b186292

2017-04-13 Thread Juergen Gross
erwall <ross.lagerw...@citrix.com> Cc: xen-de...@lists.xenproject.org Cc: linux-ker...@vger.kernel.org Cc: linux-...@vger.kernel.org Cc: Anthony Liguori <aligu...@amazon.com> Cc: KarimAllah Ahmed <karah...@amazon.de> Signed-off-by: Juergen Gross <jgr...@suse.com> --- Please apply to stable

[Xen-devel] [PATCH v2 03/11] x86/xen: use capabilities instead of fake cpuid values for aperf

2017-04-13 Thread Juergen Gross
-off-by: Juergen Gross <jgr...@suse.com> --- arch/x86/xen/enlighten_pv.c | 9 + 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/arch/x86/xen/enlighten_pv.c b/arch/x86/xen/enlighten_pv.c index bd69868909b4..4c8cd7278189 100644 --- a/arch/x86/xen/enlighten_pv.c +++ b/arch/x

[Xen-devel] [PATCH v2 09/11] x86/xen: use capabilities instead of fake cpuid values for xsave

2017-04-13 Thread Juergen Gross
-by: Juergen Gross <jgr...@suse.com> --- arch/x86/xen/enlighten_pv.c | 28 +++- 1 file changed, 11 insertions(+), 17 deletions(-) diff --git a/arch/x86/xen/enlighten_pv.c b/arch/x86/xen/enlighten_pv.c index 38dec28a8e6d..dad1db61e40b 100644 --- a/arch/x86/xen/enlighten_pv.c

[Xen-devel] [PATCH v2 10/11] vmware: set cpu capabilities during platform initialization

2017-04-13 Thread Juergen Gross
quot; <h...@zytor.com> Cc: x...@kernel.org Cc: virtualizat...@lists.linux-foundation.org Signed-off-by: Juergen Gross <jgr...@suse.com> --- arch/x86/kernel/cpu/vmware.c | 39 --- 1 file changed, 20 insertions(+), 19 deletions(-) diff --git a/arch/x86/k

[Xen-devel] [PATCH v2 04/11] x86/xen: use capabilities instead of fake cpuid values for mtrr

2017-04-13 Thread Juergen Gross
When running as pv domain xen_cpuid() is being used instead of native_cpuid(). In xen_cpuid() the mtrr feature is indicated as not being present by special casing the related cpuid leaf. Instead of delivering fake cpuid values clear the cpu capability bit for mtrr instead. Signed-off-by: Juergen

[Xen-devel] [PATCH v2 00/11] x86: xen cpuid() cleanup

2017-04-13 Thread Juergen Gross
com> Cc: Thomas Gleixner <t...@linutronix.de> Cc: Ingo Molnar <mi...@redhat.com> Cc: "H. Peter Anvin" <h...@zytor.com> Cc: x...@kernel.org Cc: virtualizat...@lists.linux-foundation.org Juergen Gross (11): xen: set cpu capabilities from xen_start_kernel() x86/xen

[Xen-devel] [PATCH v2 06/11] x86/xen: use capabilities instead of fake cpuid values for acpi

2017-04-13 Thread Juergen Gross
for acpi instead. Signed-off-by: Juergen Gross <jgr...@suse.com> --- arch/x86/xen/enlighten_pv.c | 11 +++ 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/arch/x86/xen/enlighten_pv.c b/arch/x86/xen/enlighten_pv.c index 53fe97bd720f..3196961862bc 100644 --- a/arch/x

<    6   7   8   9   10   11   12   13   14   15   >