Re: [Xen-devel] [PATCH] xen: sched: use default scheduler upon an invalid "sched="

: Jan Beulich <jbeul...@suse.com> > Cc: Doug Goldstein <car...@cardoe.com> > Cc: Jonathan Creekmore <jonathan.creekm...@gmail.com> Reviewed-By: Jonathan Creekmore <jonathan.creekm...@gmail.com> ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] Problem Reading from XenStore in DomU

Dagaen Golomb writes: > It does, being the custom kernel on version 4.1.0. But Dom0 uses this same > exact kernel and reads/writes just fine! The only solution if this is > indeed the problem appears to be changing the kernel source we build on or > some hacky method such as symlinking /proc/..

Re: [Xen-devel] [PATCH v5 1/8] Kconfig: import kconfig.h from Linux 4.3

Shannon Zhao writes: > From: Shannon Zhao > > To support using CONFIG_ options in C/CPP expressions, import kconfig.h > from the Linux v4.3 tag (commit id > 6a13feb9c82803e2b815eca72fa7a9f5561d7861). > > CC: Doug Goldstein > Signed-off-by: Shannon

Re: [Xen-devel] [PATCH v5 2/8] ACPI: add config for BIOS table scan

Shannon Zhao writes: > From: Graeme Gregory > > With the addition of ARM64 that does not have a traditional BIOS to > scan, add a config option which is selected on x86 (ia64 doesn't need > it either, it is EFI/UEFI based system) to do the traditional BIOS > scanning

Re: [Xen-devel] [PATCH v6 3/5] build: Alloc space for sched list in the link file

> On Jan 15, 2016, at 11:01 AM, Jonathan Creekmore > <jonathan.creekm...@gmail.com> wrote: > > Creates a section to contain scheduler entry pointers that are gathered > together into an array. This will allow, in a follow-on patch, scheduler > entries to be automat

[Xen-devel] [PATCH v6 2/5] build: Hook the schedulers into Kconfig

dun...@eu.citrix.com> CC: Dario Faggioli <dario.faggi...@citrix.com> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> Reviewed-by: Doug Goldstein <car...@cardoe.com> --- Changed since v5: * Remove extra defaults for schedulers since credit is always

[Xen-devel] [PATCH v6 3/5] build: Alloc space for sched list in the link file

ellini <stefano.stabell...@citrix.com> CC: Keir Fraser <k...@xen.org> CC: Jan Beulich <jbeul...@suse.com> CC: Andrew Cooper <andrew.coop...@citrix.com> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> Reviewed-by: Andrew Cooper <andrew.coop...@citri

[Xen-devel] [PATCH v6 5/5] sched: Use the auto-generated list of schedulers

m> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> Acked-by: Dario Faggioli <dario.faggi...@citrix.com> Reviewed-by: Andrew Cooper <andrew.coop...@citrix.com> Reviewed-by: Doug Goldstein <car...@cardoe.com> --- Changed since v6: * Make schedulers

[Xen-devel] [PATCH v6 4/5] sched: Register the schedulers into the list

<josh.whiteh...@dornerworks.com> CC: Robert VanVossen <robert.vanvos...@dornerworks.com> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> Acked-by: Dario Faggioli <dario.faggi...@citrix.com> Reviewed-by: Andrew Cooper <andrew.coop...@citrix.com> Revi

[Xen-devel] [PATCH v6 0/5] Allow schedulers to be selectable through Kconfig

the rest of the file * Simplify the calculation of the number of schedulers * Make the scheduler ops structures static to their files Jonathan Creekmore (5): build: Env var to enable expert config options build: Hook the schedulers into Kconfig build: Alloc space for sched list

[Xen-devel] [PATCH v6 1/5] build: Env var to enable expert config options

k...@xen.org> CC: Tim Deegan <t...@xen.org> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> Reviewed-by: Doug Goldstein <car...@cardoe.com> Reviewed-by: Konrad Rzeszutek Wilk <konrad.w...@oracle.com> --- xen/Kconfig | 4 xen/Makefile | 1 + 2 files chan

Re: [Xen-devel] [PATCH v6 1/5] build: Env var to enable expert config options

> On Jan 15, 2016, at 11:20 AM, Jan Beulich wrote: > On 15.01.16 at 18:01, wrote: >> --- a/xen/Makefile >> +++ b/xen/Makefile >> @@ -11,6 +11,7 @@ export XEN_DOMAIN ?= $(shell ([ -x /bin/dnsdomainname ] >> && /bin/dnsdomainname) | >>

Re: [Xen-devel] [PATCH v5 5/5] sched: Use the auto-generated list of schedulers

Jan Beulich writes: On 14.01.16 at 15:49, wrote: >> --- a/xen/common/schedule.c >> +++ b/xen/common/schedule.c >> @@ -64,20 +64,9 @@ static void poll_timer_fn(void *data); >> DEFINE_PER_CPU(struct schedule_data, schedule_data); >> DEFINE_PER_CPU(struct

Re: [Xen-devel] [PATCH v5 2/5] build: Hook the schedulers into Kconfig

Jan Beulich writes: >>>> On 14.01.16 at 17:34, <ian.campb...@citrix.com> wrote: >> On Thu, 2016-01-14 at 10:23 -0600, Jonathan Creekmore wrote: >>> Jan Beulich writes: >>> >>> > > > > On 14.01.16 at 15:49, <jonathan.creekm...@gm

Re: [Xen-devel] [PATCH v5 2/5] build: Hook the schedulers into Kconfig

Jan Beulich writes: On 14.01.16 at 15:49, wrote: >> --- a/xen/common/Kconfig >> +++ b/xen/common/Kconfig >> @@ -51,4 +51,63 @@ config KEXEC >> >>If unsure, say Y. >> >> +# Enable schedulers >> +menu "Schedulers" >> +visible if EXPERT = "y" >> + >>

[Xen-devel] [PATCH v5 3/5] build: Alloc space for sched list in the link file

ellini <stefano.stabell...@citrix.com> CC: Keir Fraser <k...@xen.org> CC: Jan Beulich <jbeul...@suse.com> CC: Andrew Cooper <andrew.coop...@citrix.com> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> Reviewed-by: Andrew Cooper <andrew.coop...@citri

[Xen-devel] [PATCH v5 5/5] sched: Use the auto-generated list of schedulers

m> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> Acked-by: Dario Faggioli <dario.faggi...@citrix.com> Reviewed-by: Andrew Cooper <andrew.coop...@citrix.com> Reviewed-by: Doug Goldstein <car...@cardoe.com> --- Changed since v4: * Remove extra s

[Xen-devel] [PATCH v5 0/5] Allow schedulers to be selectable through Kconfig

* Removed confusing language from the commit message * Alphabetize the schedulers * rename the __start and __end symbols to better match the rest of the file * Simplify the calculation of the number of schedulers * Make the scheduler ops structures static to their files Jonathan Creekmore (5

[Xen-devel] [PATCH v5 4/5] sched: Register the schedulers into the list

<josh.whiteh...@dornerworks.com> CC: Robert VanVossen <robert.vanvos...@dornerworks.com> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> Acked-by: Dario Faggioli <dario.faggi...@citrix.com> Reviewed-by: Andrew Cooper <andrew.coop...@citrix.com> Revi

[Xen-devel] [PATCH v5 2/5] build: Hook the schedulers into Kconfig

Dunlap <george.dun...@eu.citrix.com> CC: Dario Faggioli <dario.faggi...@citrix.com> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> Reviewed-by: Doug Goldstein <car...@cardoe.com> --- Changed since v4: * Removed the "if unsure" language * Remo

[Xen-devel] [PATCH v5 1/5] build: Env var to enable expert config options

k...@xen.org> CC: Tim Deegan <t...@xen.org> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> Reviewed-by: Doug Goldstein <car...@cardoe.com> Reviewed-by: Konrad Rzeszutek Wilk <konrad.w...@oracle.com> --- xen/Kconfig | 4 xen/Makefile | 1 + 2 files chan

Re: [Xen-devel] [PATCH v4 2/5] build: Hook the schedulers into Kconfig

Jan Beulich writes: On 08.01.16 at 22:22, wrote: >> --- a/xen/common/Kconfig >> +++ b/xen/common/Kconfig >> @@ -51,4 +51,71 @@ config KEXEC >> >>If unsure, say Y. >> >> +# Enable schedulers >> +menu "Schedulers" >> +visible if EXPERT = "y" > > Does

Re: [Xen-devel] [PATCH v4 2/5] build: Hook the schedulers into Kconfig

Andrew Cooper writes: > On 08/01/16 21:22, Jonathan Creekmore wrote: >> +# Enable schedulers >> +menu "Schedulers" >> +visible if EXPERT = "y" >> + >> +config SCHED_CREDIT >> +bool "Credit scheduler support" >> +

Re: [Xen-devel] [PATCH v4 5/5] sched: Use the auto-generated list of schedulers

Andrew Cooper writes: > On 08/01/16 21:22, Jonathan Creekmore wrote: >> diff --git a/xen/common/schedule.c b/xen/common/schedule.c >> index 2f98a48..91e53c1 100644 >> --- a/xen/common/schedule.c >> +++ b/xen/common/schedule.c >> @@ -64,20 +64,10 @@ sta

Re: [Xen-devel] [PATCH v4 3/5] build: Alloc space for sched list in the link file

Andrew Cooper writes: > On 08/01/16 21:22, Jonathan Creekmore wrote: >> Creates a section to contain scheduler entry pointers that are gathered >> together into an array. This will allow, in a follow-on patch, scheduler >> entries to be automatically gathered to

Re: [Xen-devel] [PATCH v4 2/5] build: Hook the schedulers into Kconfig

Andrew Cooper writes: > On 09/01/16 17:50, Jonathan Creekmore wrote: >> Andrew Cooper writes: >> >>> On 08/01/16 21:22, Jonathan Creekmore wrote: >>>> +# Enable schedulers >>>> +menu "Schedulers" >>>> + visible if EXPERT

[Xen-devel] [PATCH v4 1/5] build: Env var to enable expert config options

k...@xen.org> CC: Tim Deegan <t...@xen.org> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> Reviewed-by: Doug Goldstein <car...@cardoe.com> Reviewed-by: Konrad Rzeszutek Wilk <konrad.w...@oracle.com> --- xen/Kconfig | 4 xen/Makefile | 1 + 2 files chan

[Xen-devel] [PATCH v4 2/5] build: Hook the schedulers into Kconfig

dun...@eu.citrix.com> CC: Dario Faggioli <dario.faggi...@citrix.com> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> Reviewed-by: Doug Goldstein <car...@cardoe.com> --- Changed since v2: * Hid the scheduler menu behind the EXPERT option * Provide static inli

[Xen-devel] [PATCH v4 4/5] sched: Register the schedulers into the list

<josh.whiteh...@dornerworks.com> CC: Robert VanVossen <robert.vanvos...@dornerworks.com> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> Acked-by: Dario Faggioli <dario.faggi...@citrix.com> Reviewed-by: Andrew Cooper <andrew.coop...@citrix.com> Revi

[Xen-devel] [PATCH v4 0/5] Allow schedulers to be selectable through Kconfig

* Alphabetize the schedulers * rename the __start and __end symbols to better match the rest of the file * Simplify the calculation of the number of schedulers * Make the scheduler ops structures static to their files Jonathan Creekmore (5): build: Env var to enable expert config options

[Xen-devel] [PATCH v4 5/5] sched: Use the auto-generated list of schedulers

m> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> Acked-by: Dario Faggioli <dario.faggi...@citrix.com> Reviewed-by: Andrew Cooper <andrew.coop...@citrix.com> Reviewed-by: Doug Goldstein <car...@cardoe.com> --- Changed since v1: * Simplify the cal

[Xen-devel] [PATCH v4 3/5] build: Alloc space for sched list in the link file

ellini <stefano.stabell...@citrix.com> CC: Keir Fraser <k...@xen.org> CC: Jan Beulich <jbeul...@suse.com> CC: Andrew Cooper <andrew.coop...@citrix.com> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> Reviewed-by: Doug Goldstein <car...@cardoe.com>

Re: [Xen-devel] [PATCH v3 2/5] build: Hook the schedulers into Kconfig

Juergen Gross writes: > On 08/01/16 17:02, Jonathan Creekmore wrote: >> >> Andrew Cooper writes: >> >>> On 08/01/16 15:47, Konrad Rzeszutek Wilk wrote: >>>> On Thu, Jan 07, 2016 at 11:29:18AM -0600, Jonathan Creekmore wrote: >>>>> All

Re: [Xen-devel] [PATCH v3 2/5] build: Hook the schedulers into Kconfig

Doug Goldstein writes: > On 1/8/16 10:49 AM, Jan Beulich wrote: > On 08.01.16 at 17:30, wrote: >>> So, based on the Kconfig setup and the linker ASSERT, there should be no >>> way to have a default scheduler that is not in the build. I wish Kconfig >>> allowed

Re: [Xen-devel] [PATCH v3 2/5] build: Hook the schedulers into Kconfig

Andrew Cooper writes: > On 08/01/16 15:47, Konrad Rzeszutek Wilk wrote: >> On Thu, Jan 07, 2016 at 11:29:18AM -0600, Jonathan Creekmore wrote: >>> Allow the schedulers to be independently enabled or disabled at >>> compile-time. To match existing behavior, all four

[Xen-devel] [PATCH v3 0/5] Allow schedulers to be selectable through Kconfig

to better match the rest of the file * Simplify the calculation of the number of schedulers * Make the scheduler ops structures static to their files Jonathan Creekmore (5): build: Env var to enable expert config options build: Hook the schedulers into Kconfig build: Alloc space for sched

[Xen-devel] [PATCH v3 2/5] build: Hook the schedulers into Kconfig

dun...@eu.citrix.com> CC: Dario Faggioli <dario.faggi...@citrix.com> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> Reviewed-by: Doug Goldstein <car...@cardoe.com> --- Changed since v2: * Hid the scheduler menu behind the EXPERT option * Provide static inli

[Xen-devel] [PATCH v3 1/5] build: Env var to enable expert config options

k...@xen.org> CC: Tim Deegan <t...@xen.org> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> Reviewed-by: Doug Goldstein <car...@cardoe.com> --- xen/Kconfig | 4 xen/Makefile | 1 + 2 files changed, 5 insertions(+) diff --git a/xen/Kconfig b/xen/Kconfig index

[Xen-devel] [PATCH v3 4/5] sched: Register the schedulers into the list

<josh.whiteh...@dornerworks.com> CC: Robert VanVossen <robert.vanvos...@dornerworks.com> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> Acked-by: Dario Faggioli <dario.faggi...@citrix.com> Reviewed-by: Andrew Cooper <andrew.coop...@citrix.com> Revi

Re: [Xen-devel] [PATCH 4/4] sched: Use the auto-generated list of schedulers

Jan Beulich writes: On 17.12.15 at 21:59, wrote: >> +extern const struct scheduler *__schedulers_start[], *__schedulers_end[]; >> +#define NUM_SCHEDULERS >> (((uintptr_t)__schedulers_end-(uintptr_t)__schedulers_start) \ >> +/

Re: [Xen-devel] [PATCH 2/4] build: Alloc space for sched list in the link file

> On Dec 18, 2015, at 3:01 AM, Andrew Cooper <andrew.coop...@citrix.com> wrote: > > On 17/12/2015 20:59, Jonathan Creekmore wrote: >> Creates a section to contain scheduler entry pointers that are gathered >> together into an array. This will allow, in a follow-on

Re: [Xen-devel] [PATCH 2/4] build: Alloc space for sched list in the link file

> On Dec 18, 2015, at 11:07 AM, Jan Beulich <jbeul...@suse.com> wrote: > >>>> On 18.12.15 at 17:48, <andrew.coop...@citrix.com> wrote: >> On 18/12/15 16:40, Jonathan Creekmore wrote: >>>> On Dec 18, 2015, at 3:01 AM, Andrew Cooper <andrew.coop..

Re: [Xen-devel] [PATCH 4/4] sched: Use the auto-generated list of schedulers

> On Dec 18, 2015, at 10:43 AM, Jan Beulich wrote: > On 18.12.15 at 17:00, wrote: >> Jan Beulich writes: >> On 17.12.15 at 21:59, wrote: +extern const struct scheduler *__schedulers_start[],

Re: [Xen-devel] [PATCH 4/4] sched: Use the auto-generated list of schedulers

Andrew Cooper writes: > On 17/12/2015 20:59, Jonathan Creekmore wrote: >> Instead of having a manually-curated list of schedulers, use the array >> that was auto-generated simply by compiling in the scheduler files as >> the sole source of truth of the available schedul

Re: [Xen-devel] [PATCH 1/4] build: Hook the schedulers into Kconfig

Andrew Cooper writes: > On 17/12/2015 20:59, Jonathan Creekmore wrote: >> Allow the schedulers to be independently enabled or disabled at >> compile-time instead of just allowing the scheduler to be selected on >> the command line. To match existing behavior, all four sched

[Xen-devel] [PATCH v2 1/4] build: Hook the schedulers into Kconfig

dun...@eu.citrix.com> CC: Dario Faggioli <dario.faggi...@citrix.com> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> Acked-by: Dario Faggioli <dario.faggi...@citrix.com> Reviewed-by: Andrew Cooper <andrew.coop...@citrix.com> --- Changed since v1: *

[Xen-devel] [PATCH v2 0/4] Allow schedulers to be selectable through Kconfig

* rename the __start and __end symbols to better match the rest of the file * Simplify the calculation of the number of schedulers * Make the scheduler ops structures static to their files Jonathan Creekmore (4): build: Hook the schedulers into Kconfig build: Alloc space for sched list

[Xen-devel] [PATCH v2 4/4] sched: Use the auto-generated list of schedulers

m> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> Acked-by: Dario Faggioli <dario.faggi...@citrix.com> --- Changed since v1: * Simplify the calculation of the number of schedulers * Make the scheduler ops structures static to their files --- xen/common/sched

[Xen-devel] [PATCH v2 2/4] build: Alloc space for sched list in the link file

ellini <stefano.stabell...@citrix.com> CC: Keir Fraser <k...@xen.org> CC: Jan Beulich <jbeul...@suse.com> CC: Andrew Cooper <andrew.coop...@citrix.com> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> Reviewed-by: Andrew Cooper <andrew.coop...@citrix

[Xen-devel] [PATCH v2 3/4] sched: Register the schedulers into the list

<josh.whiteh...@dornerworks.com> CC: Robert VanVossen <robert.vanvos...@dornerworks.com> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> Acked-by: Dario Faggioli <dario.faggi...@citrix.com> Reviewed-by: Andrew Cooper <andrew.coop...@citrix.com> --

[Xen-devel] [PATCH 0/4] Allow schedulers to be selectable through Kconfig

the scheduler list from being manually curated into a model where just compiling any schedulers into the hypervisor causes the scheduler list to be built up. Jonathan Creekmore (4): build: Hook the schedulers into Kconfig build: Alloc space for sched list in the link file sched: Register

[Xen-devel] [PATCH 2/4] build: Alloc space for sched list in the link file

ellini <stefano.stabell...@citrix.com> CC: Keir Fraser <k...@xen.org> CC: Jan Beulich <jbeul...@suse.com> CC: Andrew Cooper <andrew.coop...@citrix.com> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> --- xen/arch/arm/xen.lds.S | 4 xen/arch/x86/xe

[Xen-devel] [PATCH 1/4] build: Hook the schedulers into Kconfig

their not currently supported status. CC: George Dunlap <george.dun...@eu.citrix.com> CC: Dario Faggioli <dario.faggi...@citrix.com> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> --- xen/common/Kconfig| 64 +++ x

[Xen-devel] [PATCH 3/4] sched: Register the schedulers into the list

<josh.whiteh...@dornerworks.com> CC: Robert VanVossen <robert.vanvos...@dornerworks.com> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> --- xen/common/sched_arinc653.c | 2 ++ xen/common/sched_credit.c | 2 ++ xen/common/sched_credit2.c | 2 ++ xen/commo

[Xen-devel] [PATCH 4/4] sched: Use the auto-generated list of schedulers

m> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> --- xen/common/schedule.c | 24 +++- xen/include/xen/sched-if.h | 5 - 2 files changed, 7 insertions(+), 22 deletions(-) diff --git a/xen/common/schedule.c b/xen/common/schedule.c index 2f98a

Re: [Xen-devel] [PATCH 1/4] build: Hook the schedulers into Kconfig

> On Dec 17, 2015, at 7:35 PM, Dario Faggioli <dario.faggi...@citrix.com> wrote: > > On Thu, 2015-12-17 at 14:59 -0600, Jonathan Creekmore wrote: >> Allow the schedulers to be independently enabled or disabled at >> compile-time instead of just allowing t

Re: [Xen-devel] [PATCH v2] build: fix clean to remove all .o and .d files

> On Dec 2, 2015, at 9:36 AM, Jan Beulich wrote: > On 02.12.15 at 16:29, wrote: >> --- a/xen/Makefile >> +++ b/xen/Makefile >> @@ -91,7 +91,8 @@ _clean: delete-unfresh-files >> $(MAKE) -f $(BASEDIR)/Rules.mk -C xsm clean >>

Re: [Xen-devel] [PATCH v2] build: fix clean to remove all .o and .d files

> On Dec 2, 2015, at 9:46 AM, Jan Beulich wrote: > On 02.12.15 at 16:41, wrote: > >>> On Dec 2, 2015, at 9:36 AM, Jan Beulich wrote: >>> >> On 02.12.15 at 16:29, wrote: ---

[Xen-devel] [PATCH v2] build: fix clean to remove all .o and .d files

behind. This method does not have the same disadvantage. CC: Ian Campbell <ian.campb...@citrix.com> CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Jan Beulich <jbeul...@suse.com> CC: Keir Fraser <k...@xen.org> CC: Tim Deegan <t...@xen.org> Signed-off-by: Jonat

Re: [Xen-devel] [PATCH] build: fix clean rule to cover objects in unvisited subdirs

> On Dec 1, 2015, at 6:41 AM, Jan Beulich wrote: > On 30.11.15 at 18:29, wrote: >> --- a/xen/Rules.mk >> +++ b/xen/Rules.mk >> @@ -173,7 +173,7 @@ FORCE: >> >> .PHONY: clean >> clean:: $(addprefix _clean_, $(subdir-all)) >> -rm -f *.o

Re: [Xen-devel] [PATCH] build: fix clean rule to cover objects in unvisited subdirs

> On Dec 1, 2015, at 10:07 AM, Jan Beulich wrote: > > For one build run, yes. But then you can (a) build individual object > files and (b) as mentioned above change configuration (implying > that you know what you're doing). Also you could, using the > example above, do a

Re: [Xen-devel] [PATCH] build: fix dependencies for files compiled from their parent directory

On Wed, Nov 25, 2015 at 10:16 AM, Jan Beulich wrote: > The use of $(basename ...) here was wrong (yet I'm sure I tested it). > > Signed-off-by: Jan Beulich > > --- a/xen/Rules.mk > +++ b/xen/Rules.mk > @@ -105,7 +105,7 @@ include Makefile > DEPS = .*.d >

[Xen-devel] [PATCH] build: fix clean rule to cover objects in unvisited subdirs

in the visited directory. CC: Ian Campbell <ian.campb...@citrix.com> CC: Ian Jackson <ian.jack...@eu.citrix.com> CC: Jan Beulich <jbeul...@suse.com> CC: Keir Fraser <k...@xen.org> CC: Tim Deegan <t...@xen.org> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail

Re: [Xen-devel] [PATCH] build: remove .d files from xen/ on a clean

>>>> On 24.11.15 at 18:22, <jonathan.creekm...@gmail.com> wrote: >>> >>>>> On Nov 24, 2015, at 11:16 AM, Jonathan Creekmore >>>> <jonathan.creekm...@gmail.com> wrote: >>>>> >>>>> So, the files in xen/ were the depende

Re: [Xen-devel] [PATCH] build: remove .d files from xen/ on a clean

> On Nov 24, 2015, at 11:16 AM, Jonathan Creekmore > <jonathan.creekm...@gmail.com> wrote: > >> >> On Nov 24, 2015, at 11:07 AM, Jan Beulich <jbeul...@suse.com> wrote: >> >>>>> On 24.11.15 at 17:56, <jonathan.creekm...@gmail.

Re: [Xen-devel] [PATCH] build: remove .d files from xen/ on a clean

> On Nov 24, 2015, at 11:30 AM, Jan Beulich <jbeul...@suse.com> wrote: > >>>> On 24.11.15 at 18:22, <jonathan.creekm...@gmail.com> wrote: > >>> On Nov 24, 2015, at 11:16 AM, Jonathan Creekmore >> <jonathan.creekm...@gmail.com> wrote:

[Xen-devel] [PATCH] build: remove .d files from xen/ on a clean

Jan Beulich <jbeul...@suse.com> CC: Keir Fraser <k...@xen.org> CC: Tim Deegan <t...@xen.org> Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> --- xen/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xen/Makefile b/xen/Makefile ind

Re: [Xen-devel] [PATCH] build: remove .d files from xen/ on a clean

> On Nov 24, 2015, at 11:07 AM, Jan Beulich wrote: > On 24.11.15 at 17:56, wrote: >> --- a/xen/Makefile >> +++ b/xen/Makefile >> @@ -88,7 +88,7 @@ _clean: delete-unfresh-files >> $(MAKE) -f $(BASEDIR)/Rules.mk -C xsm clean >>

Re: [Xen-devel] [PATCH] x86: wrap kexec feature with CONFIG_KEXEC

"Jan Beulich" writes: On 01.09.15 at 16:29, wrote: Which is both appreciated and understandable. I suppose you agree though that if you were to follow the model used for the kexec part, things would quickly become unwieldy. Hence I would

[Xen-devel] [PATCH v5] x86: wrap kexec feature with CONFIG_KEXEC

Add the appropriate #if checks around the kexec code in the x86 codebase so that the feature can actually be turned off by the flag instead of always required to be enabled on x86. Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> Reviewed-by: Andrew Cooper <an

Re: [Xen-devel] [PATCH] x86: wrap kexec feature with CONFIG_KEXEC

Andrew Cooper writes: On 01/09/15 11:54, Jan Beulich wrote: On 01.09.15 at 12:44, wrote: On 01/09/15 11:36, Ian Campbell wrote: In general (i.e. not 100% consistently, I think) we have tended to avoid making things user-facing compile

[Xen-devel] [PATCH v3] x86: wrap kexec feature with CONFIG_KEXEC

Add the appropriate #if checks around the kexec code in the x86 codebase so that the feature can actually be turned off by the flag instead of always required to be enabled on x86. Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> --- Changed since v2: * Switch macro

Re: [Xen-devel] [PATCH v2] libxenstore: prefer using the character device

refer using the character device"): On Thu, Aug 27, 2015 at 09:04:38AM -0500, Jonathan Creekmore wrote: With the addition of FMODE_ATOMIC_POS in the Linux 3.14 kernel, concurrent blocking file accesses to a single open file descriptor can cause a deadlock trying to grab the file position lock. If

[Xen-devel] [PATCH v4] x86: wrap kexec feature with CONFIG_KEXEC

Add the appropriate #if checks around the kexec code in the x86 codebase so that the feature can actually be turned off by the flag instead of always required to be enabled on x86. Signed-off-by: Jonathan Creekmore <jonathan.creekm...@gmail.com> --- Changed since v3: * Correct makefile t

[Xen-devel] [PATCH v2] x86: wrap kexec feature with CONFIG_KEXEC

Add the appropriate #if checks around the kexec code in the x86 codebase so that the feature can actually be turned off by the flag instead of always required to be enabled on x86. Signed-off-by: Jonathan Creekmore jonathan.creekm...@gmail.com --- Changed since v1: * Reorder kexec files

[Xen-devel] [PATCH v2] libxenstore: prefer using the character device

-by: Jonathan Creekmore jonathan.creekm...@gmail.com --- tools/xenstore/xs_lib.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tools/xenstore/xs_lib.c b/tools/xenstore/xs_lib.c index af4f75a..0c7744e 100644 --- a/tools/xenstore/xs_lib.c +++ b/tools/xenstore/xs_lib.c @@ -81,6 +81,8 @@ const char

Re: [Xen-devel] [PATCH] x86: wrap kexec feature with CONFIG_KEXEC

On Aug 27, 2015, at 10:27 AM, David Vrabel david.vra...@citrix.com wrote: On 27/08/15 15:47, Jonathan Creekmore wrote: Add the appropriate #if checks around the kexec code in the x86 codebase so that the feature can actually be turned off by the flag instead of always required

Re: [Xen-devel] [PATCH v2] libxenstore: prefer using the character device

Ian Jackson ian.jack...@eu.citrix.com writes: Wei Liu writes (Re: [Xen-devel] [PATCH v2] libxenstore: prefer using the character device): On Thu, Aug 27, 2015 at 09:04:38AM -0500, Jonathan Creekmore wrote: With the addition of FMODE_ATOMIC_POS in the Linux 3.14 kernel, concurrent blocking

Re: [Xen-devel] [PATCH] libxenstore: Use poll() with a non-blocking read()

David Vrabel david.vra...@citrix.com writes: On 16/08/15 09:59, Ian Campbell wrote: On Thu, 2015-08-13 at 16:44 -0500, Jonathan Creekmore wrote: With the addition of FMODE_ATOMIC_POS in the Linux 3.14 kernel, concurrent blocking file accesses to a single open file descriptor can cause

Re: [Xen-devel] [PATCH] libxenstore: Use poll() with a non-blocking read()

On Aug 16, 2015, at 1:59 AM, Ian Campbell ian.campb...@citrix.com wrote: On Thu, 2015-08-13 at 16:44 -0500, Jonathan Creekmore wrote: With the addition of FMODE_ATOMIC_POS in the Linux 3.14 kernel, concurrent blocking file accesses to a single open file descriptor can cause a deadlock

[Xen-devel] [PATCH] libxenstore: Use poll() with a non-blocking read()

. On the other hand, using the poll() API to implement the blocking for the read_all() function prevents the file descriptor from being switched back and forth between blocking and non-blocking modes between two threads. Signed-off-by: Jonathan Creekmore jonathan.creekm...@gmail.com --- tools/xenstore/xs.c

[Xen-devel] [PATCH v2] oxenstored: link in the systemd system library

If systemd is configured for use AND you are building oxenstored, the C systemd library must be linked in to the systemd.cxma library. Signed-off-by: Jonathan Creekmore jonathan.creekm...@gmail.com --- Changed since v1: * Link the systemd library in to the systemd.cxma instead of the final

Re: [Xen-devel] [PATCH v2] oxenstored: link in the systemd system library

On Jul 23, 2015, at 9:00 AM, Ian Campbell ian.campb...@citrix.com wrote: On Thu, 2015-07-23 at 08:40 -0500, Jonathan Creekmore wrote: If systemd is configured for use AND you are building oxenstored, the C systemd library must be linked in to the systemd.cxma library. Signed-off

[Xen-devel] [PATCH] oxenstored: link in the systemd system library

If systemd is configured for use AND you are building oxenstored, the C systemd library must be linked in to the oxenstored binary instead of just into the static ocaml stub. Signed-off-by: Jonathan Creekmore jonathan.creekm...@gmail.com --- tools/ocaml/xenstored/Makefile | 4 +++- 1 file