Re: [Xen-devel] [PATCH] arm/monitor vm-events: Implement guest-request support

2016-02-19 Thread Stefano Stabellini
On Fri, 19 Feb 2016, Corneliu ZUZU wrote: > On 2/19/2016 3:49 PM, Stefano Stabellini wrote: > > On Thu, 18 Feb 2016, Corneliu ZUZU wrote: > > > + > > > +if ( sync ) > > > +{ > > > +req->flags |= VM_EVENT_FLAG_VCPU_PAUSED; > > > +vm_event_vcpu_pause(v); > > > +} > > > +

[Xen-devel] [xen-4.4-testing test] 83002: regressions - trouble: blocked/broken/fail/pass

2016-02-19 Thread osstest service owner
flight 83002 xen-4.4-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/83002/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qcow2 3 host-install(3) broken REGR. vs. 78924

Re: [Xen-devel] [PATCH] arm/monitor vm-events: Implement guest-request support

2016-02-19 Thread Corneliu ZUZU
On 2/19/2016 3:49 PM, Stefano Stabellini wrote: On Thu, 18 Feb 2016, Corneliu ZUZU wrote: + +if ( sync ) +{ +req->flags |= VM_EVENT_FLAG_VCPU_PAUSED; +vm_event_vcpu_pause(v); +} + +#if CONFIG_X86 +if ( altp2m_active(d) ) I would rather #define altp2m_active(d)

Re: [Xen-devel] [PATCH] xen/x86: Use 2M superpages for text/data/bss mappings

2016-02-19 Thread Andrew Cooper
On 19/02/16 14:58, Jan Beulich wrote: On 18.02.16 at 19:03, wrote: >> --- a/xen/arch/x86/setup.c >> +++ b/xen/arch/x86/setup.c >> @@ -921,13 +921,51 @@ void __init noreturn __start_xen(unsigned long mbi_p) >> /* The only data mappings to be relocated

Re: [Xen-devel] [PATCH] x86/PVHv2: add XEN_ prefix to HVM_START_MAGIC_VALUE

2016-02-19 Thread Roger Pau Monné
Ccing the tools maintainers, according to the output from get_maintaner.pl Also adding the Linux folks. El 19/2/16 a les 11:28, Roger Pau Monne ha escrit: > Signed-off-by: Roger Pau Monné > Reported by: Jan Beulich > --- > Cc: Jan Beulich

Re: [Xen-devel] [PATCHv1 3/5] x86/fpu: Add a per-domain field to set the width of FIP/FDP

2016-02-19 Thread David Vrabel
On 19/02/16 15:14, Jan Beulich wrote: On 19.02.16 at 15:49, wrote: >> On 19/02/16 14:36, Jan Beulich wrote: >> On 19.02.16 at 15:16, wrote: On 19/02/16 14:08, Jan Beulich wrote: On 18.02.16 at 19:52,

Re: [Xen-devel] [PATCH] xen/blback: Fit the important information of the thread in 17 characters

2016-02-19 Thread Roger Pau Monné
El 17/2/16 a les 17:31, Konrad Rzeszutek Wilk ha escrit: > The processes names are truncated to 17, while we had the length > of the process as name 20 - which meant that while we filled > it out with various details - the last 3 characters (which had > the queue number) never surfaced to the

[Xen-devel] [PATCH] tools: libxl: Simplify logic in libxl__realloc

2016-02-19 Thread Ian Jackson
Replace the loop exit and separate test for loop overrun with an assert in the loop body. This simplifies the code. It also (hopefully) avoids Coverity thinking that gc->alloc_maxsize might change, resulting in the loop failing to find the right answer but also failing to abort.

Re: [Xen-devel] [PATCH] libxl: add basic spice support for pv domUs

2016-02-19 Thread Fabio Fantoni
Il 16/02/2016 18:44, Wei Liu ha scritto: On Wed, Jan 13, 2016 at 11:47:55AM +0100, Fabio Fantoni wrote: [...] diff --git a/docs/man/xl.cfg.pod.5 b/docs/man/xl.cfg.pod.5 index 3b695bd..04a96ba 100644 --- a/docs/man/xl.cfg.pod.5 +++ b/docs/man/xl.cfg.pod.5 @@ -1607,82 +1607,6 @@ it. =back

[Xen-devel] [linux-4.1 test] 83005: regressions - FAIL

2016-02-19 Thread osstest service owner
flight 83005 linux-4.1 real [real] http://logs.test-lab.xenproject.org/osstest/logs/83005/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64-rumpuserxen 6 xen-build fail REGR. vs. 66399 build-i386-rumpuserxen

[Xen-devel] [PATCH] libxl: small indentation fix in libxl_types.idl

2016-02-19 Thread Fabio Fantoni
Signed-off-by: Fabio Fantoni --- tools/libxl/libxl_types.idl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/libxl/libxl_types.idl b/tools/libxl/libxl_types.idl index 9ad7eba..8eb24e1 100644 --- a/tools/libxl/libxl_types.idl +++

Re: [Xen-devel] [PATCH v8 05/13] tools/libxc: support to resume uncooperative HVM guests

2016-02-19 Thread Wei Liu
On Fri, Feb 19, 2016 at 02:52:11PM +, Ian Campbell wrote: > On Fri, 2016-02-19 at 14:43 +, Wei Liu wrote: > > On Fri, Feb 19, 2016 at 09:15:38AM -0500, Konrad Rzeszutek Wilk wrote: > > > On Thu, Feb 18, 2016 at 12:13:36PM +, Wei Liu wrote: > > > > On Thu, Feb 18, 2016 at 10:43:15AM

Re: [Xen-devel] [RFC v2 1/6] x86/boot: add BIT() to boot/bitops.h

2016-02-19 Thread Boris Ostrovsky
On 02/19/2016 09:15 AM, Luis R. Rodriguez wrote: diff --git a/arch/x86/boot/bitops.h b/arch/x86/boot/bitops.h index 878e4b9940d9..232cff0ff4e3 100644 --- a/arch/x86/boot/bitops.h +++ b/arch/x86/boot/bitops.h @@ -40,4 +40,6 @@ static inline void set_bit(int nr, void *addr) asm("btsl

Re: [Xen-devel] [PATCHv1 3/5] x86/fpu: Add a per-domain field to set the width of FIP/FDP

2016-02-19 Thread Jan Beulich
>>> On 19.02.16 at 15:49, wrote: > On 19/02/16 14:36, Jan Beulich wrote: > On 19.02.16 at 15:16, wrote: >>> On 19/02/16 14:08, Jan Beulich wrote: >>> On 18.02.16 at 19:52, wrote: > @@ -261,28 +261,8 @@ void

[Xen-devel] [PATCH] travis: add IRC notifications

2016-02-19 Thread Doug Goldstein
This will cause failed builds and when the build flips back to success to be reported to #xentest on FreeNode. The syntax of the message will be: xen-project/xen#BUILDID (BRANCH - REVISION : COMMITTER) Change view : https://github.com/xen-project/xen/compare/RANGE Build details :

Re: [Xen-devel] [PATCH 5/9] apm32: remove paravirt_enabled() use

2016-02-19 Thread Boris Ostrovsky
On 02/19/2016 08:08 AM, Luis R. Rodriguez wrote: There is already a check for apm_info.bios == 0, the apm_info.bios is set from the boot_params.apm_bios_info. Both Xen and lguest, which are also the only ones that set paravirt_enabled to true) do never set the apm_bios_info, the

Re: [Xen-devel] [PATCH] xen/x86: Unilaterally remove .init mappings

2016-02-19 Thread Jan Beulich
>>> On 18.02.16 at 19:03, wrote: > Because of the new 2M alignment of .init and .bss, the existing memory > guarding infrastructure causes a shattered 2M superpage with non-present > entries for .init, and present entries for the alignment space. > > Do away with the

Re: [Xen-devel] [PATCH] xen/x86: Use 2M superpages for text/data/bss mappings

2016-02-19 Thread Jan Beulich
>>> On 18.02.16 at 19:03, wrote: > --- a/xen/arch/x86/setup.c > +++ b/xen/arch/x86/setup.c > @@ -921,13 +921,51 @@ void __init noreturn __start_xen(unsigned long mbi_p) > /* The only data mappings to be relocated are in the Xen area. */ > pl2e

Re: [Xen-devel] [RFC v2 7/7] kprobes: port to linker table

2016-02-19 Thread Luis R. Rodriguez
On Fri, Feb 19, 2016 at 02:15:11PM +, Russell King - ARM Linux wrote: > On Fri, Feb 19, 2016 at 05:45:59AM -0800, Luis R. Rodriguez wrote: > > kprobe makes use of two custom sections: > > > > type name beginend > > init.data _kprobe_blacklist

Re: [Xen-devel] [PATCH v8 05/13] tools/libxc: support to resume uncooperative HVM guests

2016-02-19 Thread Ian Campbell
On Fri, 2016-02-19 at 14:43 +, Wei Liu wrote: > On Fri, Feb 19, 2016 at 09:15:38AM -0500, Konrad Rzeszutek Wilk wrote: > > On Thu, Feb 18, 2016 at 12:13:36PM +, Wei Liu wrote: > > > On Thu, Feb 18, 2016 at 10:43:15AM +0800, Wen Congyang wrote: > > > > Before this patch: > > > > 1. suspend

Re: [Xen-devel] [PATCH] xen/memguard: Drop memguard_init() entirely

2016-02-19 Thread Jan Beulich
>>> On 18.02.16 at 19:03, wrote: > It is not obvious what this code is doing. Most of it dates from 2007/2008, > and there have been substantial changes in Xen's memory handling since then. Deleting code which isn't understood what it is or was once used for is

Re: [Xen-devel] [PATCHv1 3/5] x86/fpu: Add a per-domain field to set the width of FIP/FDP

2016-02-19 Thread David Vrabel
On 19/02/16 14:36, Jan Beulich wrote: On 19.02.16 at 15:16, wrote: >> On 19/02/16 14:08, Jan Beulich wrote: >> On 18.02.16 at 19:52, wrote: @@ -261,28 +261,8 @@ void xsave(struct vcpu *v, uint64_t mask)

Re: [Xen-devel] [PATCH 8/9] x86/rtc: replace paravirt_enabled() check with subarch check

2016-02-19 Thread Luis R. Rodriguez
On Fri, Feb 19, 2016 at 02:22:12PM +0100, Juergen Gross wrote: > On 19/02/16 14:08, Luis R. Rodriguez wrote: > > The current check is a super long winded way of asking if this > > is on lguest. The flags is used for legacy features, this is > > What about Xen pv-domU? I wouldn't expect those to

Re: [Xen-devel] [PATCH 0/9] x86/init: replace paravirt_enabled() were possible

2016-02-19 Thread Luis R. Rodriguez
On Fri, Feb 19, 2016 at 01:34:59PM +, David Vrabel wrote: > On 19/02/16 13:08, Luis R. Rodriguez wrote: > > I originally set out to rename paravirt_enabled() to paravirt_legacy() > > but we instead decided to remove paravirt_enabled() completely. Although > > I have some linker table work

Re: [Xen-devel] [PATCH v8 05/13] tools/libxc: support to resume uncooperative HVM guests

2016-02-19 Thread Wei Liu
On Fri, Feb 19, 2016 at 09:15:38AM -0500, Konrad Rzeszutek Wilk wrote: > On Thu, Feb 18, 2016 at 12:13:36PM +, Wei Liu wrote: > > On Thu, Feb 18, 2016 at 10:43:15AM +0800, Wen Congyang wrote: > > > Before this patch: > > > 1. suspend > > > a. PVHVM and PV: we use the same way to suspend the

Re: [Xen-devel] [PATCH 1/9] x86/boot: enumerate documentation for the x86 hardware_subarch

2016-02-19 Thread Luis R. Rodriguez
On Fri, Feb 19, 2016 at 01:40:33PM +, David Vrabel wrote: > On 19/02/16 13:08, Luis R. Rodriguez wrote: > > Although hardware_subarch has been in place since the x86 boot > > protocol 2.07 it hasn't been used much. Enumerate current possible > > values to avoid misuses and help with semantics

Re: [Xen-devel] [PATCHv1 3/5] x86/fpu: Add a per-domain field to set the width of FIP/FDP

2016-02-19 Thread Jan Beulich
>>> On 19.02.16 at 15:16, wrote: > On 19/02/16 14:08, Jan Beulich wrote: > On 18.02.16 at 19:52, wrote: >>> @@ -261,28 +261,8 @@ void xsave(struct vcpu *v, uint64_t mask) >>> "=m" (*ptr), \ >>>

Re: [Xen-devel] [PATCH] version: Document guest_handle

2016-02-19 Thread Razvan Cojocaru
On 02/19/2016 04:32 PM, Konrad Rzeszutek Wilk wrote: > And what it is usually used for. > > Signed-off-by: Konrad Rzeszutek Wilk > --- > xen/include/public/version.h | 4 > 1 file changed, 4 insertions(+) > > diff --git a/xen/include/public/version.h

[Xen-devel] [PATCH] version: Document guest_handle

2016-02-19 Thread Konrad Rzeszutek Wilk
And what it is usually used for. Signed-off-by: Konrad Rzeszutek Wilk --- xen/include/public/version.h | 4 1 file changed, 4 insertions(+) diff --git a/xen/include/public/version.h b/xen/include/public/version.h index adca602..b97fc68 100644 ---

Re: [Xen-devel] [PATCH v2 1/4] tools/configure: only require bcc/ld86/as86 when needed

2016-02-19 Thread Doug Goldstein
On 2/19/16 8:00 AM, Konrad Rzeszutek Wilk wrote: > On Thu, Feb 18, 2016 at 08:57:02PM -0600, Doug Goldstein wrote: >> bcc/ld86/as86 are only necessary when we build rombios and not always so >> failing the build when they aren't available should not happen if the >> user isn't building rombios. >

Re: [Xen-devel] [PATCH] arm/monitor vm-events: Implement guest-request support

2016-02-19 Thread Jan Beulich
>>> On 18.02.16 at 20:35, wrote: > --- > MAINTAINERS | 1 + > xen/arch/arm/hvm.c | 8 +++ > xen/arch/x86/hvm/event.c| 116 > ++-- > xen/arch/x86/hvm/hvm.c | 1 + >

Re: [Xen-devel] [RFC v2 7/7] kprobes: port to linker table

2016-02-19 Thread Russell King - ARM Linux
On Fri, Feb 19, 2016 at 05:45:59AM -0800, Luis R. Rodriguez wrote: > kprobe makes use of two custom sections: > > type name beginend > init.data _kprobe_blacklist __start_kprobe_blacklist __stop_kprobe_blacklist > text .kprobes.text

[Xen-devel] [RFC v2 3/6] x86/init: move ebda reservations into linker table

2016-02-19 Thread Luis R. Rodriguez
This lets us annotate its requirements specifically for PC and lguest subarchitectures. While at it since head.c just has ebda data rename it. Since we're using linker tables and both x86 32-bit and 64-bit have them we don't need to declare a call for this separately. This lets us also keep this

[Xen-devel] [RFC v2 0/6] x86/init: use linker table

2016-02-19 Thread Luis R. Rodriguez
/log/?h=20160219-linker-table-v2 [4] https://git.kernel.org/cgit/linux/kernel/git/mcgrof/linux.git/log/?h=20160219-linker-table-v2 Luis R. Rodriguez (6): x86/boot: add BIT() to boot/bitops.h x86/init: use linker tables to simplify x86 init and annotate dependencies x86/init: move ebda

[Xen-devel] [RFC v2 5/6] x86/init: user linker table for ce4100 early setup

2016-02-19 Thread Luis R. Rodriguez
Using the linker table removes the need for the #ifdef'ery and clutter on head32.c. Signed-off-by: Luis R. Rodriguez --- arch/x86/include/asm/setup.h | 6 -- arch/x86/kernel/head32.c | 3 --- arch/x86/platform/ce4100/ce4100.c | 4 +++- 3 files changed, 3

[Xen-devel] [RFC v2 1/6] x86/boot: add BIT() to boot/bitops.h

2016-02-19 Thread Luis R. Rodriguez
The boot/bitops.h has guards against including the regular bitops (include/asm-generic/bitops.h), it only implements what we need at early boot. We'll be making use of BIT() later so add it. Users of boot/boot.h must include it prior to asm/setup.h otherwise the guard protection devised against

[Xen-devel] [RFC v2 2/6] x86/init: use linker tables to simplify x86 init and annotate dependencies

2016-02-19 Thread Luis R. Rodriguez
Any failure on the x86 init path can be catastrophic. A simple shift of a call from one place to another can easily break things. Likewise adding a new call to one path without considering all x86 requirements can make certain x86 run time environments crash. We currently account for these

Re: [Xen-devel] [PATCHv1 3/5] x86/fpu: Add a per-domain field to set the width of FIP/FDP

2016-02-19 Thread David Vrabel
On 19/02/16 14:08, Jan Beulich wrote: On 18.02.16 at 19:52, wrote: >> The x86 architecture allows either: a) the 64-bit FIP/FDP registers to be >> restored (clearing FCS and FDS); or b) the 32-bit FIP/FDP and FCS/FDS >> registers to be restored (clearing the upper

Re: [Xen-devel] [PATCH v8 05/13] tools/libxc: support to resume uncooperative HVM guests

2016-02-19 Thread Konrad Rzeszutek Wilk
On Thu, Feb 18, 2016 at 12:13:36PM +, Wei Liu wrote: > On Thu, Feb 18, 2016 at 10:43:15AM +0800, Wen Congyang wrote: > > Before this patch: > > 1. suspend > > a. PVHVM and PV: we use the same way to suspend the guest (send the suspend > >request to the guest). If the guest doesn't support

[Xen-devel] [RFC v2 4/6] x86/init: use linker table for i386 early setup

2016-02-19 Thread Luis R. Rodriguez
This also annotates this is only used for PC and lguest hardware subarchitectures. v2: add X86_SUBARCH_XEN as well, as noted by Konrad, now tested by 0-day bot. Signed-off-by: Luis R. Rodriguez --- arch/x86/kernel/head32.c | 9 + 1 file changed, 5 insertions(+),

[Xen-devel] [RFC v2 6/6] x86/init: use linker table for mid early setup

2016-02-19 Thread Luis R. Rodriguez
Using the linker table removes the need for the #ifdef'ery and clutter on head32.c and head64.c, if this is linked in and the subarch matches it will run. Cc: Andy Shevchenko Signed-off-by: Luis R. Rodriguez --- arch/x86/include/asm/setup.h

Re: [Xen-devel] [PATCHv1 5/5] x86/domctl: Add XEN_DOMCTL_PARAM_ARCH_X86_FIP_WIDTH parameter

2016-02-19 Thread Jan Beulich
>>> On 19.02.16 at 11:05, wrote: > On 18/02/16 18:52, David Vrabel wrote: >> Add a parameter to allow the toolstack to set the x87 FIP width in case >> the hypervisor's heuristics do the wrong thing. > > I think this would be better as a HVM param since: a) it only needs

Re: [Xen-devel] [PATCHv1 4/5] x86/viridian: set x87 FIP width to 4 for Windows guests

2016-02-19 Thread Jan Beulich
>>> On 18.02.16 at 19:52, wrote: > --- a/xen/arch/x86/hvm/viridian.c > +++ b/xen/arch/x86/hvm/viridian.c > @@ -148,6 +148,30 @@ static void dump_guest_os_id(const struct domain *d) > goi->fields.service_pack, goi->fields.build_number); > } > > +static void

Re: [Xen-devel] [PATCHv1 3/5] x86/fpu: Add a per-domain field to set the width of FIP/FDP

2016-02-19 Thread Jan Beulich
>>> On 18.02.16 at 19:52, wrote: > The x86 architecture allows either: a) the 64-bit FIP/FDP registers to be > restored (clearing FCS and FDS); or b) the 32-bit FIP/FDP and FCS/FDS > registers to be restored (clearing the upper 32-bits). > > Add a per-domain field to

Re: [Xen-devel] [PATCH v2 2/4] m4/python: fix typo in LDFLAGS variable name

2016-02-19 Thread Konrad Rzeszutek Wilk
On Thu, Feb 18, 2016 at 08:57:03PM -0600, Doug Goldstein wrote: > Reported-by: Jonathan Creekmore > Signed-off-by: Doug Goldstein Reviewed-by: Konrad Rzeszutek Wilk > --- > CC: Ian Jackson >

Re: [Xen-devel] [PATCH v2 1/4] tools/configure: only require bcc/ld86/as86 when needed

2016-02-19 Thread Konrad Rzeszutek Wilk
On Thu, Feb 18, 2016 at 08:57:02PM -0600, Doug Goldstein wrote: > bcc/ld86/as86 are only necessary when we build rombios and not always so > failing the build when they aren't available should not happen if the > user isn't building rombios. That is quite the long sentence. Perhaps trim it down

Re: [Xen-devel] [PATCHv1 1/5] domctl: Add op to get/set generic numeric parameters

2016-02-19 Thread Jan Beulich
>>> On 18.02.16 at 19:52, wrote: > --- > xen/arch/x86/domctl.c | 14 ++ What about ARM? > --- a/xen/include/public/domctl.h > +++ b/xen/include/public/domctl.h > @@ -1092,6 +1092,25 @@ struct xen_domctl_psr_cat_op { > typedef struct

Re: [Xen-devel] [PATCH] arm/monitor vm-events: Implement guest-request support

2016-02-19 Thread Razvan Cojocaru
On 02/19/2016 03:49 PM, Stefano Stabellini wrote: > On Thu, 18 Feb 2016, Corneliu ZUZU wrote: >> diff --git a/MAINTAINERS b/MAINTAINERS >> index cd4da04..768ad32 100644 >> --- a/MAINTAINERS >> +++ b/MAINTAINERS >> @@ -356,6 +356,7 @@ M: Tamas K Lengyel >> S: Supported

Re: [Xen-devel] [PATCH] arm/monitor vm-events: Implement guest-request support

2016-02-19 Thread Stefano Stabellini
On Thu, 18 Feb 2016, Corneliu ZUZU wrote: > diff --git a/MAINTAINERS b/MAINTAINERS > index cd4da04..768ad32 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -356,6 +356,7 @@ M:Tamas K Lengyel > S: Supported > F: xen/common/vm_event.c > F:

[Xen-devel] [RFC v2 5/7] jump_label: port __jump_table to linker tables

2016-02-19 Thread Luis R. Rodriguez
Move the __jump_table from the a custom section solution to a generic solution, this avoiding extra vmlinux.lds.h customizations. This also demos the use of the .data (SECTION_DATA) linker tables and of push_section_tbl(). Signed-off-by: Luis R. Rodriguez ---

[Xen-devel] [RFC v2 7/7] kprobes: port to linker table

2016-02-19 Thread Luis R. Rodriguez
kprobe makes use of two custom sections: type name beginend init.data _kprobe_blacklist __start_kprobe_blacklist__stop_kprobe_blacklist text .kprobes.text __kprobes_text_start__kprobes_text_end Port these to the linker table generic

[Xen-devel] [RFC v2 3/7] firmware: port built-in section to linker table

2016-02-19 Thread Luis R. Rodriguez
This ports built-in firmware to use linker tables, this replaces the custom section solution with a generic solution. This also demos the use of the .rodata (SECTION_RO) linker tables. Tested with 0 built-in firmware, 1 and 2 built-in firmwares successfully. Signed-off-by: Luis R. Rodriguez

[Xen-devel] [RFC v2 4/7] asm/sections: add a generic push_section_tbl()

2016-02-19 Thread Luis R. Rodriguez
With a generic linker tables solution in place we need a general asm solution for declaring entries with asm. The first easy target is to cover the C asm declarations, guard the header file for now and define a first generic entry push_section_tbl() to be used later for custom linker table

[Xen-devel] [RFC v2 6/7] dynamic_debug: port to use linker tables

2016-02-19 Thread Luis R. Rodriguez
This removes the custom vmlinux.lds.h hacks and uses the generalized solution for .data (SECTION_DATA) entries. This is much more potential for further fine tuning here though in the future. For instance, linker tables enable an extra postfix for order level annotations, this could easily be used

[Xen-devel] [RFC v2 1/7] sections.h: add sections header to collect all section info

2016-02-19 Thread Luis R. Rodriguez
Linux makes extensive use of custom ELF header sections, documentation for these are well scatterred. Unify this documentation in a central place. Signed-off-by: Luis R. Rodriguez --- Documentation/DocBook/Makefile | 3 +- Documentation/DocBook/sections.tmpl | 99

[Xen-devel] [RFC v2 2/7] tables.h: add linker table support

2016-02-19 Thread Luis R. Rodriguez
A linker table is a data structure that is stitched together from items in multiple object files. Linux has historically implicitly used linker tables for ages, however they were all built in an adhoc manner which requires linker script modifications, per architecture. This adds a general linker

[Xen-devel] [RFC v2 0/7] linux: add linker tables

2016-02-19 Thread Luis R. Rodriguez
This is my v2 of the original linker table work [0], now with six proof of concepts ports of existing code using custom section with custom linker script modifications: * DEFINE_LINKTABLE_TEXT(char, kprobes); * DEFINE_LINKTABLE_DATA(struct jump_entry, __jump_table); *

Re: [Xen-devel] [PATCH 1/9] x86/boot: enumerate documentation for the x86 hardware_subarch

2016-02-19 Thread David Vrabel
On 19/02/16 13:08, Luis R. Rodriguez wrote: > Although hardware_subarch has been in place since the x86 boot > protocol 2.07 it hasn't been used much. Enumerate current possible > values to avoid misuses and help with semantics later at boot > time should this be used further. > > Cc: Andy

Re: [Xen-devel] [PATCH 0/9] x86/init: replace paravirt_enabled() were possible

2016-02-19 Thread David Vrabel
On 19/02/16 13:08, Luis R. Rodriguez wrote: > I originally set out to rename paravirt_enabled() to paravirt_legacy() > but we instead decided to remove paravirt_enabled() completely. Although > I have some linker table work which will help make this cleaner, instead > of waiting for that to go in,

Re: [Xen-devel] [PATCH 8/9] x86/rtc: replace paravirt_enabled() check with subarch check

2016-02-19 Thread David Vrabel
On 19/02/16 13:08, Luis R. Rodriguez wrote: > The current check is a super long winded way of asking if this > is on lguest. The flags is used for legacy features, this is > likely inspired by the ACPI IA-PC boot architecture flags, where > as for RTC it annotates No CMOS real-time clock present.

Re: [Xen-devel] [PATCH 8/9] x86/rtc: replace paravirt_enabled() check with subarch check

2016-02-19 Thread Juergen Gross
On 19/02/16 14:08, Luis R. Rodriguez wrote: > The current check is a super long winded way of asking if this > is on lguest. The flags is used for legacy features, this is What about Xen pv-domU? I wouldn't expect those to have PV_SUPPORTED_RTC set. Juergen > likely inspired by the ACPI IA-PC

Re: [Xen-devel] [PATCH 1/9] x86/boot: enumerate documentation for the x86 hardware_subarch

2016-02-19 Thread Juergen Gross
On 19/02/16 14:08, Luis R. Rodriguez wrote: > Although hardware_subarch has been in place since the x86 boot > protocol 2.07 it hasn't been used much. Enumerate current possible > values to avoid misuses and help with semantics later at boot > time should this be used further. > > Cc: Andy

[Xen-devel] [PATCH 7/9] x86/cpu/intel: replace paravirt_enabled() for f00f work around

2016-02-19 Thread Luis R. Rodriguez
Use the harware subarch instead now that they are set. If we want to test removing this work around on Xen we can do so separatley as another independent change, for now we just want to remove paravirt_enabled(). Signed-off-by: Luis R. Rodriguez ---

[Xen-devel] [PATCH 2/9] tools/lguest: make lguest launcher use X86_SUBARCH_LGUEST explicitly

2016-02-19 Thread Luis R. Rodriguez
Be explicit and make use of X86_SUBARCH_LGUEST directly. Signed-off-by: Luis R. Rodriguez --- tools/lguest/lguest.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/lguest/lguest.c b/tools/lguest/lguest.c index 80159e6811c2..ff0aa580c6e1 100644

[Xen-devel] [PATCH 6/9] x86/tboot: remove paravirt_enabled()

2016-02-19 Thread Luis R. Rodriguez
There is already a check for boot_params.tboot_addr prior to paravirt_enabled() and both Xen and lguest never set this. This check is not needed. Signed-off-by: Luis R. Rodriguez --- arch/x86/kernel/tboot.c | 6 -- 1 file changed, 6 deletions(-) diff --git

[Xen-devel] [PATCH 1/9] x86/boot: enumerate documentation for the x86 hardware_subarch

2016-02-19 Thread Luis R. Rodriguez
Although hardware_subarch has been in place since the x86 boot protocol 2.07 it hasn't been used much. Enumerate current possible values to avoid misuses and help with semantics later at boot time should this be used further. Cc: Andy Shevchenko Signed-off-by:

[Xen-devel] [PATCH 4/9] x86/init: make ebda depend on PC subarch

2016-02-19 Thread Luis R. Rodriguez
This lets us remove its use of paravirt_enabled(). The other subarchs are not needed here given that on 32-bit there is a switch already that negates access to this code on X86_SUBARCH_INTEL_MID, and X86_SUBARCH_CE4100. Both lguest and Xen had paravirt_enabled so that excludes them.

[Xen-devel] [PATCH 9/9] pnpbios: replace paravirt_enabled() check with subarch checks

2016-02-19 Thread Luis R. Rodriguez
Since we are removing paravirt_enabled() replace it with a logical equivalent. Signed-off-by: Luis R. Rodriguez --- drivers/pnp/pnpbios/core.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/pnp/pnpbios/core.c b/drivers/pnp/pnpbios/core.c index

[Xen-devel] [PATCH 3/9] x86/xen: use X86_SUBARCH_XEN for PV guest boots

2016-02-19 Thread Luis R. Rodriguez
The use of subarch should have no current effect on Xen PV guests, as such this should have no current functional effects. Signed-off-by: Luis R. Rodriguez --- arch/x86/xen/enlighten.c | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/x86/xen/enlighten.c

[Xen-devel] [PATCH 5/9] apm32: remove paravirt_enabled() use

2016-02-19 Thread Luis R. Rodriguez
There is already a check for apm_info.bios == 0, the apm_info.bios is set from the boot_params.apm_bios_info. Both Xen and lguest, which are also the only ones that set paravirt_enabled to true) do never set the apm_bios_info, the paravirt_enabled() check is simply not needed. Signed-off-by: Luis

[Xen-devel] [PATCH 8/9] x86/rtc: replace paravirt_enabled() check with subarch check

2016-02-19 Thread Luis R. Rodriguez
The current check is a super long winded way of asking if this is on lguest. The flags is used for legacy features, this is likely inspired by the ACPI IA-PC boot architecture flags, where as for RTC it annotates No CMOS real-time clock present. I don't expect we will be implementing more legacy

[Xen-devel] [PATCH 0/9] x86/init: replace paravirt_enabled() were possible

2016-02-19 Thread Luis R. Rodriguez
I originally set out to rename paravirt_enabled() to paravirt_legacy() but we instead decided to remove paravirt_enabled() completely. Although I have some linker table work which will help make this cleaner, instead of waiting for that to go in, just remove the known cases that should be safe for

Re: [Xen-devel] [PATCH V13 5/5] xl: add pvusb commands

2016-02-19 Thread Olaf Hering
On Fri, Feb 19, Olaf Hering wrote: > Not sure how to handle it, perhaps exit when xl -N is called? Also the interface is 'xl -N' is not clearly defined. What is it supposed to do with the newly introduced ctrl types? Should it display the json just for the dev, just for the ctrl+dev or the

[Xen-devel] Xen Project Infrastructure Maintenance: Feb 22nd from 8:00 AM UTC (to fix CVE-2015-7547)

2016-02-19 Thread Lars Kurth
Hi everyone, due to the recently discovered glibc security vulnerability we need to reboot all our VMs: this will affect the blog, bugs, etherpad, lists, mail, wiki and xenbits. Each service should only be unavailable for 5 minutes. If you notice any issues after the reboot please reply to

Re: [Xen-devel] [PATCH] MAINTAINERS: Drop myself as xen netback maintainer

2016-02-19 Thread Wei Liu
On Fri, Feb 19, 2016 at 11:44:51AM +, Ian Campbell wrote: > Wei has been picking this up for quite a while now. > > Signed-off-by: Ian Campbell > Cc: Wei Liu Acked-by: Wei Liu > --- > MAINTAINERS | 1 - > 1 file changed,

Re: [Xen-devel] dom0 USB failing with "ehci-pci: probe of 0000:00:1d.0 failed with error -110"

2016-02-19 Thread Jan Beulich
>>> On 19.02.16 at 09:30, wrote: > On Thu, Feb 18, 2016 at 5:40 PM, Jan Beulich wrote: >> (copy to xen-devel) But then please don't cross-post (I'm therefore moving xen-users to BC). > The issue still exists with microcode updated. > And on this

[Xen-devel] [PATCH] MAINTAINERS: Drop myself as xen netback maintainer

2016-02-19 Thread Ian Campbell
Wei has been picking this up for quite a while now. Signed-off-by: Ian Campbell Cc: Wei Liu --- MAINTAINERS | 1 - 1 file changed, 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index 28eb61b..ef30060 100644 --- a/MAINTAINERS +++

Re: [Xen-devel] [PATCH v3 5/5] mkelf32: Close those file descriptors in the error paths.

2016-02-19 Thread Ian Jackson
Konrad Rzeszutek Wilk writes ("Re: [PATCH v3 5/5] mkelf32: Close those file descriptors in the error paths."): > > > If I turned them in: > > if (..blah..) > { > close(infd); > return -1; > } > > would that satisfy you? I would strongly resist

Re: [Xen-devel] [PATCH v3 5/5] mkelf32: Close those file descriptors in the error paths.

2016-02-19 Thread Jan Beulich
>>> On 18.02.16 at 22:12, wrote: > On Thu, Feb 18, 2016 at 09:45:30AM -0700, Jan Beulich wrote: >> >>> On 18.02.16 at 17:39, wrote: >> > Jan Beulich writes ("Re: [PATCH v3 5/5] mkelf32: Close those file > descriptors >> > in the error

Re: [Xen-devel] [PATCH] x86, microcode: Remove unnecessary paravirt_enabled check

2016-02-19 Thread Borislav Petkov
On Fri, Feb 19, 2016 at 01:21:10PM +0200, Andy Shevchenko wrote: > Heh, rebuilt on clean repository -> everything works. Looks like false > alarm and practical proof to do clean build first. Always, like *really* *always*, do $ make mrproper or even $ git clean -dqfx before testing kernels.

Re: [Xen-devel] [PATCH] x86, microcode: Remove unnecessary paravirt_enabled check

2016-02-19 Thread Andy Shevchenko
On Fri, Feb 19, 2016 at 1:10 PM, Borislav Petkov wrote: > On Fri, Feb 19, 2016 at 01:00:35PM +0200, Andy Shevchenko wrote: >> That what I see last with earlycon enabled + `debug` in cmdline: > > That doesn't help. Heh, rebuilt on clean repository -> everything works. Looks like

Re: [Xen-devel] [PATCH] hvmloader: Use xen/errno.h rather than the host systems errno.h

2016-02-19 Thread Ian Campbell
On Fri, 2016-02-19 at 11:09 +, Andrew Cooper wrote: > On 19/02/16 11:00, Ian Campbell wrote: > > On Fri, 2016-02-19 at 10:50 +, Andrew Cooper wrote: > > > On 19/02/16 10:40, Wei Liu wrote: > > > > On Thu, Feb 18, 2016 at 10:10:09PM +, Andrew Cooper wrote: > > > > > Signed-off-by:

Re: [Xen-devel] [PATCH] x86, microcode: Remove unnecessary paravirt_enabled check

2016-02-19 Thread Borislav Petkov
On Fri, Feb 19, 2016 at 01:00:35PM +0200, Andy Shevchenko wrote: > That what I see last with earlycon enabled + `debug` in cmdline: That doesn't help. Can you dump RIP, etc registers as to where the machine freezes? Let me confirm this: booting with "dis_ucode_ldr" works? Also, please send

Re: [Xen-devel] [PATCH] hvmloader: Use xen/errno.h rather than the host systems errno.h

2016-02-19 Thread Andrew Cooper
On 19/02/16 11:00, Ian Campbell wrote: > On Fri, 2016-02-19 at 10:50 +, Andrew Cooper wrote: >> On 19/02/16 10:40, Wei Liu wrote: >>> On Thu, Feb 18, 2016 at 10:10:09PM +, Andrew Cooper wrote: Signed-off-by: Andrew Cooper --- CC: Jan Beulich

Re: [Xen-devel] [PATCH] hvmloader: Use xen/errno.h rather than the host systems errno.h

2016-02-19 Thread Ian Campbell
On Fri, 2016-02-19 at 10:50 +, Andrew Cooper wrote: > On 19/02/16 10:40, Wei Liu wrote: > > On Thu, Feb 18, 2016 at 10:10:09PM +, Andrew Cooper wrote: > > > Signed-off-by: Andrew Cooper > > > --- > > > CC: Jan Beulich > > > CC: Ian Campbell

Re: [Xen-devel] [PATCH] x86, microcode: Remove unnecessary paravirt_enabled check

2016-02-19 Thread Andy Shevchenko
On Fri, Feb 19, 2016 at 12:50 PM, Andy Shevchenko wrote: > On Fri, Feb 19, 2016 at 12:46 PM, Borislav Petkov wrote: >> On Fri, Feb 19, 2016 at 12:33:46PM +0200, Andy Shevchenko wrote: >>> commit 84aba677f009e20185aea322563389ad56e0ef7e >>> Author:

[Xen-devel] Processed: Re: libxl and malloc failure (Re: Current LibXL Status)

2016-02-19 Thread xen
Processing commands for x...@bugs.xenproject.org: > graft 51 ^ Graft `<22214.2619.86697.724...@mariner.uk.xensource.com>' onto #51 > thanks Finished processing. Modified/created Bugs: - 51: http://bugs.xenproject.org/xen/bug/51 --- Xen Hypervisor Bug Tracker See

Re: [Xen-devel] [PATCH] x86/PVHv2: add XEN_ prefix to HVM_START_MAGIC_VALUE

2016-02-19 Thread Andrew Cooper
On 19/02/16 10:28, Roger Pau Monne wrote: > Signed-off-by: Roger Pau Monné > Reported by: Jan Beulich Reviewed-by: Andrew Cooper ___ Xen-devel mailing list Xen-devel@lists.xen.org

Re: [Xen-devel] Is there a way to get the current PV domain's UUID?

2016-02-19 Thread Ian Campbell
On Fri, 2016-02-19 at 12:44 +0200, Razvan Cojocaru wrote: > On 02/19/2016 12:20 PM, Ian Campbell wrote: > > On Fri, 2016-02-19 at 12:10 +0200, Razvan Cojocaru wrote: > > > Hello, > > > > > > Is there a way to get the UUID of a currently running PV domain, from > > > within that domain? > > > >

Re: [Xen-devel] [PATCH] hvmloader: Use xen/errno.h rather than the host systems errno.h

2016-02-19 Thread Ian Campbell
On Fri, 2016-02-19 at 10:40 +, Wei Liu wrote: > On Thu, Feb 18, 2016 at 10:10:09PM +, Andrew Cooper wrote: > > Signed-off-by: Andrew Cooper > > --- > > CC: Jan Beulich > > CC: Ian Campbell > > CC: Ian Jackson

Re: [Xen-devel] [PATCH V13 5/5] xl: add pvusb commands

2016-02-19 Thread Olaf Hering
On Fri, Feb 19, Chun Yan Liu wrote: > > > >>> On 2/17/2016 at 12:56 AM, in message <20160216165608.ga21...@gmail.com>, > >>> Olaf > Hering wrote: > > On Tue, Jan 19, Chunyan Liu wrote: > > > > > #xl usbctrl-attach test_vm version=1 ports=8 > > > > > #xl usbdev-attach

Re: [Xen-devel] [PATCH] hvmloader: Use xen/errno.h rather than the host systems errno.h

2016-02-19 Thread Wei Liu
On Fri, Feb 19, 2016 at 10:50:29AM +, Andrew Cooper wrote: > On 19/02/16 10:40, Wei Liu wrote: > > On Thu, Feb 18, 2016 at 10:10:09PM +, Andrew Cooper wrote: > >> Signed-off-by: Andrew Cooper > >> --- > >> CC: Jan Beulich > >> CC: Ian

Re: [Xen-devel] libxl and malloc failure (Re: Current LibXL Status)

2016-02-19 Thread Ian Campbell
graft 51 ^ thanks On Thu, 2016-02-18 at 18:15 +, Ian Jackson wrote: > Andrew Cooper writes ("Re: [Xen-devel] Current LibXL Status"): > > There really is a show-stopper, which I have stated before. > > > > Languages such as OCaml use -ENOMEM as a hint to run the garbage > > collector some

Re: [Xen-devel] [PATCH] x86, microcode: Remove unnecessary paravirt_enabled check

2016-02-19 Thread Andy Shevchenko
On Fri, Feb 19, 2016 at 12:46 PM, Borislav Petkov wrote: > On Fri, Feb 19, 2016 at 12:33:46PM +0200, Andy Shevchenko wrote: >> No, the commit 84aba677f009 as of today's linux-next on which I commented. > > You commented under the "> A subsequent commit, fbae4ba8c4a3" which

Re: [Xen-devel] [PATCH] hvmloader: Use xen/errno.h rather than the host systems errno.h

2016-02-19 Thread Andrew Cooper
On 19/02/16 10:40, Wei Liu wrote: > On Thu, Feb 18, 2016 at 10:10:09PM +, Andrew Cooper wrote: >> Signed-off-by: Andrew Cooper >> --- >> CC: Jan Beulich >> CC: Ian Campbell >> CC: Ian Jackson

Re: [Xen-devel] [PATCH] x86, microcode: Remove unnecessary paravirt_enabled check

2016-02-19 Thread Borislav Petkov
On Fri, Feb 19, 2016 at 12:33:46PM +0200, Andy Shevchenko wrote: > No, the commit 84aba677f009 as of today's linux-next on which I commented. You commented under the "> A subsequent commit, fbae4ba8c4a3" which confused me as to which commit is the culprit. > commit

Re: [Xen-devel] Is there a way to get the current PV domain's UUID?

2016-02-19 Thread Razvan Cojocaru
On 02/19/2016 12:20 PM, Ian Campbell wrote: > On Fri, 2016-02-19 at 12:10 +0200, Razvan Cojocaru wrote: >> Hello, >> >> Is there a way to get the UUID of a currently running PV domain, from >> within that domain? > > The type associated with the UUID in the xen public headers is >

Re: [Xen-devel] [PATCH V13 5/5] xl: add pvusb commands

2016-02-19 Thread Chun Yan Liu
>>> On 2/17/2016 at 12:56 AM, in message <20160216165608.ga21...@gmail.com>, >>> Olaf Hering wrote: > On Tue, Jan 19, Chunyan Liu wrote: > > > #xl usbctrl-attach test_vm version=1 ports=8 > > > #xl usbdev-attach test_vm hostbus=1 hostaddr=2 > > I think this does not

[Xen-devel] [PATCH V14 4/6] libxl: add pvusb API

2016-02-19 Thread Chunyan Liu
Add pvusb APIs, including: - attach/detach (create/destroy) virtual usb controller. - attach/detach usb device - list usb controller and usb devices - some other helper functions Signed-off-by: Simon Cao Signed-off-by: George Dunlap Signed-off-by:

[Xen-devel] [PATCH V14 6/6] xl: add pvusb commands

2016-02-19 Thread Chunyan Liu
Add pvusb commands: usbctrl-attach, usbctrl-detach, usb-list, usbdev-attach and usbdev-detach. To attach a usb device to guest through pvusb, one could follow following example: #xl usbctrl-attach test_vm version=1 ports=8 #xl usb-list test_vm will show the usb controllers and port usage

<    1   2   3   >