* [PATCH raisin 0/4] Fixes for 4.6, update to 4.7 @ 2016-06-13 14:44 George Dunlap 2016-06-13 14:44 ` [PATCH raisin 1/4] components/xen: Actually disable rombios George Dunlap ` (3 more replies) 0 siblings, 4 replies; 29+ messages in thread From: George Dunlap @ 2016-06-13 14:44 UTC (permalink / raw) To: xen-devel; +Cc: George Dunlap George Dunlap (4): components/xen: Actually disable rombios config: Separate config urls into a separate file Update config-4.6 and config-4.5 to point to stable branches Update to 4.7, update qemu and qemu_traditional recipes components/qemu | 21 ++++++++++++-------- components/qemu_traditional | 2 +- components/xen | 2 +- configs/config-4.5 | 45 +++---------------------------------------- configs/config-4.6 | 47 ++++----------------------------------------- configs/config-4.7 | 8 ++++++++ configs/config-master | 39 ++----------------------------------- configs/config-url-git | 8 ++++++++ configs/config-url-http | 7 +++++++ defconfig | 35 ++++++++++++++++++++++++++++++++- 10 files changed, 81 insertions(+), 133 deletions(-) create mode 100644 configs/config-4.7 create mode 100644 configs/config-url-git create mode 100644 configs/config-url-http mode change 120000 => 100644 defconfig -- 2.1.4 _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 29+ messages in thread
* [PATCH raisin 1/4] components/xen: Actually disable rombios 2016-06-13 14:44 [PATCH raisin 0/4] Fixes for 4.6, update to 4.7 George Dunlap @ 2016-06-13 14:44 ` George Dunlap 2016-06-14 9:32 ` Stefano Stabellini 2016-06-13 14:44 ` [PATCH raisin 2/4] config: Separate config urls into a separate file George Dunlap ` (2 subsequent siblings) 3 siblings, 1 reply; 29+ messages in thread From: George Dunlap @ 2016-06-13 14:44 UTC (permalink / raw) To: xen-devel; +Cc: Stefano Stabellini, George Dunlap Commit 5fe3855 meant to disable rombios, but didn't. This causes the following build failure: gcc -O2 -fomit-frame-pointer -m32 -march=i686 -fno-strict-aliasing -std=gnu99 -Wall -Wstrict-prototypes -Wdeclaration-after-statement -Wno-unused-but-set-variable -Wno-unused-local-typedefs -D__XEN_TOOLS__ -MMD -MF .rombios.o.d -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -mno-tls-direct-seg-refs -DNDEBUG -Werror -fno-stack-protector -fno-exceptions -fno-builtin -msoft-float -I/usr/local/src/xenboil/raisin/xen-dir-remote/tools/firmware/hvmloader/../../../tools/include -U__XEN_TOOLS__ -D__XEN_INTERFACE_VERSION__=__XEN_LATEST_INTERFACE_VERSION__ -DENABLE_OVMF -DENABLE_ROMBIOS -DENABLE_SEABIOS -c -o rombios.o rombios.c rombios.c: In function ‘rombios_load_roms’: rombios.c:103:39: error: ‘etherboot’ undeclared (first use in this function) etherboot); Disable rombios instead. Reported-by: Holger Schramm <lists@schramm.by> Signed-off-by: George Dunlap <george.dunlap@citrix.com> --- CC: Stefano Stabellini <sstabellini@kernel.org> --- components/xen | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/components/xen b/components/xen index 894d119..ea290aa 100644 --- a/components/xen +++ b/components/xen @@ -45,7 +45,7 @@ function xen_build() { export ETHERBOOT_NICS="" ./configure --prefix=$PREFIX --with-system-qemu=$PREFIX/lib/xen/bin/qemu-system-i386 \ --disable-stubdom --disable-qemu-traditional \ - --enable-rombios $seabios_opt $ovmf_opt + --disable-rombios $seabios_opt $ovmf_opt $RAISIN_MAKE $RAISIN_MAKE install DESTDIR="$INST_DIR" unset ETHERBOOT_NICS -- 2.1.4 _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply related [flat|nested] 29+ messages in thread
* Re: [PATCH raisin 1/4] components/xen: Actually disable rombios 2016-06-13 14:44 ` [PATCH raisin 1/4] components/xen: Actually disable rombios George Dunlap @ 2016-06-14 9:32 ` Stefano Stabellini 0 siblings, 0 replies; 29+ messages in thread From: Stefano Stabellini @ 2016-06-14 9:32 UTC (permalink / raw) To: George Dunlap; +Cc: xen-devel, Stefano Stabellini [-- Attachment #1: Type: TEXT/PLAIN, Size: 1872 bytes --] On Mon, 13 Jun 2016, George Dunlap wrote: > Commit 5fe3855 meant to disable rombios, but didn't. This causes the following > build failure: > > gcc -O2 -fomit-frame-pointer -m32 -march=i686 -fno-strict-aliasing > -std=gnu99 -Wall -Wstrict-prototypes -Wdeclaration-after-statement > -Wno-unused-but-set-variable -Wno-unused-local-typedefs > -D__XEN_TOOLS__ -MMD -MF .rombios.o.d -D_LARGEFILE_SOURCE > -D_LARGEFILE64_SOURCE -mno-tls-direct-seg-refs -DNDEBUG -Werror > -fno-stack-protector -fno-exceptions -fno-builtin -msoft-float > -I/usr/local/src/xenboil/raisin/xen-dir-remote/tools/firmware/hvmloader/../../../tools/include > -U__XEN_TOOLS__ > -D__XEN_INTERFACE_VERSION__=__XEN_LATEST_INTERFACE_VERSION__ > -DENABLE_OVMF -DENABLE_ROMBIOS -DENABLE_SEABIOS -c -o rombios.o rombios.c > rombios.c: In function ‘rombios_load_roms’: > rombios.c:103:39: error: ‘etherboot’ undeclared (first use in this function) > etherboot); > > Disable rombios instead. > > Reported-by: Holger Schramm <lists@schramm.by> > Signed-off-by: George Dunlap <george.dunlap@citrix.com> Acked-by: Stefano Stabellini <sstabellini@kernel.org> > CC: Stefano Stabellini <sstabellini@kernel.org> > --- > components/xen | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/components/xen b/components/xen > index 894d119..ea290aa 100644 > --- a/components/xen > +++ b/components/xen > @@ -45,7 +45,7 @@ function xen_build() { > export ETHERBOOT_NICS="" > ./configure --prefix=$PREFIX --with-system-qemu=$PREFIX/lib/xen/bin/qemu-system-i386 \ > --disable-stubdom --disable-qemu-traditional \ > - --enable-rombios $seabios_opt $ovmf_opt > + --disable-rombios $seabios_opt $ovmf_opt > $RAISIN_MAKE > $RAISIN_MAKE install DESTDIR="$INST_DIR" > unset ETHERBOOT_NICS > -- > 2.1.4 > [-- Attachment #2: Type: text/plain, Size: 126 bytes --] _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 29+ messages in thread
* [PATCH raisin 2/4] config: Separate config urls into a separate file 2016-06-13 14:44 [PATCH raisin 0/4] Fixes for 4.6, update to 4.7 George Dunlap 2016-06-13 14:44 ` [PATCH raisin 1/4] components/xen: Actually disable rombios George Dunlap @ 2016-06-13 14:44 ` George Dunlap 2016-06-14 9:34 ` Stefano Stabellini 2016-06-13 14:44 ` [PATCH raisin 3/4] Update config-4.6 and config-4.5 to point to stable branches George Dunlap 2016-06-13 14:44 ` [PATCH raisin 4/4] Update to 4.7, update qemu and qemu_traditional recipes George Dunlap 3 siblings, 1 reply; 29+ messages in thread From: George Dunlap @ 2016-06-13 14:44 UTC (permalink / raw) To: xen-devel; +Cc: Stefano Stabellini, George Dunlap So that we're not duplicating information. Signed-off-by: George Dunlap <george.dunlap@citrix.com> --- CC: Stefano Stabellini <sstabellini@kernel.org> --- configs/config-4.5 | 39 --------------------------------------- configs/config-4.6 | 39 --------------------------------------- configs/config-master | 38 -------------------------------------- configs/config-url-git | 8 ++++++++ configs/config-url-http | 7 +++++++ defconfig | 35 ++++++++++++++++++++++++++++++++++- 6 files changed, 49 insertions(+), 117 deletions(-) diff --git a/configs/config-4.5 b/configs/config-4.5 index e3b92d5..4163b68 100644 --- a/configs/config-4.5 +++ b/configs/config-4.5 @@ -1,37 +1,3 @@ -# Config variables for raisin -# Setup a Xen 4.5 based system - -# Components -## All components: seabios ovmf xen qemu qemu_traditional grub libvirt linux -## Core xen functionality: xen -## Remove a component from the list below, if you want to disable it -## You can manually overwrite this list using the COMPONENTS -## environmental variable. -ENABLED_COMPONENTS="seabios ovmf xen qemu qemu_traditional grub libvirt" - -# Build config -## Make command to run -MAKE="make -j2" -## Installation prefix (configure --prefix) -PREFIX="/usr" -## Install everything under DESTDIR -## If you want to install under / run raise.sh -i -DESTDIR=dist - -# Git urls. Use the http urls if you are behind a firewall. -#XEN_URL="http://xenbits.xen.org/git-http/xen.git" -#GRUB_URL="http://git.savannah.gnu.org/r/grub.git" -#LIBVIRT_URL="https://gitorious.org/libvirt/libvirt.git" -XEN_URL="git://xenbits.xen.org/xen.git" -QEMU_URL="git://xenbits.xen.org/qemu-upstream-4.5-testing.git" -QEMU_TRADITIONAL_URL="git://xenbits.xen.org/qemu-xen-4.5-testing.git" -SEABIOS_URL="git://xenbits.xen.org/seabios.git" -GRUB_URL="git://git.savannah.gnu.org/grub.git" -LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" -OVMF_URL="git://xenbits.xen.org/ovmf.git" -LINUX_URL="git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git" - -# Software versions. XEN_REVISION="origin/stable-4.5" QEMU_REVISION="master" QEMU_TRADITIONAL_REVISION="master" @@ -40,8 +6,3 @@ GRUB_REVISION="master" LIBVIRT_REVISION="origin/xen-tested-master" OVMF_REVISION="cb9a7ebabcd6b8a49dc0854b2f9592d732b5afbd" LINUX_REVISION="master" - -# Tests -## All tests: busybox-pv busybox-hvm -## ENABLED_TESTS is the list of test run by raise test -ENABLED_TESTS="busybox-pv busybox-hvm" diff --git a/configs/config-4.6 b/configs/config-4.6 index ebd45e7..e8b2a09 100644 --- a/configs/config-4.6 +++ b/configs/config-4.6 @@ -1,37 +1,3 @@ -# Config variables for raisin -# Setup a Xen 4.6 based system - -# Components -## All components: seabios ovmf xen qemu qemu_traditional grub libvirt linux -## Core xen functionality: xen -## Remove a component from the list below, if you want to disable it -## You can manually overwrite this list using the COMPONENTS -## environmental variable. -ENABLED_COMPONENTS="seabios ovmf xen qemu qemu_traditional grub libvirt" - -# Build config -## Make command to run -MAKE="make -j2" -## Installation prefix (configure --prefix) -PREFIX="/usr" -## Install everything under DESTDIR -## If you want to install under / run raise.sh -i -DESTDIR=dist - -# Git urls. Use the http urls if you are behind a firewall. -#XEN_URL="http://xenbits.xen.org/git-http/xen.git" -#GRUB_URL="http://git.savannah.gnu.org/r/grub.git" -#LIBVIRT_URL="https://gitorious.org/libvirt/libvirt.git" -XEN_URL="git://xenbits.xen.org/xen.git" -QEMU_URL="git://xenbits.xen.org/qemu-upstream-4.6-testing.git" -QEMU_TRADITIONAL_URL="git://xenbits.xen.org/qemu-xen-4.6-testing.git" -SEABIOS_URL="git://xenbits.xen.org/seabios.git" -GRUB_URL="git://git.savannah.gnu.org/grub.git" -LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" -OVMF_URL="git://xenbits.xen.org/ovmf.git" -LINUX_URL="git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git" - -# Software versions. XEN_REVISION="origin/stable-4.6" QEMU_REVISION="master" QEMU_TRADITIONAL_REVISION="master" @@ -40,8 +6,3 @@ GRUB_REVISION="master" LIBVIRT_REVISION="origin/xen-tested-master" OVMF_REVISION="cb9a7ebabcd6b8a49dc0854b2f9592d732b5afbd" LINUX_REVISION="master" - -# Tests -## All tests: busybox-pv busybox-hvm -## ENABLED_TESTS is the list of test run by raise test -ENABLED_TESTS="busybox-pv busybox-hvm" diff --git a/configs/config-master b/configs/config-master index b218708..bd26ce3 100644 --- a/configs/config-master +++ b/configs/config-master @@ -1,36 +1,3 @@ -# Config variables for raisin - -# Components -## All components: seabios ovmf xen qemu qemu_traditional grub libvirt linux -## Core xen functionality: xen -## Remove a component from the list below, if you want to disable it -## You can manually overwrite this list using the COMPONENTS -## environmental variable. -ENABLED_COMPONENTS="seabios ovmf xen qemu qemu_traditional grub libvirt" - -# Build config -## Make command to run -MAKE="make -j2" -## Installation prefix (configure --prefix) -PREFIX="/usr" -## Install everything under DESTDIR -## If you want to install under / run raise.sh -i -DESTDIR=dist - -# Git urls. Use the http urls if you are behind a firewall. -#XEN_URL="http://xenbits.xen.org/git-http/xen.git" -#GRUB_URL="http://git.savannah.gnu.org/r/grub.git" -#LIBVIRT_URL="https://gitorious.org/libvirt/libvirt.git" -XEN_URL="git://xenbits.xen.org/xen.git" -QEMU_URL="git://xenbits.xen.org/qemu-upstream-unstable.git" -QEMU_TRADITIONAL_URL="git://xenbits.xen.org/qemu-xen-unstable.git" -SEABIOS_URL="git://xenbits.xen.org/osstest/seabios.git" -GRUB_URL="git://git.savannah.gnu.org/grub.git" -LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" -OVMF_URL="git://xenbits.xen.org/osstest/ovmf.git" -LINUX_URL="git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git" - -# Software versions. XEN_REVISION="master" QEMU_REVISION="master" QEMU_TRADITIONAL_REVISION="master" @@ -39,8 +6,3 @@ GRUB_REVISION="master" LIBVIRT_REVISION="origin/xen-tested-master" OVMF_REVISION="origin/xen-tested-master" LINUX_REVISION="master" - -# Tests -## All tests: busybox-pv busybox-hvm -## ENABLED_TESTS is the list of test run by raise test -ENABLED_TESTS="busybox-pv busybox-hvm" diff --git a/configs/config-url-git b/configs/config-url-git new file mode 100644 index 0000000..79813c4 --- /dev/null +++ b/configs/config-url-git @@ -0,0 +1,8 @@ +XEN_URL="git://xenbits.xen.org/xen.git" +QEMU_URL="git://xenbits.xen.org/qemu-xen.git" +QEMU_TRADITIONAL_URL="git://xenbits.xen.org/qemu-xen-traditional.git" +SEABIOS_URL="git://xenbits.xen.org/seabios.git" +GRUB_URL="git://git.savannah.gnu.org/grub.git" +LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" +OVMF_URL="git://xenbits.xen.org/ovmf.git" +LINUX_URL="git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git" diff --git a/configs/config-url-http b/configs/config-url-http new file mode 100644 index 0000000..0ea8484 --- /dev/null +++ b/configs/config-url-http @@ -0,0 +1,7 @@ +XEN_URL="http://xenbits.xen.org/git-http/xen.git" +QEMU_URL="http://xenbits.xen.org/git-http/qemu-xen.git" +QEMU_TRADITIONAL_URL="http://xenbits.xen.org/git-http/qemu-xen-traditional.git" +SEABIOS_URL="http://xenbits.xen.org/git-http/seabios.git" +GRUB_URL="http://git.savannah.gnu.org/r/grub.git" +LIBVIRT_URL="https://gitorious.org/libvirt/libvirt.git" +OVMF_URL="http://xenbits.xen.org/git-http/ovmf.git" diff --git a/defconfig b/defconfig deleted file mode 120000 index 7ba643e..0000000 --- a/defconfig +++ /dev/null @@ -1 +0,0 @@ -configs/config-4.6 \ No newline at end of file diff --git a/defconfig b/defconfig new file mode 100644 index 0000000..8e79a43 --- /dev/null +++ b/defconfig @@ -0,0 +1,34 @@ +# Config variables for raisin +# Setup a Xen based system. + +# Available options: 4.5, 4.6, master (for development branch) +XEN_RELEASE="4.6" + +# Components +## All components: seabios ovmf xen qemu qemu_traditional grub libvirt linux +## Core xen functionality: xen +## Remove a component from the list below, if you want to disable it +## You can manually overwrite this list using the COMPONENTS +## environmental variable. +ENABLED_COMPONENTS="seabios ovmf xen qemu qemu_traditional grub libvirt" + +# Build config +## Make command to run +MAKE="make -j2" +## Installation prefix (configure --prefix) +PREFIX="/usr" +## Install everything under DESTDIR +## If you want to install under / run raise.sh -i +DESTDIR=dist + +# Git transport. Use "http" if you're behind a firewall +GIT_TRANSPORT="git" +. configs/config-url-${GIT_TRANSPORT} + +# Software versions. +. configs/config-${XEN_RELEASE} + +# Tests +## All tests: busybox-pv busybox-hvm +## ENABLED_TESTS is the list of test run by raise test +ENABLED_TESTS="busybox-pv busybox-hvm" -- 2.1.4 _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply related [flat|nested] 29+ messages in thread
* Re: [PATCH raisin 2/4] config: Separate config urls into a separate file 2016-06-13 14:44 ` [PATCH raisin 2/4] config: Separate config urls into a separate file George Dunlap @ 2016-06-14 9:34 ` Stefano Stabellini 0 siblings, 0 replies; 29+ messages in thread From: Stefano Stabellini @ 2016-06-14 9:34 UTC (permalink / raw) To: George Dunlap; +Cc: xen-devel, Stefano Stabellini On Mon, 13 Jun 2016, George Dunlap wrote: > So that we're not duplicating information. > > Signed-off-by: George Dunlap <george.dunlap@citrix.com> Acked-by: Stefano Stabellini <sstabellini@kernel.org> > configs/config-4.5 | 39 --------------------------------------- > configs/config-4.6 | 39 --------------------------------------- > configs/config-master | 38 -------------------------------------- > configs/config-url-git | 8 ++++++++ > configs/config-url-http | 7 +++++++ > defconfig | 35 ++++++++++++++++++++++++++++++++++- > 6 files changed, 49 insertions(+), 117 deletions(-) > > diff --git a/configs/config-4.5 b/configs/config-4.5 > index e3b92d5..4163b68 100644 > --- a/configs/config-4.5 > +++ b/configs/config-4.5 > @@ -1,37 +1,3 @@ > -# Config variables for raisin > -# Setup a Xen 4.5 based system > - > -# Components > -## All components: seabios ovmf xen qemu qemu_traditional grub libvirt linux > -## Core xen functionality: xen > -## Remove a component from the list below, if you want to disable it > -## You can manually overwrite this list using the COMPONENTS > -## environmental variable. > -ENABLED_COMPONENTS="seabios ovmf xen qemu qemu_traditional grub libvirt" > - > -# Build config > -## Make command to run > -MAKE="make -j2" > -## Installation prefix (configure --prefix) > -PREFIX="/usr" > -## Install everything under DESTDIR > -## If you want to install under / run raise.sh -i > -DESTDIR=dist > - > -# Git urls. Use the http urls if you are behind a firewall. > -#XEN_URL="http://xenbits.xen.org/git-http/xen.git" > -#GRUB_URL="http://git.savannah.gnu.org/r/grub.git" > -#LIBVIRT_URL="https://gitorious.org/libvirt/libvirt.git" > -XEN_URL="git://xenbits.xen.org/xen.git" > -QEMU_URL="git://xenbits.xen.org/qemu-upstream-4.5-testing.git" > -QEMU_TRADITIONAL_URL="git://xenbits.xen.org/qemu-xen-4.5-testing.git" > -SEABIOS_URL="git://xenbits.xen.org/seabios.git" > -GRUB_URL="git://git.savannah.gnu.org/grub.git" > -LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" > -OVMF_URL="git://xenbits.xen.org/ovmf.git" > -LINUX_URL="git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git" > - > -# Software versions. > XEN_REVISION="origin/stable-4.5" > QEMU_REVISION="master" > QEMU_TRADITIONAL_REVISION="master" > @@ -40,8 +6,3 @@ GRUB_REVISION="master" > LIBVIRT_REVISION="origin/xen-tested-master" > OVMF_REVISION="cb9a7ebabcd6b8a49dc0854b2f9592d732b5afbd" > LINUX_REVISION="master" > - > -# Tests > -## All tests: busybox-pv busybox-hvm > -## ENABLED_TESTS is the list of test run by raise test > -ENABLED_TESTS="busybox-pv busybox-hvm" > diff --git a/configs/config-4.6 b/configs/config-4.6 > index ebd45e7..e8b2a09 100644 > --- a/configs/config-4.6 > +++ b/configs/config-4.6 > @@ -1,37 +1,3 @@ > -# Config variables for raisin > -# Setup a Xen 4.6 based system > - > -# Components > -## All components: seabios ovmf xen qemu qemu_traditional grub libvirt linux > -## Core xen functionality: xen > -## Remove a component from the list below, if you want to disable it > -## You can manually overwrite this list using the COMPONENTS > -## environmental variable. > -ENABLED_COMPONENTS="seabios ovmf xen qemu qemu_traditional grub libvirt" > - > -# Build config > -## Make command to run > -MAKE="make -j2" > -## Installation prefix (configure --prefix) > -PREFIX="/usr" > -## Install everything under DESTDIR > -## If you want to install under / run raise.sh -i > -DESTDIR=dist > - > -# Git urls. Use the http urls if you are behind a firewall. > -#XEN_URL="http://xenbits.xen.org/git-http/xen.git" > -#GRUB_URL="http://git.savannah.gnu.org/r/grub.git" > -#LIBVIRT_URL="https://gitorious.org/libvirt/libvirt.git" > -XEN_URL="git://xenbits.xen.org/xen.git" > -QEMU_URL="git://xenbits.xen.org/qemu-upstream-4.6-testing.git" > -QEMU_TRADITIONAL_URL="git://xenbits.xen.org/qemu-xen-4.6-testing.git" > -SEABIOS_URL="git://xenbits.xen.org/seabios.git" > -GRUB_URL="git://git.savannah.gnu.org/grub.git" > -LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" > -OVMF_URL="git://xenbits.xen.org/ovmf.git" > -LINUX_URL="git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git" > - > -# Software versions. > XEN_REVISION="origin/stable-4.6" > QEMU_REVISION="master" > QEMU_TRADITIONAL_REVISION="master" > @@ -40,8 +6,3 @@ GRUB_REVISION="master" > LIBVIRT_REVISION="origin/xen-tested-master" > OVMF_REVISION="cb9a7ebabcd6b8a49dc0854b2f9592d732b5afbd" > LINUX_REVISION="master" > - > -# Tests > -## All tests: busybox-pv busybox-hvm > -## ENABLED_TESTS is the list of test run by raise test > -ENABLED_TESTS="busybox-pv busybox-hvm" > diff --git a/configs/config-master b/configs/config-master > index b218708..bd26ce3 100644 > --- a/configs/config-master > +++ b/configs/config-master > @@ -1,36 +1,3 @@ > -# Config variables for raisin > - > -# Components > -## All components: seabios ovmf xen qemu qemu_traditional grub libvirt linux > -## Core xen functionality: xen > -## Remove a component from the list below, if you want to disable it > -## You can manually overwrite this list using the COMPONENTS > -## environmental variable. > -ENABLED_COMPONENTS="seabios ovmf xen qemu qemu_traditional grub libvirt" > - > -# Build config > -## Make command to run > -MAKE="make -j2" > -## Installation prefix (configure --prefix) > -PREFIX="/usr" > -## Install everything under DESTDIR > -## If you want to install under / run raise.sh -i > -DESTDIR=dist > - > -# Git urls. Use the http urls if you are behind a firewall. > -#XEN_URL="http://xenbits.xen.org/git-http/xen.git" > -#GRUB_URL="http://git.savannah.gnu.org/r/grub.git" > -#LIBVIRT_URL="https://gitorious.org/libvirt/libvirt.git" > -XEN_URL="git://xenbits.xen.org/xen.git" > -QEMU_URL="git://xenbits.xen.org/qemu-upstream-unstable.git" > -QEMU_TRADITIONAL_URL="git://xenbits.xen.org/qemu-xen-unstable.git" > -SEABIOS_URL="git://xenbits.xen.org/osstest/seabios.git" > -GRUB_URL="git://git.savannah.gnu.org/grub.git" > -LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" > -OVMF_URL="git://xenbits.xen.org/osstest/ovmf.git" > -LINUX_URL="git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git" > - > -# Software versions. > XEN_REVISION="master" > QEMU_REVISION="master" > QEMU_TRADITIONAL_REVISION="master" > @@ -39,8 +6,3 @@ GRUB_REVISION="master" > LIBVIRT_REVISION="origin/xen-tested-master" > OVMF_REVISION="origin/xen-tested-master" > LINUX_REVISION="master" > - > -# Tests > -## All tests: busybox-pv busybox-hvm > -## ENABLED_TESTS is the list of test run by raise test > -ENABLED_TESTS="busybox-pv busybox-hvm" > diff --git a/configs/config-url-git b/configs/config-url-git > new file mode 100644 > index 0000000..79813c4 > --- /dev/null > +++ b/configs/config-url-git > @@ -0,0 +1,8 @@ > +XEN_URL="git://xenbits.xen.org/xen.git" > +QEMU_URL="git://xenbits.xen.org/qemu-xen.git" > +QEMU_TRADITIONAL_URL="git://xenbits.xen.org/qemu-xen-traditional.git" > +SEABIOS_URL="git://xenbits.xen.org/seabios.git" > +GRUB_URL="git://git.savannah.gnu.org/grub.git" > +LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" > +OVMF_URL="git://xenbits.xen.org/ovmf.git" > +LINUX_URL="git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git" > diff --git a/configs/config-url-http b/configs/config-url-http > new file mode 100644 > index 0000000..0ea8484 > --- /dev/null > +++ b/configs/config-url-http > @@ -0,0 +1,7 @@ > +XEN_URL="http://xenbits.xen.org/git-http/xen.git" > +QEMU_URL="http://xenbits.xen.org/git-http/qemu-xen.git" > +QEMU_TRADITIONAL_URL="http://xenbits.xen.org/git-http/qemu-xen-traditional.git" > +SEABIOS_URL="http://xenbits.xen.org/git-http/seabios.git" > +GRUB_URL="http://git.savannah.gnu.org/r/grub.git" > +LIBVIRT_URL="https://gitorious.org/libvirt/libvirt.git" > +OVMF_URL="http://xenbits.xen.org/git-http/ovmf.git" > diff --git a/defconfig b/defconfig > deleted file mode 120000 > index 7ba643e..0000000 > --- a/defconfig > +++ /dev/null > @@ -1 +0,0 @@ > -configs/config-4.6 > \ No newline at end of file > diff --git a/defconfig b/defconfig > new file mode 100644 > index 0000000..8e79a43 > --- /dev/null > +++ b/defconfig > @@ -0,0 +1,34 @@ > +# Config variables for raisin > +# Setup a Xen based system. > + > +# Available options: 4.5, 4.6, master (for development branch) > +XEN_RELEASE="4.6" > + > +# Components > +## All components: seabios ovmf xen qemu qemu_traditional grub libvirt linux > +## Core xen functionality: xen > +## Remove a component from the list below, if you want to disable it > +## You can manually overwrite this list using the COMPONENTS > +## environmental variable. > +ENABLED_COMPONENTS="seabios ovmf xen qemu qemu_traditional grub libvirt" > + > +# Build config > +## Make command to run > +MAKE="make -j2" > +## Installation prefix (configure --prefix) > +PREFIX="/usr" > +## Install everything under DESTDIR > +## If you want to install under / run raise.sh -i > +DESTDIR=dist > + > +# Git transport. Use "http" if you're behind a firewall > +GIT_TRANSPORT="git" > +. configs/config-url-${GIT_TRANSPORT} > + > +# Software versions. > +. configs/config-${XEN_RELEASE} > + > +# Tests > +## All tests: busybox-pv busybox-hvm > +## ENABLED_TESTS is the list of test run by raise test > +ENABLED_TESTS="busybox-pv busybox-hvm" > -- > 2.1.4 > _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 29+ messages in thread
* [PATCH raisin 3/4] Update config-4.6 and config-4.5 to point to stable branches 2016-06-13 14:44 [PATCH raisin 0/4] Fixes for 4.6, update to 4.7 George Dunlap 2016-06-13 14:44 ` [PATCH raisin 1/4] components/xen: Actually disable rombios George Dunlap 2016-06-13 14:44 ` [PATCH raisin 2/4] config: Separate config urls into a separate file George Dunlap @ 2016-06-13 14:44 ` George Dunlap 2016-06-14 9:40 ` Stefano Stabellini 2016-06-13 14:44 ` [PATCH raisin 4/4] Update to 4.7, update qemu and qemu_traditional recipes George Dunlap 3 siblings, 1 reply; 29+ messages in thread From: George Dunlap @ 2016-06-13 14:44 UTC (permalink / raw) To: xen-devel; +Cc: Stefano Stabellini, George Dunlap Point xen, qemu, and qemu-trad to stable-4.5 and -4.6 branches. And point the default libvirt to point to the libvirt 1.3.3 maintenance branch, rather than xen-tested-master. Also update OVMF revision for 4.6 to a version that builds with modern gccs. Singed-off-by: George Dunlap <george.dunlap@citrix.com> --- CC: Stefano Stabellini <sstabellini@kernel.org> --- configs/config-4.5 | 6 +++--- configs/config-4.6 | 8 ++++---- configs/config-master | 3 +++ configs/config-url-git | 2 +- 4 files changed, 11 insertions(+), 8 deletions(-) diff --git a/configs/config-4.5 b/configs/config-4.5 index 4163b68..8db9a9d 100644 --- a/configs/config-4.5 +++ b/configs/config-4.5 @@ -1,8 +1,8 @@ XEN_REVISION="origin/stable-4.5" -QEMU_REVISION="master" -QEMU_TRADITIONAL_REVISION="master" +QEMU_REVISION="origin/stable-4.5" +QEMU_TRADITIONAL_REVISION="origin/stable-4.5" SEABIOS_REVISION="rel-1.7.5" GRUB_REVISION="master" -LIBVIRT_REVISION="origin/xen-tested-master" +LIBVIRT_REVISION="origin/v1.3.3-maint" OVMF_REVISION="cb9a7ebabcd6b8a49dc0854b2f9592d732b5afbd" LINUX_REVISION="master" diff --git a/configs/config-4.6 b/configs/config-4.6 index e8b2a09..b003a30 100644 --- a/configs/config-4.6 +++ b/configs/config-4.6 @@ -1,8 +1,8 @@ XEN_REVISION="origin/stable-4.6" -QEMU_REVISION="master" -QEMU_TRADITIONAL_REVISION="master" +QEMU_REVISION="origin/stable-4.6" +QEMU_TRADITIONAL_REVISION="origin/stable-4.6" SEABIOS_REVISION="rel-1.8.2" GRUB_REVISION="master" -LIBVIRT_REVISION="origin/xen-tested-master" -OVMF_REVISION="cb9a7ebabcd6b8a49dc0854b2f9592d732b5afbd" +LIBVIRT_REVISION="origin/v1.3.3-maint" +OVMF_REVISION="52a99493cce88a9d4ec8a02d7f1bd1a1001ce60d" LINUX_REVISION="master" diff --git a/configs/config-master b/configs/config-master index bd26ce3..fce7436 100644 --- a/configs/config-master +++ b/configs/config-master @@ -6,3 +6,6 @@ GRUB_REVISION="master" LIBVIRT_REVISION="origin/xen-tested-master" OVMF_REVISION="origin/xen-tested-master" LINUX_REVISION="master" + +# oss-tested branch +LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" diff --git a/configs/config-url-git b/configs/config-url-git index 79813c4..614f522 100644 --- a/configs/config-url-git +++ b/configs/config-url-git @@ -3,6 +3,6 @@ QEMU_URL="git://xenbits.xen.org/qemu-xen.git" QEMU_TRADITIONAL_URL="git://xenbits.xen.org/qemu-xen-traditional.git" SEABIOS_URL="git://xenbits.xen.org/seabios.git" GRUB_URL="git://git.savannah.gnu.org/grub.git" -LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" +LIBVIRT_URL="git://libvirt.org/libvirt.git" OVMF_URL="git://xenbits.xen.org/ovmf.git" LINUX_URL="git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git" -- 2.1.4 _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply related [flat|nested] 29+ messages in thread
* Re: [PATCH raisin 3/4] Update config-4.6 and config-4.5 to point to stable branches 2016-06-13 14:44 ` [PATCH raisin 3/4] Update config-4.6 and config-4.5 to point to stable branches George Dunlap @ 2016-06-14 9:40 ` Stefano Stabellini 2016-06-14 9:57 ` George Dunlap 0 siblings, 1 reply; 29+ messages in thread From: Stefano Stabellini @ 2016-06-14 9:40 UTC (permalink / raw) To: George Dunlap; +Cc: xen-devel, Stefano Stabellini On Mon, 13 Jun 2016, George Dunlap wrote: > Point xen, qemu, and qemu-trad to stable-4.5 and -4.6 branches. > > And point the default libvirt to point to the libvirt 1.3.3 > maintenance branch, rather than xen-tested-master. > > Also update OVMF revision for 4.6 to a version that builds with modern > gccs. > > Singed-off-by: George Dunlap <george.dunlap@citrix.com> > > CC: Stefano Stabellini <sstabellini@kernel.org> > --- > configs/config-4.5 | 6 +++--- > configs/config-4.6 | 8 ++++---- > configs/config-master | 3 +++ > configs/config-url-git | 2 +- > 4 files changed, 11 insertions(+), 8 deletions(-) > > diff --git a/configs/config-4.5 b/configs/config-4.5 > index 4163b68..8db9a9d 100644 > --- a/configs/config-4.5 > +++ b/configs/config-4.5 > @@ -1,8 +1,8 @@ > XEN_REVISION="origin/stable-4.5" > -QEMU_REVISION="master" > -QEMU_TRADITIONAL_REVISION="master" > +QEMU_REVISION="origin/stable-4.5" > +QEMU_TRADITIONAL_REVISION="origin/stable-4.5" > SEABIOS_REVISION="rel-1.7.5" > GRUB_REVISION="master" > -LIBVIRT_REVISION="origin/xen-tested-master" > +LIBVIRT_REVISION="origin/v1.3.3-maint" > OVMF_REVISION="cb9a7ebabcd6b8a49dc0854b2f9592d732b5afbd" > LINUX_REVISION="master" > diff --git a/configs/config-4.6 b/configs/config-4.6 > index e8b2a09..b003a30 100644 > --- a/configs/config-4.6 > +++ b/configs/config-4.6 > @@ -1,8 +1,8 @@ > XEN_REVISION="origin/stable-4.6" > -QEMU_REVISION="master" > -QEMU_TRADITIONAL_REVISION="master" > +QEMU_REVISION="origin/stable-4.6" > +QEMU_TRADITIONAL_REVISION="origin/stable-4.6" > SEABIOS_REVISION="rel-1.8.2" > GRUB_REVISION="master" > -LIBVIRT_REVISION="origin/xen-tested-master" > -OVMF_REVISION="cb9a7ebabcd6b8a49dc0854b2f9592d732b5afbd" > +LIBVIRT_REVISION="origin/v1.3.3-maint" > +OVMF_REVISION="52a99493cce88a9d4ec8a02d7f1bd1a1001ce60d" > LINUX_REVISION="master" > diff --git a/configs/config-master b/configs/config-master > index bd26ce3..fce7436 100644 > --- a/configs/config-master > +++ b/configs/config-master > @@ -6,3 +6,6 @@ GRUB_REVISION="master" > LIBVIRT_REVISION="origin/xen-tested-master" > OVMF_REVISION="origin/xen-tested-master" > LINUX_REVISION="master" > + > +# oss-tested branch > +LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" Why keep this around? I would remove it. > diff --git a/configs/config-url-git b/configs/config-url-git > index 79813c4..614f522 100644 > --- a/configs/config-url-git > +++ b/configs/config-url-git > @@ -3,6 +3,6 @@ QEMU_URL="git://xenbits.xen.org/qemu-xen.git" > QEMU_TRADITIONAL_URL="git://xenbits.xen.org/qemu-xen-traditional.git" > SEABIOS_URL="git://xenbits.xen.org/seabios.git" > GRUB_URL="git://git.savannah.gnu.org/grub.git" > -LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" > +LIBVIRT_URL="git://libvirt.org/libvirt.git" > OVMF_URL="git://xenbits.xen.org/ovmf.git" > LINUX_URL="git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git" > -- > 2.1.4 > _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH raisin 3/4] Update config-4.6 and config-4.5 to point to stable branches 2016-06-14 9:40 ` Stefano Stabellini @ 2016-06-14 9:57 ` George Dunlap 2016-06-14 10:01 ` Stefano Stabellini 0 siblings, 1 reply; 29+ messages in thread From: George Dunlap @ 2016-06-14 9:57 UTC (permalink / raw) To: Stefano Stabellini; +Cc: xen-devel On 14/06/16 10:40, Stefano Stabellini wrote: > On Mon, 13 Jun 2016, George Dunlap wrote: >> Point xen, qemu, and qemu-trad to stable-4.5 and -4.6 branches. >> >> And point the default libvirt to point to the libvirt 1.3.3 >> maintenance branch, rather than xen-tested-master. >> >> Also update OVMF revision for 4.6 to a version that builds with modern >> gccs. >> >> Singed-off-by: George Dunlap <george.dunlap@citrix.com> >> >> CC: Stefano Stabellini <sstabellini@kernel.org> >> --- >> configs/config-4.5 | 6 +++--- >> configs/config-4.6 | 8 ++++---- >> configs/config-master | 3 +++ >> configs/config-url-git | 2 +- >> 4 files changed, 11 insertions(+), 8 deletions(-) >> >> diff --git a/configs/config-4.5 b/configs/config-4.5 >> index 4163b68..8db9a9d 100644 >> --- a/configs/config-4.5 >> +++ b/configs/config-4.5 >> @@ -1,8 +1,8 @@ >> XEN_REVISION="origin/stable-4.5" >> -QEMU_REVISION="master" >> -QEMU_TRADITIONAL_REVISION="master" >> +QEMU_REVISION="origin/stable-4.5" >> +QEMU_TRADITIONAL_REVISION="origin/stable-4.5" >> SEABIOS_REVISION="rel-1.7.5" >> GRUB_REVISION="master" >> -LIBVIRT_REVISION="origin/xen-tested-master" >> +LIBVIRT_REVISION="origin/v1.3.3-maint" >> OVMF_REVISION="cb9a7ebabcd6b8a49dc0854b2f9592d732b5afbd" >> LINUX_REVISION="master" >> diff --git a/configs/config-4.6 b/configs/config-4.6 >> index e8b2a09..b003a30 100644 >> --- a/configs/config-4.6 >> +++ b/configs/config-4.6 >> @@ -1,8 +1,8 @@ >> XEN_REVISION="origin/stable-4.6" >> -QEMU_REVISION="master" >> -QEMU_TRADITIONAL_REVISION="master" >> +QEMU_REVISION="origin/stable-4.6" >> +QEMU_TRADITIONAL_REVISION="origin/stable-4.6" >> SEABIOS_REVISION="rel-1.8.2" >> GRUB_REVISION="master" >> -LIBVIRT_REVISION="origin/xen-tested-master" >> -OVMF_REVISION="cb9a7ebabcd6b8a49dc0854b2f9592d732b5afbd" >> +LIBVIRT_REVISION="origin/v1.3.3-maint" >> +OVMF_REVISION="52a99493cce88a9d4ec8a02d7f1bd1a1001ce60d" >> LINUX_REVISION="master" >> diff --git a/configs/config-master b/configs/config-master >> index bd26ce3..fce7436 100644 >> --- a/configs/config-master >> +++ b/configs/config-master >> @@ -6,3 +6,6 @@ GRUB_REVISION="master" >> LIBVIRT_REVISION="origin/xen-tested-master" >> OVMF_REVISION="origin/xen-tested-master" >> LINUX_REVISION="master" >> + >> +# oss-tested branch >> +LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" > > Why keep this around? I would remove it. Because... >> diff --git a/configs/config-url-git b/configs/config-url-git >> index 79813c4..614f522 100644 >> --- a/configs/config-url-git >> +++ b/configs/config-url-git >> @@ -3,6 +3,6 @@ QEMU_URL="git://xenbits.xen.org/qemu-xen.git" >> QEMU_TRADITIONAL_URL="git://xenbits.xen.org/qemu-xen-traditional.git" >> SEABIOS_URL="git://xenbits.xen.org/seabios.git" >> GRUB_URL="git://git.savannah.gnu.org/grub.git" >> -LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" >> +LIBVIRT_URL="git://libvirt.org/libvirt.git" ...here, the LIBVIRT_URL in config-url-git is set to the libvirt.org URL so that the stable releases can be pointed to v1.3.3-maint instead. The xen.org repo doesn't contain the v1.3.3-maint branch, and the libvirt.org repo doesn't contain the 'xen-tested-master' branch. I admit this is a bit ugly, *almost* separating things entirely but then not. But I don't see another easy way of having both the 1.3.3 maintenance branch and the oss-tested branch. (Unless we started having osstest test the maintenance branch as well.) -George _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH raisin 3/4] Update config-4.6 and config-4.5 to point to stable branches 2016-06-14 9:57 ` George Dunlap @ 2016-06-14 10:01 ` Stefano Stabellini 2016-06-14 10:10 ` George Dunlap 0 siblings, 1 reply; 29+ messages in thread From: Stefano Stabellini @ 2016-06-14 10:01 UTC (permalink / raw) To: George Dunlap; +Cc: xen-devel, Stefano Stabellini On Tue, 14 Jun 2016, George Dunlap wrote: > On 14/06/16 10:40, Stefano Stabellini wrote: > > On Mon, 13 Jun 2016, George Dunlap wrote: > >> Point xen, qemu, and qemu-trad to stable-4.5 and -4.6 branches. > >> > >> And point the default libvirt to point to the libvirt 1.3.3 > >> maintenance branch, rather than xen-tested-master. > >> > >> Also update OVMF revision for 4.6 to a version that builds with modern > >> gccs. > >> > >> Singed-off-by: George Dunlap <george.dunlap@citrix.com> > >> > >> CC: Stefano Stabellini <sstabellini@kernel.org> > >> --- > >> configs/config-4.5 | 6 +++--- > >> configs/config-4.6 | 8 ++++---- > >> configs/config-master | 3 +++ > >> configs/config-url-git | 2 +- > >> 4 files changed, 11 insertions(+), 8 deletions(-) > >> > >> diff --git a/configs/config-4.5 b/configs/config-4.5 > >> index 4163b68..8db9a9d 100644 > >> --- a/configs/config-4.5 > >> +++ b/configs/config-4.5 > >> @@ -1,8 +1,8 @@ > >> XEN_REVISION="origin/stable-4.5" > >> -QEMU_REVISION="master" > >> -QEMU_TRADITIONAL_REVISION="master" > >> +QEMU_REVISION="origin/stable-4.5" > >> +QEMU_TRADITIONAL_REVISION="origin/stable-4.5" > >> SEABIOS_REVISION="rel-1.7.5" > >> GRUB_REVISION="master" > >> -LIBVIRT_REVISION="origin/xen-tested-master" > >> +LIBVIRT_REVISION="origin/v1.3.3-maint" > >> OVMF_REVISION="cb9a7ebabcd6b8a49dc0854b2f9592d732b5afbd" > >> LINUX_REVISION="master" > >> diff --git a/configs/config-4.6 b/configs/config-4.6 > >> index e8b2a09..b003a30 100644 > >> --- a/configs/config-4.6 > >> +++ b/configs/config-4.6 > >> @@ -1,8 +1,8 @@ > >> XEN_REVISION="origin/stable-4.6" > >> -QEMU_REVISION="master" > >> -QEMU_TRADITIONAL_REVISION="master" > >> +QEMU_REVISION="origin/stable-4.6" > >> +QEMU_TRADITIONAL_REVISION="origin/stable-4.6" > >> SEABIOS_REVISION="rel-1.8.2" > >> GRUB_REVISION="master" > >> -LIBVIRT_REVISION="origin/xen-tested-master" > >> -OVMF_REVISION="cb9a7ebabcd6b8a49dc0854b2f9592d732b5afbd" > >> +LIBVIRT_REVISION="origin/v1.3.3-maint" > >> +OVMF_REVISION="52a99493cce88a9d4ec8a02d7f1bd1a1001ce60d" > >> LINUX_REVISION="master" > >> diff --git a/configs/config-master b/configs/config-master > >> index bd26ce3..fce7436 100644 > >> --- a/configs/config-master > >> +++ b/configs/config-master > >> @@ -6,3 +6,6 @@ GRUB_REVISION="master" > >> LIBVIRT_REVISION="origin/xen-tested-master" > >> OVMF_REVISION="origin/xen-tested-master" > >> LINUX_REVISION="master" > >> + > >> +# oss-tested branch > >> +LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" > > > > Why keep this around? I would remove it. > > Because... > > >> diff --git a/configs/config-url-git b/configs/config-url-git > >> index 79813c4..614f522 100644 > >> --- a/configs/config-url-git > >> +++ b/configs/config-url-git > >> @@ -3,6 +3,6 @@ QEMU_URL="git://xenbits.xen.org/qemu-xen.git" > >> QEMU_TRADITIONAL_URL="git://xenbits.xen.org/qemu-xen-traditional.git" > >> SEABIOS_URL="git://xenbits.xen.org/seabios.git" > >> GRUB_URL="git://git.savannah.gnu.org/grub.git" > >> -LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" > >> +LIBVIRT_URL="git://libvirt.org/libvirt.git" > > ...here, the LIBVIRT_URL in config-url-git is set to the libvirt.org URL > so that the stable releases can be pointed to v1.3.3-maint instead. The > xen.org repo doesn't contain the v1.3.3-maint branch, and the > libvirt.org repo doesn't contain the 'xen-tested-master' branch. > > I admit this is a bit ugly, *almost* separating things entirely but then > not. But I don't see another easy way of having both the 1.3.3 > maintenance branch and the oss-tested branch. (Unless we started having > osstest test the maintenance branch as well.) I would drop git://xenbits.xen.org/libvirt.git and xen-tested-master entirely and just stick to v1.3.3-maint everywhere. Or the other way around. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH raisin 3/4] Update config-4.6 and config-4.5 to point to stable branches 2016-06-14 10:01 ` Stefano Stabellini @ 2016-06-14 10:10 ` George Dunlap 2016-06-15 10:24 ` Stefano Stabellini 0 siblings, 1 reply; 29+ messages in thread From: George Dunlap @ 2016-06-14 10:10 UTC (permalink / raw) To: Stefano Stabellini; +Cc: xen-devel On 14/06/16 11:01, Stefano Stabellini wrote: > On Tue, 14 Jun 2016, George Dunlap wrote: >> On 14/06/16 10:40, Stefano Stabellini wrote: >>> On Mon, 13 Jun 2016, George Dunlap wrote: >>>> Point xen, qemu, and qemu-trad to stable-4.5 and -4.6 branches. >>>> >>>> And point the default libvirt to point to the libvirt 1.3.3 >>>> maintenance branch, rather than xen-tested-master. >>>> >>>> Also update OVMF revision for 4.6 to a version that builds with modern >>>> gccs. >>>> >>>> Singed-off-by: George Dunlap <george.dunlap@citrix.com> >>>> >>>> CC: Stefano Stabellini <sstabellini@kernel.org> >>>> --- >>>> configs/config-4.5 | 6 +++--- >>>> configs/config-4.6 | 8 ++++---- >>>> configs/config-master | 3 +++ >>>> configs/config-url-git | 2 +- >>>> 4 files changed, 11 insertions(+), 8 deletions(-) >>>> >>>> diff --git a/configs/config-4.5 b/configs/config-4.5 >>>> index 4163b68..8db9a9d 100644 >>>> --- a/configs/config-4.5 >>>> +++ b/configs/config-4.5 >>>> @@ -1,8 +1,8 @@ >>>> XEN_REVISION="origin/stable-4.5" >>>> -QEMU_REVISION="master" >>>> -QEMU_TRADITIONAL_REVISION="master" >>>> +QEMU_REVISION="origin/stable-4.5" >>>> +QEMU_TRADITIONAL_REVISION="origin/stable-4.5" >>>> SEABIOS_REVISION="rel-1.7.5" >>>> GRUB_REVISION="master" >>>> -LIBVIRT_REVISION="origin/xen-tested-master" >>>> +LIBVIRT_REVISION="origin/v1.3.3-maint" >>>> OVMF_REVISION="cb9a7ebabcd6b8a49dc0854b2f9592d732b5afbd" >>>> LINUX_REVISION="master" >>>> diff --git a/configs/config-4.6 b/configs/config-4.6 >>>> index e8b2a09..b003a30 100644 >>>> --- a/configs/config-4.6 >>>> +++ b/configs/config-4.6 >>>> @@ -1,8 +1,8 @@ >>>> XEN_REVISION="origin/stable-4.6" >>>> -QEMU_REVISION="master" >>>> -QEMU_TRADITIONAL_REVISION="master" >>>> +QEMU_REVISION="origin/stable-4.6" >>>> +QEMU_TRADITIONAL_REVISION="origin/stable-4.6" >>>> SEABIOS_REVISION="rel-1.8.2" >>>> GRUB_REVISION="master" >>>> -LIBVIRT_REVISION="origin/xen-tested-master" >>>> -OVMF_REVISION="cb9a7ebabcd6b8a49dc0854b2f9592d732b5afbd" >>>> +LIBVIRT_REVISION="origin/v1.3.3-maint" >>>> +OVMF_REVISION="52a99493cce88a9d4ec8a02d7f1bd1a1001ce60d" >>>> LINUX_REVISION="master" >>>> diff --git a/configs/config-master b/configs/config-master >>>> index bd26ce3..fce7436 100644 >>>> --- a/configs/config-master >>>> +++ b/configs/config-master >>>> @@ -6,3 +6,6 @@ GRUB_REVISION="master" >>>> LIBVIRT_REVISION="origin/xen-tested-master" >>>> OVMF_REVISION="origin/xen-tested-master" >>>> LINUX_REVISION="master" >>>> + >>>> +# oss-tested branch >>>> +LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" >>> >>> Why keep this around? I would remove it. >> >> Because... >> >>>> diff --git a/configs/config-url-git b/configs/config-url-git >>>> index 79813c4..614f522 100644 >>>> --- a/configs/config-url-git >>>> +++ b/configs/config-url-git >>>> @@ -3,6 +3,6 @@ QEMU_URL="git://xenbits.xen.org/qemu-xen.git" >>>> QEMU_TRADITIONAL_URL="git://xenbits.xen.org/qemu-xen-traditional.git" >>>> SEABIOS_URL="git://xenbits.xen.org/seabios.git" >>>> GRUB_URL="git://git.savannah.gnu.org/grub.git" >>>> -LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" >>>> +LIBVIRT_URL="git://libvirt.org/libvirt.git" >> >> ...here, the LIBVIRT_URL in config-url-git is set to the libvirt.org URL >> so that the stable releases can be pointed to v1.3.3-maint instead. The >> xen.org repo doesn't contain the v1.3.3-maint branch, and the >> libvirt.org repo doesn't contain the 'xen-tested-master' branch. >> >> I admit this is a bit ugly, *almost* separating things entirely but then >> not. But I don't see another easy way of having both the 1.3.3 >> maintenance branch and the oss-tested branch. (Unless we started having >> osstest test the maintenance branch as well.) > > I would drop git://xenbits.xen.org/libvirt.git and xen-tested-master > entirely and just stick to v1.3.3-maint everywhere. Or the other way > around. Well I assume that people building one of the releases want something relatively stable and supported, so xen-tested-master is obviously unsuitable (and might not actually be capable of building against Xen versions older than a certain point due to the fact that libxl cannot be both forward- and backward-compatible). And I also assume that people building with XEN_RELEASE="master" want the bleeding edge of everything so that they can test new features (or indeed, so that they can write new features). So I think we actually do want both -- a maintenance branch for releases, and xen-tested-master for developers and bleeding-edge users. -George _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH raisin 3/4] Update config-4.6 and config-4.5 to point to stable branches 2016-06-14 10:10 ` George Dunlap @ 2016-06-15 10:24 ` Stefano Stabellini 2016-06-15 10:38 ` George Dunlap 0 siblings, 1 reply; 29+ messages in thread From: Stefano Stabellini @ 2016-06-15 10:24 UTC (permalink / raw) To: George Dunlap; +Cc: xen-devel, Stefano Stabellini On Tue, 14 Jun 2016, George Dunlap wrote: > On 14/06/16 11:01, Stefano Stabellini wrote: > > On Tue, 14 Jun 2016, George Dunlap wrote: > >> On 14/06/16 10:40, Stefano Stabellini wrote: > >>> On Mon, 13 Jun 2016, George Dunlap wrote: > >>>> Point xen, qemu, and qemu-trad to stable-4.5 and -4.6 branches. > >>>> > >>>> And point the default libvirt to point to the libvirt 1.3.3 > >>>> maintenance branch, rather than xen-tested-master. > >>>> > >>>> Also update OVMF revision for 4.6 to a version that builds with modern > >>>> gccs. > >>>> > >>>> Singed-off-by: George Dunlap <george.dunlap@citrix.com> > >>>> > >>>> CC: Stefano Stabellini <sstabellini@kernel.org> > >>>> --- > >>>> configs/config-4.5 | 6 +++--- > >>>> configs/config-4.6 | 8 ++++---- > >>>> configs/config-master | 3 +++ > >>>> configs/config-url-git | 2 +- > >>>> 4 files changed, 11 insertions(+), 8 deletions(-) > >>>> > >>>> diff --git a/configs/config-4.5 b/configs/config-4.5 > >>>> index 4163b68..8db9a9d 100644 > >>>> --- a/configs/config-4.5 > >>>> +++ b/configs/config-4.5 > >>>> @@ -1,8 +1,8 @@ > >>>> XEN_REVISION="origin/stable-4.5" > >>>> -QEMU_REVISION="master" > >>>> -QEMU_TRADITIONAL_REVISION="master" > >>>> +QEMU_REVISION="origin/stable-4.5" > >>>> +QEMU_TRADITIONAL_REVISION="origin/stable-4.5" > >>>> SEABIOS_REVISION="rel-1.7.5" > >>>> GRUB_REVISION="master" > >>>> -LIBVIRT_REVISION="origin/xen-tested-master" > >>>> +LIBVIRT_REVISION="origin/v1.3.3-maint" > >>>> OVMF_REVISION="cb9a7ebabcd6b8a49dc0854b2f9592d732b5afbd" > >>>> LINUX_REVISION="master" > >>>> diff --git a/configs/config-4.6 b/configs/config-4.6 > >>>> index e8b2a09..b003a30 100644 > >>>> --- a/configs/config-4.6 > >>>> +++ b/configs/config-4.6 > >>>> @@ -1,8 +1,8 @@ > >>>> XEN_REVISION="origin/stable-4.6" > >>>> -QEMU_REVISION="master" > >>>> -QEMU_TRADITIONAL_REVISION="master" > >>>> +QEMU_REVISION="origin/stable-4.6" > >>>> +QEMU_TRADITIONAL_REVISION="origin/stable-4.6" > >>>> SEABIOS_REVISION="rel-1.8.2" > >>>> GRUB_REVISION="master" > >>>> -LIBVIRT_REVISION="origin/xen-tested-master" > >>>> -OVMF_REVISION="cb9a7ebabcd6b8a49dc0854b2f9592d732b5afbd" > >>>> +LIBVIRT_REVISION="origin/v1.3.3-maint" > >>>> +OVMF_REVISION="52a99493cce88a9d4ec8a02d7f1bd1a1001ce60d" > >>>> LINUX_REVISION="master" > >>>> diff --git a/configs/config-master b/configs/config-master > >>>> index bd26ce3..fce7436 100644 > >>>> --- a/configs/config-master > >>>> +++ b/configs/config-master > >>>> @@ -6,3 +6,6 @@ GRUB_REVISION="master" > >>>> LIBVIRT_REVISION="origin/xen-tested-master" > >>>> OVMF_REVISION="origin/xen-tested-master" > >>>> LINUX_REVISION="master" > >>>> + > >>>> +# oss-tested branch > >>>> +LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" > >>> > >>> Why keep this around? I would remove it. > >> > >> Because... > >> > >>>> diff --git a/configs/config-url-git b/configs/config-url-git > >>>> index 79813c4..614f522 100644 > >>>> --- a/configs/config-url-git > >>>> +++ b/configs/config-url-git > >>>> @@ -3,6 +3,6 @@ QEMU_URL="git://xenbits.xen.org/qemu-xen.git" > >>>> QEMU_TRADITIONAL_URL="git://xenbits.xen.org/qemu-xen-traditional.git" > >>>> SEABIOS_URL="git://xenbits.xen.org/seabios.git" > >>>> GRUB_URL="git://git.savannah.gnu.org/grub.git" > >>>> -LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" > >>>> +LIBVIRT_URL="git://libvirt.org/libvirt.git" > >> > >> ...here, the LIBVIRT_URL in config-url-git is set to the libvirt.org URL > >> so that the stable releases can be pointed to v1.3.3-maint instead. The > >> xen.org repo doesn't contain the v1.3.3-maint branch, and the > >> libvirt.org repo doesn't contain the 'xen-tested-master' branch. > >> > >> I admit this is a bit ugly, *almost* separating things entirely but then > >> not. But I don't see another easy way of having both the 1.3.3 > >> maintenance branch and the oss-tested branch. (Unless we started having > >> osstest test the maintenance branch as well.) > > > > I would drop git://xenbits.xen.org/libvirt.git and xen-tested-master > > entirely and just stick to v1.3.3-maint everywhere. Or the other way > > around. > > Well I assume that people building one of the releases want something > relatively stable and supported, so xen-tested-master is obviously > unsuitable (and might not actually be capable of building against Xen > versions older than a certain point due to the fact that libxl cannot be > both forward- and backward-compatible). And I also assume that people > building with XEN_RELEASE="master" want the bleeding edge of everything > so that they can test new features (or indeed, so that they can write > new features). So I think we actually do want both -- a maintenance > branch for releases, and xen-tested-master for developers and > bleeding-edge users. Sure, we need to support both, but do they really need to be both enabled in one of the example config files? We could just have the libvirt xenbits tree as a comment. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH raisin 3/4] Update config-4.6 and config-4.5 to point to stable branches 2016-06-15 10:24 ` Stefano Stabellini @ 2016-06-15 10:38 ` George Dunlap 2016-06-15 10:53 ` Stefano Stabellini 0 siblings, 1 reply; 29+ messages in thread From: George Dunlap @ 2016-06-15 10:38 UTC (permalink / raw) To: Stefano Stabellini; +Cc: xen-devel On 15/06/16 11:24, Stefano Stabellini wrote: > On Tue, 14 Jun 2016, George Dunlap wrote: >> On 14/06/16 11:01, Stefano Stabellini wrote: >>> On Tue, 14 Jun 2016, George Dunlap wrote: >>>> On 14/06/16 10:40, Stefano Stabellini wrote: >>>>> On Mon, 13 Jun 2016, George Dunlap wrote: >>>>>> Point xen, qemu, and qemu-trad to stable-4.5 and -4.6 branches. >>>>>> >>>>>> And point the default libvirt to point to the libvirt 1.3.3 >>>>>> maintenance branch, rather than xen-tested-master. >>>>>> >>>>>> Also update OVMF revision for 4.6 to a version that builds with modern >>>>>> gccs. >>>>>> >>>>>> Singed-off-by: George Dunlap <george.dunlap@citrix.com> >>>>>> >>>>>> CC: Stefano Stabellini <sstabellini@kernel.org> >>>>>> --- >>>>>> configs/config-4.5 | 6 +++--- >>>>>> configs/config-4.6 | 8 ++++---- >>>>>> configs/config-master | 3 +++ >>>>>> configs/config-url-git | 2 +- >>>>>> 4 files changed, 11 insertions(+), 8 deletions(-) >>>>>> >>>>>> diff --git a/configs/config-4.5 b/configs/config-4.5 >>>>>> index 4163b68..8db9a9d 100644 >>>>>> --- a/configs/config-4.5 >>>>>> +++ b/configs/config-4.5 >>>>>> @@ -1,8 +1,8 @@ >>>>>> XEN_REVISION="origin/stable-4.5" >>>>>> -QEMU_REVISION="master" >>>>>> -QEMU_TRADITIONAL_REVISION="master" >>>>>> +QEMU_REVISION="origin/stable-4.5" >>>>>> +QEMU_TRADITIONAL_REVISION="origin/stable-4.5" >>>>>> SEABIOS_REVISION="rel-1.7.5" >>>>>> GRUB_REVISION="master" >>>>>> -LIBVIRT_REVISION="origin/xen-tested-master" >>>>>> +LIBVIRT_REVISION="origin/v1.3.3-maint" >>>>>> OVMF_REVISION="cb9a7ebabcd6b8a49dc0854b2f9592d732b5afbd" >>>>>> LINUX_REVISION="master" >>>>>> diff --git a/configs/config-4.6 b/configs/config-4.6 >>>>>> index e8b2a09..b003a30 100644 >>>>>> --- a/configs/config-4.6 >>>>>> +++ b/configs/config-4.6 >>>>>> @@ -1,8 +1,8 @@ >>>>>> XEN_REVISION="origin/stable-4.6" >>>>>> -QEMU_REVISION="master" >>>>>> -QEMU_TRADITIONAL_REVISION="master" >>>>>> +QEMU_REVISION="origin/stable-4.6" >>>>>> +QEMU_TRADITIONAL_REVISION="origin/stable-4.6" >>>>>> SEABIOS_REVISION="rel-1.8.2" >>>>>> GRUB_REVISION="master" >>>>>> -LIBVIRT_REVISION="origin/xen-tested-master" >>>>>> -OVMF_REVISION="cb9a7ebabcd6b8a49dc0854b2f9592d732b5afbd" >>>>>> +LIBVIRT_REVISION="origin/v1.3.3-maint" >>>>>> +OVMF_REVISION="52a99493cce88a9d4ec8a02d7f1bd1a1001ce60d" >>>>>> LINUX_REVISION="master" >>>>>> diff --git a/configs/config-master b/configs/config-master >>>>>> index bd26ce3..fce7436 100644 >>>>>> --- a/configs/config-master >>>>>> +++ b/configs/config-master >>>>>> @@ -6,3 +6,6 @@ GRUB_REVISION="master" >>>>>> LIBVIRT_REVISION="origin/xen-tested-master" >>>>>> OVMF_REVISION="origin/xen-tested-master" >>>>>> LINUX_REVISION="master" >>>>>> + >>>>>> +# oss-tested branch >>>>>> +LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" >>>>> >>>>> Why keep this around? I would remove it. >>>> >>>> Because... >>>> >>>>>> diff --git a/configs/config-url-git b/configs/config-url-git >>>>>> index 79813c4..614f522 100644 >>>>>> --- a/configs/config-url-git >>>>>> +++ b/configs/config-url-git >>>>>> @@ -3,6 +3,6 @@ QEMU_URL="git://xenbits.xen.org/qemu-xen.git" >>>>>> QEMU_TRADITIONAL_URL="git://xenbits.xen.org/qemu-xen-traditional.git" >>>>>> SEABIOS_URL="git://xenbits.xen.org/seabios.git" >>>>>> GRUB_URL="git://git.savannah.gnu.org/grub.git" >>>>>> -LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" >>>>>> +LIBVIRT_URL="git://libvirt.org/libvirt.git" >>>> >>>> ...here, the LIBVIRT_URL in config-url-git is set to the libvirt.org URL >>>> so that the stable releases can be pointed to v1.3.3-maint instead. The >>>> xen.org repo doesn't contain the v1.3.3-maint branch, and the >>>> libvirt.org repo doesn't contain the 'xen-tested-master' branch. >>>> >>>> I admit this is a bit ugly, *almost* separating things entirely but then >>>> not. But I don't see another easy way of having both the 1.3.3 >>>> maintenance branch and the oss-tested branch. (Unless we started having >>>> osstest test the maintenance branch as well.) >>> >>> I would drop git://xenbits.xen.org/libvirt.git and xen-tested-master >>> entirely and just stick to v1.3.3-maint everywhere. Or the other way >>> around. >> >> Well I assume that people building one of the releases want something >> relatively stable and supported, so xen-tested-master is obviously >> unsuitable (and might not actually be capable of building against Xen >> versions older than a certain point due to the fact that libxl cannot be >> both forward- and backward-compatible). And I also assume that people >> building with XEN_RELEASE="master" want the bleeding edge of everything >> so that they can test new features (or indeed, so that they can write >> new features). So I think we actually do want both -- a maintenance >> branch for releases, and xen-tested-master for developers and >> bleeding-edge users. > > Sure, we need to support both, but do they really need to be both > enabled in one of the example config files? We could just have the > libvirt xenbits tree as a comment. Well for that matter, why have a config-master at all? Just say how to make one in the comments. :-) I thought part of the point of raisin was to be able to get up and running with a reasonable template without having to read or change a lot of the comments. I don't really understand why anyone would want the development versions of Xen and qemu, but the stable maintenance release of libvirt. Doesn't it make more sense to have "config-master" get you the development versions of everything? -George _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH raisin 3/4] Update config-4.6 and config-4.5 to point to stable branches 2016-06-15 10:38 ` George Dunlap @ 2016-06-15 10:53 ` Stefano Stabellini 2016-06-15 11:13 ` George Dunlap 0 siblings, 1 reply; 29+ messages in thread From: Stefano Stabellini @ 2016-06-15 10:53 UTC (permalink / raw) To: George Dunlap; +Cc: xen-devel, Stefano Stabellini On Wed, 15 Jun 2016, George Dunlap wrote: > On 15/06/16 11:24, Stefano Stabellini wrote: > > On Tue, 14 Jun 2016, George Dunlap wrote: > >> On 14/06/16 11:01, Stefano Stabellini wrote: > >>> On Tue, 14 Jun 2016, George Dunlap wrote: > >>>> On 14/06/16 10:40, Stefano Stabellini wrote: > >>>>> On Mon, 13 Jun 2016, George Dunlap wrote: > >>>>>> Point xen, qemu, and qemu-trad to stable-4.5 and -4.6 branches. > >>>>>> > >>>>>> And point the default libvirt to point to the libvirt 1.3.3 > >>>>>> maintenance branch, rather than xen-tested-master. > >>>>>> > >>>>>> Also update OVMF revision for 4.6 to a version that builds with modern > >>>>>> gccs. > >>>>>> > >>>>>> Singed-off-by: George Dunlap <george.dunlap@citrix.com> > >>>>>> > >>>>>> CC: Stefano Stabellini <sstabellini@kernel.org> > >>>>>> --- > >>>>>> configs/config-4.5 | 6 +++--- > >>>>>> configs/config-4.6 | 8 ++++---- > >>>>>> configs/config-master | 3 +++ > >>>>>> configs/config-url-git | 2 +- > >>>>>> 4 files changed, 11 insertions(+), 8 deletions(-) > >>>>>> > >>>>>> diff --git a/configs/config-4.5 b/configs/config-4.5 > >>>>>> index 4163b68..8db9a9d 100644 > >>>>>> --- a/configs/config-4.5 > >>>>>> +++ b/configs/config-4.5 > >>>>>> @@ -1,8 +1,8 @@ > >>>>>> XEN_REVISION="origin/stable-4.5" > >>>>>> -QEMU_REVISION="master" > >>>>>> -QEMU_TRADITIONAL_REVISION="master" > >>>>>> +QEMU_REVISION="origin/stable-4.5" > >>>>>> +QEMU_TRADITIONAL_REVISION="origin/stable-4.5" > >>>>>> SEABIOS_REVISION="rel-1.7.5" > >>>>>> GRUB_REVISION="master" > >>>>>> -LIBVIRT_REVISION="origin/xen-tested-master" > >>>>>> +LIBVIRT_REVISION="origin/v1.3.3-maint" > >>>>>> OVMF_REVISION="cb9a7ebabcd6b8a49dc0854b2f9592d732b5afbd" > >>>>>> LINUX_REVISION="master" > >>>>>> diff --git a/configs/config-4.6 b/configs/config-4.6 > >>>>>> index e8b2a09..b003a30 100644 > >>>>>> --- a/configs/config-4.6 > >>>>>> +++ b/configs/config-4.6 > >>>>>> @@ -1,8 +1,8 @@ > >>>>>> XEN_REVISION="origin/stable-4.6" > >>>>>> -QEMU_REVISION="master" > >>>>>> -QEMU_TRADITIONAL_REVISION="master" > >>>>>> +QEMU_REVISION="origin/stable-4.6" > >>>>>> +QEMU_TRADITIONAL_REVISION="origin/stable-4.6" > >>>>>> SEABIOS_REVISION="rel-1.8.2" > >>>>>> GRUB_REVISION="master" > >>>>>> -LIBVIRT_REVISION="origin/xen-tested-master" > >>>>>> -OVMF_REVISION="cb9a7ebabcd6b8a49dc0854b2f9592d732b5afbd" > >>>>>> +LIBVIRT_REVISION="origin/v1.3.3-maint" > >>>>>> +OVMF_REVISION="52a99493cce88a9d4ec8a02d7f1bd1a1001ce60d" > >>>>>> LINUX_REVISION="master" > >>>>>> diff --git a/configs/config-master b/configs/config-master > >>>>>> index bd26ce3..fce7436 100644 > >>>>>> --- a/configs/config-master > >>>>>> +++ b/configs/config-master > >>>>>> @@ -6,3 +6,6 @@ GRUB_REVISION="master" > >>>>>> LIBVIRT_REVISION="origin/xen-tested-master" > >>>>>> OVMF_REVISION="origin/xen-tested-master" > >>>>>> LINUX_REVISION="master" > >>>>>> + > >>>>>> +# oss-tested branch > >>>>>> +LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" > >>>>> > >>>>> Why keep this around? I would remove it. > >>>> > >>>> Because... > >>>> > >>>>>> diff --git a/configs/config-url-git b/configs/config-url-git > >>>>>> index 79813c4..614f522 100644 > >>>>>> --- a/configs/config-url-git > >>>>>> +++ b/configs/config-url-git > >>>>>> @@ -3,6 +3,6 @@ QEMU_URL="git://xenbits.xen.org/qemu-xen.git" > >>>>>> QEMU_TRADITIONAL_URL="git://xenbits.xen.org/qemu-xen-traditional.git" > >>>>>> SEABIOS_URL="git://xenbits.xen.org/seabios.git" > >>>>>> GRUB_URL="git://git.savannah.gnu.org/grub.git" > >>>>>> -LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" > >>>>>> +LIBVIRT_URL="git://libvirt.org/libvirt.git" > >>>> > >>>> ...here, the LIBVIRT_URL in config-url-git is set to the libvirt.org URL > >>>> so that the stable releases can be pointed to v1.3.3-maint instead. The > >>>> xen.org repo doesn't contain the v1.3.3-maint branch, and the > >>>> libvirt.org repo doesn't contain the 'xen-tested-master' branch. > >>>> > >>>> I admit this is a bit ugly, *almost* separating things entirely but then > >>>> not. But I don't see another easy way of having both the 1.3.3 > >>>> maintenance branch and the oss-tested branch. (Unless we started having > >>>> osstest test the maintenance branch as well.) > >>> > >>> I would drop git://xenbits.xen.org/libvirt.git and xen-tested-master > >>> entirely and just stick to v1.3.3-maint everywhere. Or the other way > >>> around. > >> > >> Well I assume that people building one of the releases want something > >> relatively stable and supported, so xen-tested-master is obviously > >> unsuitable (and might not actually be capable of building against Xen > >> versions older than a certain point due to the fact that libxl cannot be > >> both forward- and backward-compatible). And I also assume that people > >> building with XEN_RELEASE="master" want the bleeding edge of everything > >> so that they can test new features (or indeed, so that they can write > >> new features). So I think we actually do want both -- a maintenance > >> branch for releases, and xen-tested-master for developers and > >> bleeding-edge users. > > > > Sure, we need to support both, but do they really need to be both > > enabled in one of the example config files? We could just have the > > libvirt xenbits tree as a comment. > > Well for that matter, why have a config-master at all? Just say how to > make one in the comments. :-) > > I thought part of the point of raisin was to be able to get up and > running with a reasonable template without having to read or change a > lot of the comments. I don't really understand why anyone would want > the development versions of Xen and qemu, but the stable maintenance > release of libvirt. Doesn't it make more sense to have "config-master" > get you the development versions of everything? The idea is to provide good templates and be very transparent on what get cloned and built. It's the transparency here that it is weakening: as a user I would prefer the libvirt tree not to change depending on the version I write in the main config file. As a user, I would be surprised if I saw this happening. That's why I was suggesting to keep to one libvirt tree. Or if you would like to have two, then make the choice obvious, not a consequence of a seemingly unrelated version setting. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH raisin 3/4] Update config-4.6 and config-4.5 to point to stable branches 2016-06-15 10:53 ` Stefano Stabellini @ 2016-06-15 11:13 ` George Dunlap 2016-06-15 14:02 ` Stefano Stabellini 0 siblings, 1 reply; 29+ messages in thread From: George Dunlap @ 2016-06-15 11:13 UTC (permalink / raw) To: Stefano Stabellini; +Cc: xen-devel On 15/06/16 11:53, Stefano Stabellini wrote: > On Wed, 15 Jun 2016, George Dunlap wrote: >> On 15/06/16 11:24, Stefano Stabellini wrote: >>> On Tue, 14 Jun 2016, George Dunlap wrote: >>>> On 14/06/16 11:01, Stefano Stabellini wrote: >>>>> On Tue, 14 Jun 2016, George Dunlap wrote: >>>>>> On 14/06/16 10:40, Stefano Stabellini wrote: >>>>>>> On Mon, 13 Jun 2016, George Dunlap wrote: >>>>>>>> Point xen, qemu, and qemu-trad to stable-4.5 and -4.6 branches. >>>>>>>> >>>>>>>> And point the default libvirt to point to the libvirt 1.3.3 >>>>>>>> maintenance branch, rather than xen-tested-master. >>>>>>>> >>>>>>>> Also update OVMF revision for 4.6 to a version that builds with modern >>>>>>>> gccs. >>>>>>>> >>>>>>>> Singed-off-by: George Dunlap <george.dunlap@citrix.com> >>>>>>>> >>>>>>>> CC: Stefano Stabellini <sstabellini@kernel.org> >>>>>>>> --- >>>>>>>> configs/config-4.5 | 6 +++--- >>>>>>>> configs/config-4.6 | 8 ++++---- >>>>>>>> configs/config-master | 3 +++ >>>>>>>> configs/config-url-git | 2 +- >>>>>>>> 4 files changed, 11 insertions(+), 8 deletions(-) >>>>>>>> >>>>>>>> diff --git a/configs/config-4.5 b/configs/config-4.5 >>>>>>>> index 4163b68..8db9a9d 100644 >>>>>>>> --- a/configs/config-4.5 >>>>>>>> +++ b/configs/config-4.5 >>>>>>>> @@ -1,8 +1,8 @@ >>>>>>>> XEN_REVISION="origin/stable-4.5" >>>>>>>> -QEMU_REVISION="master" >>>>>>>> -QEMU_TRADITIONAL_REVISION="master" >>>>>>>> +QEMU_REVISION="origin/stable-4.5" >>>>>>>> +QEMU_TRADITIONAL_REVISION="origin/stable-4.5" >>>>>>>> SEABIOS_REVISION="rel-1.7.5" >>>>>>>> GRUB_REVISION="master" >>>>>>>> -LIBVIRT_REVISION="origin/xen-tested-master" >>>>>>>> +LIBVIRT_REVISION="origin/v1.3.3-maint" >>>>>>>> OVMF_REVISION="cb9a7ebabcd6b8a49dc0854b2f9592d732b5afbd" >>>>>>>> LINUX_REVISION="master" >>>>>>>> diff --git a/configs/config-4.6 b/configs/config-4.6 >>>>>>>> index e8b2a09..b003a30 100644 >>>>>>>> --- a/configs/config-4.6 >>>>>>>> +++ b/configs/config-4.6 >>>>>>>> @@ -1,8 +1,8 @@ >>>>>>>> XEN_REVISION="origin/stable-4.6" >>>>>>>> -QEMU_REVISION="master" >>>>>>>> -QEMU_TRADITIONAL_REVISION="master" >>>>>>>> +QEMU_REVISION="origin/stable-4.6" >>>>>>>> +QEMU_TRADITIONAL_REVISION="origin/stable-4.6" >>>>>>>> SEABIOS_REVISION="rel-1.8.2" >>>>>>>> GRUB_REVISION="master" >>>>>>>> -LIBVIRT_REVISION="origin/xen-tested-master" >>>>>>>> -OVMF_REVISION="cb9a7ebabcd6b8a49dc0854b2f9592d732b5afbd" >>>>>>>> +LIBVIRT_REVISION="origin/v1.3.3-maint" >>>>>>>> +OVMF_REVISION="52a99493cce88a9d4ec8a02d7f1bd1a1001ce60d" >>>>>>>> LINUX_REVISION="master" >>>>>>>> diff --git a/configs/config-master b/configs/config-master >>>>>>>> index bd26ce3..fce7436 100644 >>>>>>>> --- a/configs/config-master >>>>>>>> +++ b/configs/config-master >>>>>>>> @@ -6,3 +6,6 @@ GRUB_REVISION="master" >>>>>>>> LIBVIRT_REVISION="origin/xen-tested-master" >>>>>>>> OVMF_REVISION="origin/xen-tested-master" >>>>>>>> LINUX_REVISION="master" >>>>>>>> + >>>>>>>> +# oss-tested branch >>>>>>>> +LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" >>>>>>> >>>>>>> Why keep this around? I would remove it. >>>>>> >>>>>> Because... >>>>>> >>>>>>>> diff --git a/configs/config-url-git b/configs/config-url-git >>>>>>>> index 79813c4..614f522 100644 >>>>>>>> --- a/configs/config-url-git >>>>>>>> +++ b/configs/config-url-git >>>>>>>> @@ -3,6 +3,6 @@ QEMU_URL="git://xenbits.xen.org/qemu-xen.git" >>>>>>>> QEMU_TRADITIONAL_URL="git://xenbits.xen.org/qemu-xen-traditional.git" >>>>>>>> SEABIOS_URL="git://xenbits.xen.org/seabios.git" >>>>>>>> GRUB_URL="git://git.savannah.gnu.org/grub.git" >>>>>>>> -LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" >>>>>>>> +LIBVIRT_URL="git://libvirt.org/libvirt.git" >>>>>> >>>>>> ...here, the LIBVIRT_URL in config-url-git is set to the libvirt.org URL >>>>>> so that the stable releases can be pointed to v1.3.3-maint instead. The >>>>>> xen.org repo doesn't contain the v1.3.3-maint branch, and the >>>>>> libvirt.org repo doesn't contain the 'xen-tested-master' branch. >>>>>> >>>>>> I admit this is a bit ugly, *almost* separating things entirely but then >>>>>> not. But I don't see another easy way of having both the 1.3.3 >>>>>> maintenance branch and the oss-tested branch. (Unless we started having >>>>>> osstest test the maintenance branch as well.) >>>>> >>>>> I would drop git://xenbits.xen.org/libvirt.git and xen-tested-master >>>>> entirely and just stick to v1.3.3-maint everywhere. Or the other way >>>>> around. >>>> >>>> Well I assume that people building one of the releases want something >>>> relatively stable and supported, so xen-tested-master is obviously >>>> unsuitable (and might not actually be capable of building against Xen >>>> versions older than a certain point due to the fact that libxl cannot be >>>> both forward- and backward-compatible). And I also assume that people >>>> building with XEN_RELEASE="master" want the bleeding edge of everything >>>> so that they can test new features (or indeed, so that they can write >>>> new features). So I think we actually do want both -- a maintenance >>>> branch for releases, and xen-tested-master for developers and >>>> bleeding-edge users. >>> >>> Sure, we need to support both, but do they really need to be both >>> enabled in one of the example config files? We could just have the >>> libvirt xenbits tree as a comment. >> >> Well for that matter, why have a config-master at all? Just say how to >> make one in the comments. :-) >> >> I thought part of the point of raisin was to be able to get up and >> running with a reasonable template without having to read or change a >> lot of the comments. I don't really understand why anyone would want >> the development versions of Xen and qemu, but the stable maintenance >> release of libvirt. Doesn't it make more sense to have "config-master" >> get you the development versions of everything? > > The idea is to provide good templates and be very transparent on what > get cloned and built. It's the transparency here that it is weakening: > as a user I would prefer the libvirt tree not to change depending on the > version I write in the main config file. As a user, I would be surprised > if I saw this happening. That's why I was suggesting to keep to one > libvirt tree. Or if you would like to have two, then make the choice > obvious, not a consequence of a seemingly unrelated version setting. By "libvirt tree", you mean the location of the libvirt repository? I guess we have fairly different ideas about what users care about and what they'd find unexpected. :-) I as a user wouldn't care a bit about URL I was downloading from, as long as I got the appropriate version. And I would be quite surprised to specify the "master" config, and get the bleeding edge of everything except libvirt. We could just have it point to libvirt-master I suppose. That risks having things in a broken state in the time between the inevitable regressions are introduced in libvirt-master and the time they get noticed and fixed. But that doesn't seem to be terribly often these days. What do you think? -George _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH raisin 3/4] Update config-4.6 and config-4.5 to point to stable branches 2016-06-15 11:13 ` George Dunlap @ 2016-06-15 14:02 ` Stefano Stabellini 2016-06-15 16:11 ` George Dunlap 0 siblings, 1 reply; 29+ messages in thread From: Stefano Stabellini @ 2016-06-15 14:02 UTC (permalink / raw) To: George Dunlap; +Cc: xen-devel, Stefano Stabellini On Wed, 15 Jun 2016, George Dunlap wrote: > On 15/06/16 11:53, Stefano Stabellini wrote: > > On Wed, 15 Jun 2016, George Dunlap wrote: > >> On 15/06/16 11:24, Stefano Stabellini wrote: > >>> On Tue, 14 Jun 2016, George Dunlap wrote: > >>>> On 14/06/16 11:01, Stefano Stabellini wrote: > >>>>> On Tue, 14 Jun 2016, George Dunlap wrote: > >>>>>> On 14/06/16 10:40, Stefano Stabellini wrote: > >>>>>>> On Mon, 13 Jun 2016, George Dunlap wrote: > >>>>>>>> Point xen, qemu, and qemu-trad to stable-4.5 and -4.6 branches. > >>>>>>>> > >>>>>>>> And point the default libvirt to point to the libvirt 1.3.3 > >>>>>>>> maintenance branch, rather than xen-tested-master. > >>>>>>>> > >>>>>>>> Also update OVMF revision for 4.6 to a version that builds with modern > >>>>>>>> gccs. > >>>>>>>> > >>>>>>>> Singed-off-by: George Dunlap <george.dunlap@citrix.com> > >>>>>>>> > >>>>>>>> CC: Stefano Stabellini <sstabellini@kernel.org> > >>>>>>>> --- > >>>>>>>> configs/config-4.5 | 6 +++--- > >>>>>>>> configs/config-4.6 | 8 ++++---- > >>>>>>>> configs/config-master | 3 +++ > >>>>>>>> configs/config-url-git | 2 +- > >>>>>>>> 4 files changed, 11 insertions(+), 8 deletions(-) > >>>>>>>> > >>>>>>>> diff --git a/configs/config-4.5 b/configs/config-4.5 > >>>>>>>> index 4163b68..8db9a9d 100644 > >>>>>>>> --- a/configs/config-4.5 > >>>>>>>> +++ b/configs/config-4.5 > >>>>>>>> @@ -1,8 +1,8 @@ > >>>>>>>> XEN_REVISION="origin/stable-4.5" > >>>>>>>> -QEMU_REVISION="master" > >>>>>>>> -QEMU_TRADITIONAL_REVISION="master" > >>>>>>>> +QEMU_REVISION="origin/stable-4.5" > >>>>>>>> +QEMU_TRADITIONAL_REVISION="origin/stable-4.5" > >>>>>>>> SEABIOS_REVISION="rel-1.7.5" > >>>>>>>> GRUB_REVISION="master" > >>>>>>>> -LIBVIRT_REVISION="origin/xen-tested-master" > >>>>>>>> +LIBVIRT_REVISION="origin/v1.3.3-maint" > >>>>>>>> OVMF_REVISION="cb9a7ebabcd6b8a49dc0854b2f9592d732b5afbd" > >>>>>>>> LINUX_REVISION="master" > >>>>>>>> diff --git a/configs/config-4.6 b/configs/config-4.6 > >>>>>>>> index e8b2a09..b003a30 100644 > >>>>>>>> --- a/configs/config-4.6 > >>>>>>>> +++ b/configs/config-4.6 > >>>>>>>> @@ -1,8 +1,8 @@ > >>>>>>>> XEN_REVISION="origin/stable-4.6" > >>>>>>>> -QEMU_REVISION="master" > >>>>>>>> -QEMU_TRADITIONAL_REVISION="master" > >>>>>>>> +QEMU_REVISION="origin/stable-4.6" > >>>>>>>> +QEMU_TRADITIONAL_REVISION="origin/stable-4.6" > >>>>>>>> SEABIOS_REVISION="rel-1.8.2" > >>>>>>>> GRUB_REVISION="master" > >>>>>>>> -LIBVIRT_REVISION="origin/xen-tested-master" > >>>>>>>> -OVMF_REVISION="cb9a7ebabcd6b8a49dc0854b2f9592d732b5afbd" > >>>>>>>> +LIBVIRT_REVISION="origin/v1.3.3-maint" > >>>>>>>> +OVMF_REVISION="52a99493cce88a9d4ec8a02d7f1bd1a1001ce60d" > >>>>>>>> LINUX_REVISION="master" > >>>>>>>> diff --git a/configs/config-master b/configs/config-master > >>>>>>>> index bd26ce3..fce7436 100644 > >>>>>>>> --- a/configs/config-master > >>>>>>>> +++ b/configs/config-master > >>>>>>>> @@ -6,3 +6,6 @@ GRUB_REVISION="master" > >>>>>>>> LIBVIRT_REVISION="origin/xen-tested-master" > >>>>>>>> OVMF_REVISION="origin/xen-tested-master" > >>>>>>>> LINUX_REVISION="master" > >>>>>>>> + > >>>>>>>> +# oss-tested branch > >>>>>>>> +LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" > >>>>>>> > >>>>>>> Why keep this around? I would remove it. > >>>>>> > >>>>>> Because... > >>>>>> > >>>>>>>> diff --git a/configs/config-url-git b/configs/config-url-git > >>>>>>>> index 79813c4..614f522 100644 > >>>>>>>> --- a/configs/config-url-git > >>>>>>>> +++ b/configs/config-url-git > >>>>>>>> @@ -3,6 +3,6 @@ QEMU_URL="git://xenbits.xen.org/qemu-xen.git" > >>>>>>>> QEMU_TRADITIONAL_URL="git://xenbits.xen.org/qemu-xen-traditional.git" > >>>>>>>> SEABIOS_URL="git://xenbits.xen.org/seabios.git" > >>>>>>>> GRUB_URL="git://git.savannah.gnu.org/grub.git" > >>>>>>>> -LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" > >>>>>>>> +LIBVIRT_URL="git://libvirt.org/libvirt.git" > >>>>>> > >>>>>> ...here, the LIBVIRT_URL in config-url-git is set to the libvirt.org URL > >>>>>> so that the stable releases can be pointed to v1.3.3-maint instead. The > >>>>>> xen.org repo doesn't contain the v1.3.3-maint branch, and the > >>>>>> libvirt.org repo doesn't contain the 'xen-tested-master' branch. > >>>>>> > >>>>>> I admit this is a bit ugly, *almost* separating things entirely but then > >>>>>> not. But I don't see another easy way of having both the 1.3.3 > >>>>>> maintenance branch and the oss-tested branch. (Unless we started having > >>>>>> osstest test the maintenance branch as well.) > >>>>> > >>>>> I would drop git://xenbits.xen.org/libvirt.git and xen-tested-master > >>>>> entirely and just stick to v1.3.3-maint everywhere. Or the other way > >>>>> around. > >>>> > >>>> Well I assume that people building one of the releases want something > >>>> relatively stable and supported, so xen-tested-master is obviously > >>>> unsuitable (and might not actually be capable of building against Xen > >>>> versions older than a certain point due to the fact that libxl cannot be > >>>> both forward- and backward-compatible). And I also assume that people > >>>> building with XEN_RELEASE="master" want the bleeding edge of everything > >>>> so that they can test new features (or indeed, so that they can write > >>>> new features). So I think we actually do want both -- a maintenance > >>>> branch for releases, and xen-tested-master for developers and > >>>> bleeding-edge users. > >>> > >>> Sure, we need to support both, but do they really need to be both > >>> enabled in one of the example config files? We could just have the > >>> libvirt xenbits tree as a comment. > >> > >> Well for that matter, why have a config-master at all? Just say how to > >> make one in the comments. :-) > >> > >> I thought part of the point of raisin was to be able to get up and > >> running with a reasonable template without having to read or change a > >> lot of the comments. I don't really understand why anyone would want > >> the development versions of Xen and qemu, but the stable maintenance > >> release of libvirt. Doesn't it make more sense to have "config-master" > >> get you the development versions of everything? > > > > The idea is to provide good templates and be very transparent on what > > get cloned and built. It's the transparency here that it is weakening: > > as a user I would prefer the libvirt tree not to change depending on the > > version I write in the main config file. As a user, I would be surprised > > if I saw this happening. That's why I was suggesting to keep to one > > libvirt tree. Or if you would like to have two, then make the choice > > obvious, not a consequence of a seemingly unrelated version setting. > > By "libvirt tree", you mean the location of the libvirt repository? > > I guess we have fairly different ideas about what users care about and > what they'd find unexpected. :-) I as a user wouldn't care a bit about > URL I was downloading from, as long as I got the appropriate version. > And I would be quite surprised to specify the "master" config, and get > the bleeding edge of everything except libvirt. > > We could just have it point to libvirt-master I suppose. That risks > having things in a broken state in the time between the inevitable > regressions are introduced in libvirt-master and the time they get > noticed and fixed. But that doesn't seem to be terribly often these > days. What do you think? libvirt master is good. After all master is for developers, so it makes sense. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH raisin 3/4] Update config-4.6 and config-4.5 to point to stable branches 2016-06-15 14:02 ` Stefano Stabellini @ 2016-06-15 16:11 ` George Dunlap 0 siblings, 0 replies; 29+ messages in thread From: George Dunlap @ 2016-06-15 16:11 UTC (permalink / raw) To: Stefano Stabellini; +Cc: xen-devel On 15/06/16 15:02, Stefano Stabellini wrote: > On Wed, 15 Jun 2016, George Dunlap wrote: >> On 15/06/16 11:53, Stefano Stabellini wrote: >>> On Wed, 15 Jun 2016, George Dunlap wrote: >>>> On 15/06/16 11:24, Stefano Stabellini wrote: >>>>> On Tue, 14 Jun 2016, George Dunlap wrote: >>>>>> On 14/06/16 11:01, Stefano Stabellini wrote: >>>>>>> On Tue, 14 Jun 2016, George Dunlap wrote: >>>>>>>> On 14/06/16 10:40, Stefano Stabellini wrote: >>>>>>>>> On Mon, 13 Jun 2016, George Dunlap wrote: >>>>>>>>>> Point xen, qemu, and qemu-trad to stable-4.5 and -4.6 branches. >>>>>>>>>> >>>>>>>>>> And point the default libvirt to point to the libvirt 1.3.3 >>>>>>>>>> maintenance branch, rather than xen-tested-master. >>>>>>>>>> >>>>>>>>>> Also update OVMF revision for 4.6 to a version that builds with modern >>>>>>>>>> gccs. >>>>>>>>>> >>>>>>>>>> Singed-off-by: George Dunlap <george.dunlap@citrix.com> >>>>>>>>>> >>>>>>>>>> CC: Stefano Stabellini <sstabellini@kernel.org> >>>>>>>>>> --- >>>>>>>>>> configs/config-4.5 | 6 +++--- >>>>>>>>>> configs/config-4.6 | 8 ++++---- >>>>>>>>>> configs/config-master | 3 +++ >>>>>>>>>> configs/config-url-git | 2 +- >>>>>>>>>> 4 files changed, 11 insertions(+), 8 deletions(-) >>>>>>>>>> >>>>>>>>>> diff --git a/configs/config-4.5 b/configs/config-4.5 >>>>>>>>>> index 4163b68..8db9a9d 100644 >>>>>>>>>> --- a/configs/config-4.5 >>>>>>>>>> +++ b/configs/config-4.5 >>>>>>>>>> @@ -1,8 +1,8 @@ >>>>>>>>>> XEN_REVISION="origin/stable-4.5" >>>>>>>>>> -QEMU_REVISION="master" >>>>>>>>>> -QEMU_TRADITIONAL_REVISION="master" >>>>>>>>>> +QEMU_REVISION="origin/stable-4.5" >>>>>>>>>> +QEMU_TRADITIONAL_REVISION="origin/stable-4.5" >>>>>>>>>> SEABIOS_REVISION="rel-1.7.5" >>>>>>>>>> GRUB_REVISION="master" >>>>>>>>>> -LIBVIRT_REVISION="origin/xen-tested-master" >>>>>>>>>> +LIBVIRT_REVISION="origin/v1.3.3-maint" >>>>>>>>>> OVMF_REVISION="cb9a7ebabcd6b8a49dc0854b2f9592d732b5afbd" >>>>>>>>>> LINUX_REVISION="master" >>>>>>>>>> diff --git a/configs/config-4.6 b/configs/config-4.6 >>>>>>>>>> index e8b2a09..b003a30 100644 >>>>>>>>>> --- a/configs/config-4.6 >>>>>>>>>> +++ b/configs/config-4.6 >>>>>>>>>> @@ -1,8 +1,8 @@ >>>>>>>>>> XEN_REVISION="origin/stable-4.6" >>>>>>>>>> -QEMU_REVISION="master" >>>>>>>>>> -QEMU_TRADITIONAL_REVISION="master" >>>>>>>>>> +QEMU_REVISION="origin/stable-4.6" >>>>>>>>>> +QEMU_TRADITIONAL_REVISION="origin/stable-4.6" >>>>>>>>>> SEABIOS_REVISION="rel-1.8.2" >>>>>>>>>> GRUB_REVISION="master" >>>>>>>>>> -LIBVIRT_REVISION="origin/xen-tested-master" >>>>>>>>>> -OVMF_REVISION="cb9a7ebabcd6b8a49dc0854b2f9592d732b5afbd" >>>>>>>>>> +LIBVIRT_REVISION="origin/v1.3.3-maint" >>>>>>>>>> +OVMF_REVISION="52a99493cce88a9d4ec8a02d7f1bd1a1001ce60d" >>>>>>>>>> LINUX_REVISION="master" >>>>>>>>>> diff --git a/configs/config-master b/configs/config-master >>>>>>>>>> index bd26ce3..fce7436 100644 >>>>>>>>>> --- a/configs/config-master >>>>>>>>>> +++ b/configs/config-master >>>>>>>>>> @@ -6,3 +6,6 @@ GRUB_REVISION="master" >>>>>>>>>> LIBVIRT_REVISION="origin/xen-tested-master" >>>>>>>>>> OVMF_REVISION="origin/xen-tested-master" >>>>>>>>>> LINUX_REVISION="master" >>>>>>>>>> + >>>>>>>>>> +# oss-tested branch >>>>>>>>>> +LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" >>>>>>>>> >>>>>>>>> Why keep this around? I would remove it. >>>>>>>> >>>>>>>> Because... >>>>>>>> >>>>>>>>>> diff --git a/configs/config-url-git b/configs/config-url-git >>>>>>>>>> index 79813c4..614f522 100644 >>>>>>>>>> --- a/configs/config-url-git >>>>>>>>>> +++ b/configs/config-url-git >>>>>>>>>> @@ -3,6 +3,6 @@ QEMU_URL="git://xenbits.xen.org/qemu-xen.git" >>>>>>>>>> QEMU_TRADITIONAL_URL="git://xenbits.xen.org/qemu-xen-traditional.git" >>>>>>>>>> SEABIOS_URL="git://xenbits.xen.org/seabios.git" >>>>>>>>>> GRUB_URL="git://git.savannah.gnu.org/grub.git" >>>>>>>>>> -LIBVIRT_URL="git://xenbits.xen.org/libvirt.git" >>>>>>>>>> +LIBVIRT_URL="git://libvirt.org/libvirt.git" >>>>>>>> >>>>>>>> ...here, the LIBVIRT_URL in config-url-git is set to the libvirt.org URL >>>>>>>> so that the stable releases can be pointed to v1.3.3-maint instead. The >>>>>>>> xen.org repo doesn't contain the v1.3.3-maint branch, and the >>>>>>>> libvirt.org repo doesn't contain the 'xen-tested-master' branch. >>>>>>>> >>>>>>>> I admit this is a bit ugly, *almost* separating things entirely but then >>>>>>>> not. But I don't see another easy way of having both the 1.3.3 >>>>>>>> maintenance branch and the oss-tested branch. (Unless we started having >>>>>>>> osstest test the maintenance branch as well.) >>>>>>> >>>>>>> I would drop git://xenbits.xen.org/libvirt.git and xen-tested-master >>>>>>> entirely and just stick to v1.3.3-maint everywhere. Or the other way >>>>>>> around. >>>>>> >>>>>> Well I assume that people building one of the releases want something >>>>>> relatively stable and supported, so xen-tested-master is obviously >>>>>> unsuitable (and might not actually be capable of building against Xen >>>>>> versions older than a certain point due to the fact that libxl cannot be >>>>>> both forward- and backward-compatible). And I also assume that people >>>>>> building with XEN_RELEASE="master" want the bleeding edge of everything >>>>>> so that they can test new features (or indeed, so that they can write >>>>>> new features). So I think we actually do want both -- a maintenance >>>>>> branch for releases, and xen-tested-master for developers and >>>>>> bleeding-edge users. >>>>> >>>>> Sure, we need to support both, but do they really need to be both >>>>> enabled in one of the example config files? We could just have the >>>>> libvirt xenbits tree as a comment. >>>> >>>> Well for that matter, why have a config-master at all? Just say how to >>>> make one in the comments. :-) >>>> >>>> I thought part of the point of raisin was to be able to get up and >>>> running with a reasonable template without having to read or change a >>>> lot of the comments. I don't really understand why anyone would want >>>> the development versions of Xen and qemu, but the stable maintenance >>>> release of libvirt. Doesn't it make more sense to have "config-master" >>>> get you the development versions of everything? >>> >>> The idea is to provide good templates and be very transparent on what >>> get cloned and built. It's the transparency here that it is weakening: >>> as a user I would prefer the libvirt tree not to change depending on the >>> version I write in the main config file. As a user, I would be surprised >>> if I saw this happening. That's why I was suggesting to keep to one >>> libvirt tree. Or if you would like to have two, then make the choice >>> obvious, not a consequence of a seemingly unrelated version setting. >> >> By "libvirt tree", you mean the location of the libvirt repository? >> >> I guess we have fairly different ideas about what users care about and >> what they'd find unexpected. :-) I as a user wouldn't care a bit about >> URL I was downloading from, as long as I got the appropriate version. >> And I would be quite surprised to specify the "master" config, and get >> the bleeding edge of everything except libvirt. >> >> We could just have it point to libvirt-master I suppose. That risks >> having things in a broken state in the time between the inevitable >> regressions are introduced in libvirt-master and the time they get >> noticed and fixed. But that doesn't seem to be terribly often these >> days. What do you think? > > libvirt master is good. After all master is for developers, so it makes > sense. OK, cool. While testing libvirt-master I discovered that config-master was broken anyway, because it was pointing to xen-tested-master for ovmf and seabios as well, which don't actually exist (!). New spin of the series on its way. -George _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 29+ messages in thread
* [PATCH raisin 4/4] Update to 4.7, update qemu and qemu_traditional recipes 2016-06-13 14:44 [PATCH raisin 0/4] Fixes for 4.6, update to 4.7 George Dunlap ` (2 preceding siblings ...) 2016-06-13 14:44 ` [PATCH raisin 3/4] Update config-4.6 and config-4.5 to point to stable branches George Dunlap @ 2016-06-13 14:44 ` George Dunlap 2016-06-14 9:46 ` Stefano Stabellini 3 siblings, 1 reply; 29+ messages in thread From: George Dunlap @ 2016-06-13 14:44 UTC (permalink / raw) To: xen-devel; +Cc: Stefano Stabellini, George Dunlap Add a 4.7 config file, make it the default. Also update the qemu and qemu_traditional recipies after Ian Cambell's work to split off separate libraries. Signed-off-by: George Dunlap <george.dunlap@citrix.com> --- CC: Stefano Stabellini <sstabellini@kernel.org> --- components/qemu | 21 +++++++++++++-------- components/qemu_traditional | 2 +- configs/config-4.7 | 8 ++++++++ defconfig | 2 +- 4 files changed, 23 insertions(+), 10 deletions(-) diff --git a/components/qemu b/components/qemu index e0d92a5..8624b50 100644 --- a/components/qemu +++ b/components/qemu @@ -23,15 +23,20 @@ function qemu_build() { cd "$BASEDIR" git-checkout $QEMU_URL $QEMU_REVISION qemu-dir cd qemu-dir - ./configure --enable-xen --target-list=i386-softmmu --prefix=$PREFIX \ - --extra-cflags="-I$INST_DIR/$PREFIX/include" \ - --extra-ldflags="-L$INST_DIR/$PREFIX/lib -Wl,-rpath-link=$INST_DIR/$PREFIX/lib \ + + ./configure --enable-xen --target-list=i386-softmmu \ + --prefix=$PREFIX \ + --extra-cflags="-DXC_WANT_COMPAT_EVTCHN_API=1 \ + -DXC_WANT_COMPAT_GNTTAB_API=1 \ + -DXC_WANT_COMPAT_MAP_FOREIGN_API=1 \ + -I$INST_DIR/$PREFIX/include" \ + --extra-ldflags="-L$INST_DIR/$PREFIX/lib -Wl,-rpath-link=$INST_DIR/$PREFIX/lib \ -L$INST_DIR/$PREFIX/lib64 -Wl,-rpath-link=$INST_DIR/$PREFIX/lib64" \ - --disable-kvm \ - --disable-docs \ - --bindir=$PREFIX/lib/xen/bin \ - --datadir=$PREFIX/share/qemu-xen \ - --disable-guest-agent + --bindir=$PREFIX/lib/xen/bin \ + --datadir=$PREFIX/share/qemu-xen \ + --disable-kvm \ + --disable-docs \ + --disable-guest-agent $RAISIN_MAKE all $RAISIN_MAKE install DESTDIR="$INST_DIR" cd "$BASEDIR" diff --git a/components/qemu_traditional b/components/qemu_traditional index 3150c3e..8732fe2 100644 --- a/components/qemu_traditional +++ b/components/qemu_traditional @@ -30,7 +30,7 @@ function qemu_traditional_build() { export CONFIG_BLKTAP1=n export XEN_ROOT="$BASEDIR"/xen-dir - ./xen-setup + ./xen-setup --extra-cflags="-D__XEN_TOOLS__" $RAISIN_MAKE all $RAISIN_MAKE install DESTDIR="$INST_DIR" cd "$BASEDIR" diff --git a/configs/config-4.7 b/configs/config-4.7 new file mode 100644 index 0000000..66fe467 --- /dev/null +++ b/configs/config-4.7 @@ -0,0 +1,8 @@ +XEN_REVISION="origin/stable-4.7" +QEMU_REVISION="origin/stable-4.7" +QEMU_TRADITIONAL_REVISION="origin/stable-4.7" +SEABIOS_REVISION="rel-1.9.2" +GRUB_REVISION="master" +LIBVIRT_REVISION="origin/v1.3.3-maint" +OVMF_REVISION="52a99493cce88a9d4ec8a02d7f1bd1a1001ce60d" +LINUX_REVISION="master" diff --git a/defconfig b/defconfig index 8e79a43..f8ef398 100644 --- a/defconfig +++ b/defconfig @@ -2,7 +2,7 @@ # Setup a Xen based system. # Available options: 4.5, 4.6, master (for development branch) -XEN_RELEASE="4.6" +XEN_RELEASE="4.7" # Components ## All components: seabios ovmf xen qemu qemu_traditional grub libvirt linux -- 2.1.4 _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply related [flat|nested] 29+ messages in thread
* Re: [PATCH raisin 4/4] Update to 4.7, update qemu and qemu_traditional recipes 2016-06-13 14:44 ` [PATCH raisin 4/4] Update to 4.7, update qemu and qemu_traditional recipes George Dunlap @ 2016-06-14 9:46 ` Stefano Stabellini 2016-06-14 10:03 ` George Dunlap 0 siblings, 1 reply; 29+ messages in thread From: Stefano Stabellini @ 2016-06-14 9:46 UTC (permalink / raw) To: George Dunlap; +Cc: xen-devel, Stefano Stabellini On Mon, 13 Jun 2016, George Dunlap wrote: > Add a 4.7 config file, make it the default. > > Also update the qemu and qemu_traditional recipies after Ian Cambell's > work to split off separate libraries. > > Signed-off-by: George Dunlap <george.dunlap@citrix.com> > --- > CC: Stefano Stabellini <sstabellini@kernel.org> > --- > components/qemu | 21 +++++++++++++-------- > components/qemu_traditional | 2 +- > configs/config-4.7 | 8 ++++++++ > defconfig | 2 +- > 4 files changed, 23 insertions(+), 10 deletions(-) > > diff --git a/components/qemu b/components/qemu > index e0d92a5..8624b50 100644 > --- a/components/qemu > +++ b/components/qemu > @@ -23,15 +23,20 @@ function qemu_build() { > cd "$BASEDIR" > git-checkout $QEMU_URL $QEMU_REVISION qemu-dir > cd qemu-dir > - ./configure --enable-xen --target-list=i386-softmmu --prefix=$PREFIX \ > - --extra-cflags="-I$INST_DIR/$PREFIX/include" \ > - --extra-ldflags="-L$INST_DIR/$PREFIX/lib -Wl,-rpath-link=$INST_DIR/$PREFIX/lib \ > + > + ./configure --enable-xen --target-list=i386-softmmu \ > + --prefix=$PREFIX \ > + --extra-cflags="-DXC_WANT_COMPAT_EVTCHN_API=1 \ > + -DXC_WANT_COMPAT_GNTTAB_API=1 \ > + -DXC_WANT_COMPAT_MAP_FOREIGN_API=1 \ > + -I$INST_DIR/$PREFIX/include" \ > + --extra-ldflags="-L$INST_DIR/$PREFIX/lib -Wl,-rpath-link=$INST_DIR/$PREFIX/lib \ > -L$INST_DIR/$PREFIX/lib64 -Wl,-rpath-link=$INST_DIR/$PREFIX/lib64" \ > - --disable-kvm \ > - --disable-docs \ > - --bindir=$PREFIX/lib/xen/bin \ > - --datadir=$PREFIX/share/qemu-xen \ > - --disable-guest-agent > + --bindir=$PREFIX/lib/xen/bin \ > + --datadir=$PREFIX/share/qemu-xen \ > + --disable-kvm \ > + --disable-docs \ > + --disable-guest-agent This is adding XC_WANT_COMPAT_EVTCHN_API=1, etc, unconditionally. If we make this change, will raisin still be able to buil an older qemu against an older xen? > $RAISIN_MAKE all > $RAISIN_MAKE install DESTDIR="$INST_DIR" > cd "$BASEDIR" > diff --git a/components/qemu_traditional b/components/qemu_traditional > index 3150c3e..8732fe2 100644 > --- a/components/qemu_traditional > +++ b/components/qemu_traditional > @@ -30,7 +30,7 @@ function qemu_traditional_build() { > > export CONFIG_BLKTAP1=n > export XEN_ROOT="$BASEDIR"/xen-dir > - ./xen-setup > + ./xen-setup --extra-cflags="-D__XEN_TOOLS__" > $RAISIN_MAKE all > $RAISIN_MAKE install DESTDIR="$INST_DIR" > cd "$BASEDIR" > diff --git a/configs/config-4.7 b/configs/config-4.7 > new file mode 100644 > index 0000000..66fe467 > --- /dev/null > +++ b/configs/config-4.7 > @@ -0,0 +1,8 @@ > +XEN_REVISION="origin/stable-4.7" > +QEMU_REVISION="origin/stable-4.7" > +QEMU_TRADITIONAL_REVISION="origin/stable-4.7" > +SEABIOS_REVISION="rel-1.9.2" > +GRUB_REVISION="master" > +LIBVIRT_REVISION="origin/v1.3.3-maint" > +OVMF_REVISION="52a99493cce88a9d4ec8a02d7f1bd1a1001ce60d" > +LINUX_REVISION="master" > diff --git a/defconfig b/defconfig > index 8e79a43..f8ef398 100644 > --- a/defconfig > +++ b/defconfig > @@ -2,7 +2,7 @@ > # Setup a Xen based system. > > # Available options: 4.5, 4.6, master (for development branch) > -XEN_RELEASE="4.6" > +XEN_RELEASE="4.7" > > # Components > ## All components: seabios ovmf xen qemu qemu_traditional grub libvirt linux _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH raisin 4/4] Update to 4.7, update qemu and qemu_traditional recipes 2016-06-14 9:46 ` Stefano Stabellini @ 2016-06-14 10:03 ` George Dunlap 2016-06-14 10:08 ` Stefano Stabellini 0 siblings, 1 reply; 29+ messages in thread From: George Dunlap @ 2016-06-14 10:03 UTC (permalink / raw) To: Stefano Stabellini; +Cc: xen-devel On 14/06/16 10:46, Stefano Stabellini wrote: > On Mon, 13 Jun 2016, George Dunlap wrote: >> Add a 4.7 config file, make it the default. >> >> Also update the qemu and qemu_traditional recipies after Ian Cambell's >> work to split off separate libraries. >> >> Signed-off-by: George Dunlap <george.dunlap@citrix.com> >> --- >> CC: Stefano Stabellini <sstabellini@kernel.org> >> --- >> components/qemu | 21 +++++++++++++-------- >> components/qemu_traditional | 2 +- >> configs/config-4.7 | 8 ++++++++ >> defconfig | 2 +- >> 4 files changed, 23 insertions(+), 10 deletions(-) >> >> diff --git a/components/qemu b/components/qemu >> index e0d92a5..8624b50 100644 >> --- a/components/qemu >> +++ b/components/qemu >> @@ -23,15 +23,20 @@ function qemu_build() { >> cd "$BASEDIR" >> git-checkout $QEMU_URL $QEMU_REVISION qemu-dir >> cd qemu-dir >> - ./configure --enable-xen --target-list=i386-softmmu --prefix=$PREFIX \ >> - --extra-cflags="-I$INST_DIR/$PREFIX/include" \ >> - --extra-ldflags="-L$INST_DIR/$PREFIX/lib -Wl,-rpath-link=$INST_DIR/$PREFIX/lib \ >> + >> + ./configure --enable-xen --target-list=i386-softmmu \ >> + --prefix=$PREFIX \ >> + --extra-cflags="-DXC_WANT_COMPAT_EVTCHN_API=1 \ >> + -DXC_WANT_COMPAT_GNTTAB_API=1 \ >> + -DXC_WANT_COMPAT_MAP_FOREIGN_API=1 \ >> + -I$INST_DIR/$PREFIX/include" \ >> + --extra-ldflags="-L$INST_DIR/$PREFIX/lib -Wl,-rpath-link=$INST_DIR/$PREFIX/lib \ >> -L$INST_DIR/$PREFIX/lib64 -Wl,-rpath-link=$INST_DIR/$PREFIX/lib64" \ >> - --disable-kvm \ >> - --disable-docs \ >> - --bindir=$PREFIX/lib/xen/bin \ >> - --datadir=$PREFIX/share/qemu-xen \ >> - --disable-guest-agent >> + --bindir=$PREFIX/lib/xen/bin \ >> + --datadir=$PREFIX/share/qemu-xen \ >> + --disable-kvm \ >> + --disable-docs \ >> + --disable-guest-agent > > This is adding XC_WANT_COMPAT_EVTCHN_API=1, etc, unconditionally. If we > make this change, will raisin still be able to buil an older qemu > against an older xen? I've tested this with XEN_RELEASE set to 4.5 and 4.6, and it builds just fine. The #define is just ignored, since neither Xen nor qemu in the 4.5 and 4.6 branches know anything about it. If part of our goal is to be a repository of the canonical way to build things, though, perhaps it would be better to have the extra flags be conditional on the value of XEN_RELEASE. -George _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH raisin 4/4] Update to 4.7, update qemu and qemu_traditional recipes 2016-06-14 10:03 ` George Dunlap @ 2016-06-14 10:08 ` Stefano Stabellini 2016-06-14 10:17 ` George Dunlap 0 siblings, 1 reply; 29+ messages in thread From: Stefano Stabellini @ 2016-06-14 10:08 UTC (permalink / raw) To: George Dunlap; +Cc: xen-devel, Stefano Stabellini On Tue, 14 Jun 2016, George Dunlap wrote: > On 14/06/16 10:46, Stefano Stabellini wrote: > > On Mon, 13 Jun 2016, George Dunlap wrote: > >> Add a 4.7 config file, make it the default. > >> > >> Also update the qemu and qemu_traditional recipies after Ian Cambell's > >> work to split off separate libraries. > >> > >> Signed-off-by: George Dunlap <george.dunlap@citrix.com> > >> --- > >> CC: Stefano Stabellini <sstabellini@kernel.org> > >> --- > >> components/qemu | 21 +++++++++++++-------- > >> components/qemu_traditional | 2 +- > >> configs/config-4.7 | 8 ++++++++ > >> defconfig | 2 +- > >> 4 files changed, 23 insertions(+), 10 deletions(-) > >> > >> diff --git a/components/qemu b/components/qemu > >> index e0d92a5..8624b50 100644 > >> --- a/components/qemu > >> +++ b/components/qemu > >> @@ -23,15 +23,20 @@ function qemu_build() { > >> cd "$BASEDIR" > >> git-checkout $QEMU_URL $QEMU_REVISION qemu-dir > >> cd qemu-dir > >> - ./configure --enable-xen --target-list=i386-softmmu --prefix=$PREFIX \ > >> - --extra-cflags="-I$INST_DIR/$PREFIX/include" \ > >> - --extra-ldflags="-L$INST_DIR/$PREFIX/lib -Wl,-rpath-link=$INST_DIR/$PREFIX/lib \ > >> + > >> + ./configure --enable-xen --target-list=i386-softmmu \ > >> + --prefix=$PREFIX \ > >> + --extra-cflags="-DXC_WANT_COMPAT_EVTCHN_API=1 \ > >> + -DXC_WANT_COMPAT_GNTTAB_API=1 \ > >> + -DXC_WANT_COMPAT_MAP_FOREIGN_API=1 \ > >> + -I$INST_DIR/$PREFIX/include" \ > >> + --extra-ldflags="-L$INST_DIR/$PREFIX/lib -Wl,-rpath-link=$INST_DIR/$PREFIX/lib \ > >> -L$INST_DIR/$PREFIX/lib64 -Wl,-rpath-link=$INST_DIR/$PREFIX/lib64" \ > >> - --disable-kvm \ > >> - --disable-docs \ > >> - --bindir=$PREFIX/lib/xen/bin \ > >> - --datadir=$PREFIX/share/qemu-xen \ > >> - --disable-guest-agent > >> + --bindir=$PREFIX/lib/xen/bin \ > >> + --datadir=$PREFIX/share/qemu-xen \ > >> + --disable-kvm \ > >> + --disable-docs \ > >> + --disable-guest-agent > > > > This is adding XC_WANT_COMPAT_EVTCHN_API=1, etc, unconditionally. If we > > make this change, will raisin still be able to buil an older qemu > > against an older xen? > > I've tested this with XEN_RELEASE set to 4.5 and 4.6, and it builds just > fine. The #define is just ignored, since neither Xen nor qemu in the > 4.5 and 4.6 branches know anything about it. > > If part of our goal is to be a repository of the canonical way to build > things, though, perhaps it would be better to have the extra flags be > conditional on the value of XEN_RELEASE. If we are sure that it's ignored than it might be benign. Maybe we should just add an in-code comment to explain why we added it though. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH raisin 4/4] Update to 4.7, update qemu and qemu_traditional recipes 2016-06-14 10:08 ` Stefano Stabellini @ 2016-06-14 10:17 ` George Dunlap 2016-06-14 10:31 ` Stefano Stabellini 0 siblings, 1 reply; 29+ messages in thread From: George Dunlap @ 2016-06-14 10:17 UTC (permalink / raw) To: Stefano Stabellini; +Cc: xen-devel On 14/06/16 11:08, Stefano Stabellini wrote: > On Tue, 14 Jun 2016, George Dunlap wrote: >> On 14/06/16 10:46, Stefano Stabellini wrote: >>> On Mon, 13 Jun 2016, George Dunlap wrote: >>>> Add a 4.7 config file, make it the default. >>>> >>>> Also update the qemu and qemu_traditional recipies after Ian Cambell's >>>> work to split off separate libraries. >>>> >>>> Signed-off-by: George Dunlap <george.dunlap@citrix.com> >>>> --- >>>> CC: Stefano Stabellini <sstabellini@kernel.org> >>>> --- >>>> components/qemu | 21 +++++++++++++-------- >>>> components/qemu_traditional | 2 +- >>>> configs/config-4.7 | 8 ++++++++ >>>> defconfig | 2 +- >>>> 4 files changed, 23 insertions(+), 10 deletions(-) >>>> >>>> diff --git a/components/qemu b/components/qemu >>>> index e0d92a5..8624b50 100644 >>>> --- a/components/qemu >>>> +++ b/components/qemu >>>> @@ -23,15 +23,20 @@ function qemu_build() { >>>> cd "$BASEDIR" >>>> git-checkout $QEMU_URL $QEMU_REVISION qemu-dir >>>> cd qemu-dir >>>> - ./configure --enable-xen --target-list=i386-softmmu --prefix=$PREFIX \ >>>> - --extra-cflags="-I$INST_DIR/$PREFIX/include" \ >>>> - --extra-ldflags="-L$INST_DIR/$PREFIX/lib -Wl,-rpath-link=$INST_DIR/$PREFIX/lib \ >>>> + >>>> + ./configure --enable-xen --target-list=i386-softmmu \ >>>> + --prefix=$PREFIX \ >>>> + --extra-cflags="-DXC_WANT_COMPAT_EVTCHN_API=1 \ >>>> + -DXC_WANT_COMPAT_GNTTAB_API=1 \ >>>> + -DXC_WANT_COMPAT_MAP_FOREIGN_API=1 \ >>>> + -I$INST_DIR/$PREFIX/include" \ >>>> + --extra-ldflags="-L$INST_DIR/$PREFIX/lib -Wl,-rpath-link=$INST_DIR/$PREFIX/lib \ >>>> -L$INST_DIR/$PREFIX/lib64 -Wl,-rpath-link=$INST_DIR/$PREFIX/lib64" \ >>>> - --disable-kvm \ >>>> - --disable-docs \ >>>> - --bindir=$PREFIX/lib/xen/bin \ >>>> - --datadir=$PREFIX/share/qemu-xen \ >>>> - --disable-guest-agent >>>> + --bindir=$PREFIX/lib/xen/bin \ >>>> + --datadir=$PREFIX/share/qemu-xen \ >>>> + --disable-kvm \ >>>> + --disable-docs \ >>>> + --disable-guest-agent >>> >>> This is adding XC_WANT_COMPAT_EVTCHN_API=1, etc, unconditionally. If we >>> make this change, will raisin still be able to buil an older qemu >>> against an older xen? >> >> I've tested this with XEN_RELEASE set to 4.5 and 4.6, and it builds just >> fine. The #define is just ignored, since neither Xen nor qemu in the >> 4.5 and 4.6 branches know anything about it. >> >> If part of our goal is to be a repository of the canonical way to build >> things, though, perhaps it would be better to have the extra flags be >> conditional on the value of XEN_RELEASE. > > If we are sure that it's ignored than it might be benign. Maybe we > should just add an in-code comment to explain why we added it though. Well really, I think that the detection should be put into the qemu configure scripts -- i.e., it should detect that there's the new API and add the appropriate #defines itself. But that's the sort of change I don't think there will be a lot of enthusiasm for. :-) I'll add a comment. -George _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH raisin 4/4] Update to 4.7, update qemu and qemu_traditional recipes 2016-06-14 10:17 ` George Dunlap @ 2016-06-14 10:31 ` Stefano Stabellini 2016-06-14 10:34 ` George Dunlap 0 siblings, 1 reply; 29+ messages in thread From: Stefano Stabellini @ 2016-06-14 10:31 UTC (permalink / raw) To: George Dunlap; +Cc: xen-devel, Stefano Stabellini On Tue, 14 Jun 2016, George Dunlap wrote: > On 14/06/16 11:08, Stefano Stabellini wrote: > > On Tue, 14 Jun 2016, George Dunlap wrote: > >> On 14/06/16 10:46, Stefano Stabellini wrote: > >>> On Mon, 13 Jun 2016, George Dunlap wrote: > >>>> Add a 4.7 config file, make it the default. > >>>> > >>>> Also update the qemu and qemu_traditional recipies after Ian Cambell's > >>>> work to split off separate libraries. > >>>> > >>>> Signed-off-by: George Dunlap <george.dunlap@citrix.com> > >>>> --- > >>>> CC: Stefano Stabellini <sstabellini@kernel.org> > >>>> --- > >>>> components/qemu | 21 +++++++++++++-------- > >>>> components/qemu_traditional | 2 +- > >>>> configs/config-4.7 | 8 ++++++++ > >>>> defconfig | 2 +- > >>>> 4 files changed, 23 insertions(+), 10 deletions(-) > >>>> > >>>> diff --git a/components/qemu b/components/qemu > >>>> index e0d92a5..8624b50 100644 > >>>> --- a/components/qemu > >>>> +++ b/components/qemu > >>>> @@ -23,15 +23,20 @@ function qemu_build() { > >>>> cd "$BASEDIR" > >>>> git-checkout $QEMU_URL $QEMU_REVISION qemu-dir > >>>> cd qemu-dir > >>>> - ./configure --enable-xen --target-list=i386-softmmu --prefix=$PREFIX \ > >>>> - --extra-cflags="-I$INST_DIR/$PREFIX/include" \ > >>>> - --extra-ldflags="-L$INST_DIR/$PREFIX/lib -Wl,-rpath-link=$INST_DIR/$PREFIX/lib \ > >>>> + > >>>> + ./configure --enable-xen --target-list=i386-softmmu \ > >>>> + --prefix=$PREFIX \ > >>>> + --extra-cflags="-DXC_WANT_COMPAT_EVTCHN_API=1 \ > >>>> + -DXC_WANT_COMPAT_GNTTAB_API=1 \ > >>>> + -DXC_WANT_COMPAT_MAP_FOREIGN_API=1 \ > >>>> + -I$INST_DIR/$PREFIX/include" \ > >>>> + --extra-ldflags="-L$INST_DIR/$PREFIX/lib -Wl,-rpath-link=$INST_DIR/$PREFIX/lib \ > >>>> -L$INST_DIR/$PREFIX/lib64 -Wl,-rpath-link=$INST_DIR/$PREFIX/lib64" \ > >>>> - --disable-kvm \ > >>>> - --disable-docs \ > >>>> - --bindir=$PREFIX/lib/xen/bin \ > >>>> - --datadir=$PREFIX/share/qemu-xen \ > >>>> - --disable-guest-agent > >>>> + --bindir=$PREFIX/lib/xen/bin \ > >>>> + --datadir=$PREFIX/share/qemu-xen \ > >>>> + --disable-kvm \ > >>>> + --disable-docs \ > >>>> + --disable-guest-agent > >>> > >>> This is adding XC_WANT_COMPAT_EVTCHN_API=1, etc, unconditionally. If we > >>> make this change, will raisin still be able to buil an older qemu > >>> against an older xen? > >> > >> I've tested this with XEN_RELEASE set to 4.5 and 4.6, and it builds just > >> fine. The #define is just ignored, since neither Xen nor qemu in the > >> 4.5 and 4.6 branches know anything about it. > >> > >> If part of our goal is to be a repository of the canonical way to build > >> things, though, perhaps it would be better to have the extra flags be > >> conditional on the value of XEN_RELEASE. > > > > If we are sure that it's ignored than it might be benign. Maybe we > > should just add an in-code comment to explain why we added it though. > > Well really, I think that the detection should be put into the qemu > configure scripts -- i.e., it should detect that there's the new API and > add the appropriate #defines itself. But that's the sort of change I > don't think there will be a lot of enthusiasm for. :-) Actually there is already code in QEMU configure script for the detection. In fact I was wondering if explicitly passing XC_WANT_* is actually necessary. I don't think it is from QEMU POV. > I'll add a comment. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH raisin 4/4] Update to 4.7, update qemu and qemu_traditional recipes 2016-06-14 10:31 ` Stefano Stabellini @ 2016-06-14 10:34 ` George Dunlap 2016-06-14 13:53 ` Anthony PERARD 0 siblings, 1 reply; 29+ messages in thread From: George Dunlap @ 2016-06-14 10:34 UTC (permalink / raw) To: Stefano Stabellini; +Cc: Anthony Perard, xen-devel On 14/06/16 11:31, Stefano Stabellini wrote: > On Tue, 14 Jun 2016, George Dunlap wrote: >> On 14/06/16 11:08, Stefano Stabellini wrote: >>> On Tue, 14 Jun 2016, George Dunlap wrote: >>>> On 14/06/16 10:46, Stefano Stabellini wrote: >>>>> On Mon, 13 Jun 2016, George Dunlap wrote: >>>>>> Add a 4.7 config file, make it the default. >>>>>> >>>>>> Also update the qemu and qemu_traditional recipies after Ian Cambell's >>>>>> work to split off separate libraries. >>>>>> >>>>>> Signed-off-by: George Dunlap <george.dunlap@citrix.com> >>>>>> --- >>>>>> CC: Stefano Stabellini <sstabellini@kernel.org> >>>>>> --- >>>>>> components/qemu | 21 +++++++++++++-------- >>>>>> components/qemu_traditional | 2 +- >>>>>> configs/config-4.7 | 8 ++++++++ >>>>>> defconfig | 2 +- >>>>>> 4 files changed, 23 insertions(+), 10 deletions(-) >>>>>> >>>>>> diff --git a/components/qemu b/components/qemu >>>>>> index e0d92a5..8624b50 100644 >>>>>> --- a/components/qemu >>>>>> +++ b/components/qemu >>>>>> @@ -23,15 +23,20 @@ function qemu_build() { >>>>>> cd "$BASEDIR" >>>>>> git-checkout $QEMU_URL $QEMU_REVISION qemu-dir >>>>>> cd qemu-dir >>>>>> - ./configure --enable-xen --target-list=i386-softmmu --prefix=$PREFIX \ >>>>>> - --extra-cflags="-I$INST_DIR/$PREFIX/include" \ >>>>>> - --extra-ldflags="-L$INST_DIR/$PREFIX/lib -Wl,-rpath-link=$INST_DIR/$PREFIX/lib \ >>>>>> + >>>>>> + ./configure --enable-xen --target-list=i386-softmmu \ >>>>>> + --prefix=$PREFIX \ >>>>>> + --extra-cflags="-DXC_WANT_COMPAT_EVTCHN_API=1 \ >>>>>> + -DXC_WANT_COMPAT_GNTTAB_API=1 \ >>>>>> + -DXC_WANT_COMPAT_MAP_FOREIGN_API=1 \ >>>>>> + -I$INST_DIR/$PREFIX/include" \ >>>>>> + --extra-ldflags="-L$INST_DIR/$PREFIX/lib -Wl,-rpath-link=$INST_DIR/$PREFIX/lib \ >>>>>> -L$INST_DIR/$PREFIX/lib64 -Wl,-rpath-link=$INST_DIR/$PREFIX/lib64" \ >>>>>> - --disable-kvm \ >>>>>> - --disable-docs \ >>>>>> - --bindir=$PREFIX/lib/xen/bin \ >>>>>> - --datadir=$PREFIX/share/qemu-xen \ >>>>>> - --disable-guest-agent >>>>>> + --bindir=$PREFIX/lib/xen/bin \ >>>>>> + --datadir=$PREFIX/share/qemu-xen \ >>>>>> + --disable-kvm \ >>>>>> + --disable-docs \ >>>>>> + --disable-guest-agent >>>>> >>>>> This is adding XC_WANT_COMPAT_EVTCHN_API=1, etc, unconditionally. If we >>>>> make this change, will raisin still be able to buil an older qemu >>>>> against an older xen? >>>> >>>> I've tested this with XEN_RELEASE set to 4.5 and 4.6, and it builds just >>>> fine. The #define is just ignored, since neither Xen nor qemu in the >>>> 4.5 and 4.6 branches know anything about it. >>>> >>>> If part of our goal is to be a repository of the canonical way to build >>>> things, though, perhaps it would be better to have the extra flags be >>>> conditional on the value of XEN_RELEASE. >>> >>> If we are sure that it's ignored than it might be benign. Maybe we >>> should just add an in-code comment to explain why we added it though. >> >> Well really, I think that the detection should be put into the qemu >> configure scripts -- i.e., it should detect that there's the new API and >> add the appropriate #defines itself. But that's the sort of change I >> don't think there will be a lot of enthusiasm for. :-) > > Actually there is already code in QEMU configure script for the > detection. In fact I was wondering if explicitly passing XC_WANT_* is > actually necessary. I don't think it is from QEMU POV. That code might be in qemu-upstream, but it's not in the qemu-xen trees; if you revert this bit of the patch and try to build with raisin it will fail. That's why I even bothered to add these in in the first place; and why IanC added these runes to tools/Makefile. Anthony, can you comment more authoritatively here on what's going on? Would it be possible for 4.7.1 to backport the configure auto-detection stuff? -George _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH raisin 4/4] Update to 4.7, update qemu and qemu_traditional recipes 2016-06-14 10:34 ` George Dunlap @ 2016-06-14 13:53 ` Anthony PERARD 2016-06-14 14:00 ` George Dunlap 2016-06-15 10:39 ` Stefano Stabellini 0 siblings, 2 replies; 29+ messages in thread From: Anthony PERARD @ 2016-06-14 13:53 UTC (permalink / raw) To: George Dunlap; +Cc: xen-devel, Stefano Stabellini On Tue, Jun 14, 2016 at 11:34:43AM +0100, George Dunlap wrote: > On 14/06/16 11:31, Stefano Stabellini wrote: > > On Tue, 14 Jun 2016, George Dunlap wrote: > >> On 14/06/16 11:08, Stefano Stabellini wrote: > >>> On Tue, 14 Jun 2016, George Dunlap wrote: > >>>> On 14/06/16 10:46, Stefano Stabellini wrote: > >>>>> On Mon, 13 Jun 2016, George Dunlap wrote: > >>>>>> Add a 4.7 config file, make it the default. > >>>>>> > >>>>>> Also update the qemu and qemu_traditional recipies after Ian Cambell's > >>>>>> work to split off separate libraries. > >>>>>> > >>>>>> Signed-off-by: George Dunlap <george.dunlap@citrix.com> > >>>>>> --- > >>>>>> CC: Stefano Stabellini <sstabellini@kernel.org> > >>>>>> --- > >>>>>> components/qemu | 21 +++++++++++++-------- > >>>>>> components/qemu_traditional | 2 +- > >>>>>> configs/config-4.7 | 8 ++++++++ > >>>>>> defconfig | 2 +- > >>>>>> 4 files changed, 23 insertions(+), 10 deletions(-) > >>>>>> > >>>>>> diff --git a/components/qemu b/components/qemu > >>>>>> index e0d92a5..8624b50 100644 > >>>>>> --- a/components/qemu > >>>>>> +++ b/components/qemu > >>>>>> @@ -23,15 +23,20 @@ function qemu_build() { > >>>>>> cd "$BASEDIR" > >>>>>> git-checkout $QEMU_URL $QEMU_REVISION qemu-dir > >>>>>> cd qemu-dir > >>>>>> - ./configure --enable-xen --target-list=i386-softmmu --prefix=$PREFIX \ > >>>>>> - --extra-cflags="-I$INST_DIR/$PREFIX/include" \ > >>>>>> - --extra-ldflags="-L$INST_DIR/$PREFIX/lib -Wl,-rpath-link=$INST_DIR/$PREFIX/lib \ > >>>>>> + > >>>>>> + ./configure --enable-xen --target-list=i386-softmmu \ > >>>>>> + --prefix=$PREFIX \ > >>>>>> + --extra-cflags="-DXC_WANT_COMPAT_EVTCHN_API=1 \ > >>>>>> + -DXC_WANT_COMPAT_GNTTAB_API=1 \ > >>>>>> + -DXC_WANT_COMPAT_MAP_FOREIGN_API=1 \ > >>>>>> + -I$INST_DIR/$PREFIX/include" \ > >>>>>> + --extra-ldflags="-L$INST_DIR/$PREFIX/lib -Wl,-rpath-link=$INST_DIR/$PREFIX/lib \ > >>>>>> -L$INST_DIR/$PREFIX/lib64 -Wl,-rpath-link=$INST_DIR/$PREFIX/lib64" \ > >>>>>> - --disable-kvm \ > >>>>>> - --disable-docs \ > >>>>>> - --bindir=$PREFIX/lib/xen/bin \ > >>>>>> - --datadir=$PREFIX/share/qemu-xen \ > >>>>>> - --disable-guest-agent > >>>>>> + --bindir=$PREFIX/lib/xen/bin \ > >>>>>> + --datadir=$PREFIX/share/qemu-xen \ > >>>>>> + --disable-kvm \ > >>>>>> + --disable-docs \ > >>>>>> + --disable-guest-agent > >>>>> > >>>>> This is adding XC_WANT_COMPAT_EVTCHN_API=1, etc, unconditionally. If we > >>>>> make this change, will raisin still be able to buil an older qemu > >>>>> against an older xen? > >>>> > >>>> I've tested this with XEN_RELEASE set to 4.5 and 4.6, and it builds just > >>>> fine. The #define is just ignored, since neither Xen nor qemu in the > >>>> 4.5 and 4.6 branches know anything about it. > >>>> > >>>> If part of our goal is to be a repository of the canonical way to build > >>>> things, though, perhaps it would be better to have the extra flags be > >>>> conditional on the value of XEN_RELEASE. > >>> > >>> If we are sure that it's ignored than it might be benign. Maybe we > >>> should just add an in-code comment to explain why we added it though. > >> > >> Well really, I think that the detection should be put into the qemu > >> configure scripts -- i.e., it should detect that there's the new API and > >> add the appropriate #defines itself. But that's the sort of change I > >> don't think there will be a lot of enthusiasm for. :-) > > > > Actually there is already code in QEMU configure script for the > > detection. In fact I was wondering if explicitly passing XC_WANT_* is > > actually necessary. I don't think it is from QEMU POV. > > That code might be in qemu-upstream, but it's not in the qemu-xen trees; > if you revert this bit of the patch and try to build with raisin it will > fail. That's why I even bothered to add these in in the first place; > and why IanC added these runes to tools/Makefile. > > Anthony, can you comment more authoritatively here on what's going on? Yes, easy, xen-4.7 introduce a new API but qemu-xen-4.7 does not know about it, so qemu-xen will fail to compile. But there are some magic flags that are use to provide an compatible API and can qemu-xen compile. QEMU have been taught about the new API and will use it, without the magic flags. QEMU will actually ignore the magic flags with a bunch of #undef. > Would it be possible for 4.7.1 to backport the configure auto-detection > stuff? I think that would be possible. But backporting the use of the new api will probably be harder. I guess we could just have qemu-xen define the magic flags itself instead of relying on them been given via a configure option. Is that answer your questions? -- Anthony PERARD _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH raisin 4/4] Update to 4.7, update qemu and qemu_traditional recipes 2016-06-14 13:53 ` Anthony PERARD @ 2016-06-14 14:00 ` George Dunlap 2016-06-14 14:23 ` Anthony PERARD 2016-06-15 10:39 ` Stefano Stabellini 1 sibling, 1 reply; 29+ messages in thread From: George Dunlap @ 2016-06-14 14:00 UTC (permalink / raw) To: Anthony PERARD; +Cc: xen-devel, Stefano Stabellini On 14/06/16 14:53, Anthony PERARD wrote: > On Tue, Jun 14, 2016 at 11:34:43AM +0100, George Dunlap wrote: >> On 14/06/16 11:31, Stefano Stabellini wrote: >>> On Tue, 14 Jun 2016, George Dunlap wrote: >>>> On 14/06/16 11:08, Stefano Stabellini wrote: >>>>> On Tue, 14 Jun 2016, George Dunlap wrote: >>>>>> On 14/06/16 10:46, Stefano Stabellini wrote: >>>>>>> On Mon, 13 Jun 2016, George Dunlap wrote: >>>>>>>> Add a 4.7 config file, make it the default. >>>>>>>> >>>>>>>> Also update the qemu and qemu_traditional recipies after Ian Cambell's >>>>>>>> work to split off separate libraries. >>>>>>>> >>>>>>>> Signed-off-by: George Dunlap <george.dunlap@citrix.com> >>>>>>>> --- >>>>>>>> CC: Stefano Stabellini <sstabellini@kernel.org> >>>>>>>> --- >>>>>>>> components/qemu | 21 +++++++++++++-------- >>>>>>>> components/qemu_traditional | 2 +- >>>>>>>> configs/config-4.7 | 8 ++++++++ >>>>>>>> defconfig | 2 +- >>>>>>>> 4 files changed, 23 insertions(+), 10 deletions(-) >>>>>>>> >>>>>>>> diff --git a/components/qemu b/components/qemu >>>>>>>> index e0d92a5..8624b50 100644 >>>>>>>> --- a/components/qemu >>>>>>>> +++ b/components/qemu >>>>>>>> @@ -23,15 +23,20 @@ function qemu_build() { >>>>>>>> cd "$BASEDIR" >>>>>>>> git-checkout $QEMU_URL $QEMU_REVISION qemu-dir >>>>>>>> cd qemu-dir >>>>>>>> - ./configure --enable-xen --target-list=i386-softmmu --prefix=$PREFIX \ >>>>>>>> - --extra-cflags="-I$INST_DIR/$PREFIX/include" \ >>>>>>>> - --extra-ldflags="-L$INST_DIR/$PREFIX/lib -Wl,-rpath-link=$INST_DIR/$PREFIX/lib \ >>>>>>>> + >>>>>>>> + ./configure --enable-xen --target-list=i386-softmmu \ >>>>>>>> + --prefix=$PREFIX \ >>>>>>>> + --extra-cflags="-DXC_WANT_COMPAT_EVTCHN_API=1 \ >>>>>>>> + -DXC_WANT_COMPAT_GNTTAB_API=1 \ >>>>>>>> + -DXC_WANT_COMPAT_MAP_FOREIGN_API=1 \ >>>>>>>> + -I$INST_DIR/$PREFIX/include" \ >>>>>>>> + --extra-ldflags="-L$INST_DIR/$PREFIX/lib -Wl,-rpath-link=$INST_DIR/$PREFIX/lib \ >>>>>>>> -L$INST_DIR/$PREFIX/lib64 -Wl,-rpath-link=$INST_DIR/$PREFIX/lib64" \ >>>>>>>> - --disable-kvm \ >>>>>>>> - --disable-docs \ >>>>>>>> - --bindir=$PREFIX/lib/xen/bin \ >>>>>>>> - --datadir=$PREFIX/share/qemu-xen \ >>>>>>>> - --disable-guest-agent >>>>>>>> + --bindir=$PREFIX/lib/xen/bin \ >>>>>>>> + --datadir=$PREFIX/share/qemu-xen \ >>>>>>>> + --disable-kvm \ >>>>>>>> + --disable-docs \ >>>>>>>> + --disable-guest-agent >>>>>>> >>>>>>> This is adding XC_WANT_COMPAT_EVTCHN_API=1, etc, unconditionally. If we >>>>>>> make this change, will raisin still be able to buil an older qemu >>>>>>> against an older xen? >>>>>> >>>>>> I've tested this with XEN_RELEASE set to 4.5 and 4.6, and it builds just >>>>>> fine. The #define is just ignored, since neither Xen nor qemu in the >>>>>> 4.5 and 4.6 branches know anything about it. >>>>>> >>>>>> If part of our goal is to be a repository of the canonical way to build >>>>>> things, though, perhaps it would be better to have the extra flags be >>>>>> conditional on the value of XEN_RELEASE. >>>>> >>>>> If we are sure that it's ignored than it might be benign. Maybe we >>>>> should just add an in-code comment to explain why we added it though. >>>> >>>> Well really, I think that the detection should be put into the qemu >>>> configure scripts -- i.e., it should detect that there's the new API and >>>> add the appropriate #defines itself. But that's the sort of change I >>>> don't think there will be a lot of enthusiasm for. :-) >>> >>> Actually there is already code in QEMU configure script for the >>> detection. In fact I was wondering if explicitly passing XC_WANT_* is >>> actually necessary. I don't think it is from QEMU POV. >> >> That code might be in qemu-upstream, but it's not in the qemu-xen trees; >> if you revert this bit of the patch and try to build with raisin it will >> fail. That's why I even bothered to add these in in the first place; >> and why IanC added these runes to tools/Makefile. >> >> Anthony, can you comment more authoritatively here on what's going on? > > Yes, easy, xen-4.7 introduce a new API but qemu-xen-4.7 does not know > about it, so qemu-xen will fail to compile. But there are some magic > flags that are use to provide an compatible API and can qemu-xen > compile. > > QEMU have been taught about the new API and will use it, without the > magic flags. QEMU will actually ignore the magic flags with a bunch of > #undef. So wait, I'm confused. Isn't qemu-xen supposed to be upstream qemu plus a handful of patches on top? Or has there not been a QEMU release since the time the new API? Why on earth are we shipping in 4.7 a qemu that can't use the new API we define in 4.7? >> Would it be possible for 4.7.1 to backport the configure auto-detection >> stuff? > > I think that would be possible. But backporting the use of the new api > will probably be harder. I guess we could just have qemu-xen define the > magic flags itself instead of relying on them been given via a configure > option. Well at this point it's a bit late to import support for the new API into qemu-xen-4.7; but it would be good for 4.7.1 to have qemu-xen's configure set the necessary flags rather than bodging them in via "--extra-cflags" in the Xen makefile. -George _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH raisin 4/4] Update to 4.7, update qemu and qemu_traditional recipes 2016-06-14 14:00 ` George Dunlap @ 2016-06-14 14:23 ` Anthony PERARD 2016-06-14 14:38 ` George Dunlap 0 siblings, 1 reply; 29+ messages in thread From: Anthony PERARD @ 2016-06-14 14:23 UTC (permalink / raw) To: George Dunlap; +Cc: xen-devel, Stefano Stabellini On Tue, Jun 14, 2016 at 03:00:12PM +0100, George Dunlap wrote: > On 14/06/16 14:53, Anthony PERARD wrote: > > On Tue, Jun 14, 2016 at 11:34:43AM +0100, George Dunlap wrote: > >> That code might be in qemu-upstream, but it's not in the qemu-xen trees; > >> if you revert this bit of the patch and try to build with raisin it will > >> fail. That's why I even bothered to add these in in the first place; > >> and why IanC added these runes to tools/Makefile. > >> > >> Anthony, can you comment more authoritatively here on what's going on? > > > > Yes, easy, xen-4.7 introduce a new API but qemu-xen-4.7 does not know > > about it, so qemu-xen will fail to compile. But there are some magic > > flags that are use to provide an compatible API and can qemu-xen > > compile. > > > > QEMU have been taught about the new API and will use it, without the > > magic flags. QEMU will actually ignore the magic flags with a bunch of > > #undef. > > So wait, I'm confused. Isn't qemu-xen supposed to be upstream qemu plus > a handful of patches on top? Or has there not been a QEMU release since > the time the new API? The first release to include the new stuff appear to be 2.6, which have been released a month ago. So the answer is no. > Why on earth are we shipping in 4.7 a qemu that > can't use the new API we define in 4.7? Who knows... > >> Would it be possible for 4.7.1 to backport the configure auto-detection > >> stuff? > > > > I think that would be possible. But backporting the use of the new api > > will probably be harder. I guess we could just have qemu-xen define the > > magic flags itself instead of relying on them been given via a configure > > option. > > Well at this point it's a bit late to import support for the new API > into qemu-xen-4.7; but it would be good for 4.7.1 to have qemu-xen's > configure set the necessary flags rather than bodging them in via > "--extra-cflags" in the Xen makefile. Yes. -- Anthony PERARD _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH raisin 4/4] Update to 4.7, update qemu and qemu_traditional recipes 2016-06-14 14:23 ` Anthony PERARD @ 2016-06-14 14:38 ` George Dunlap 0 siblings, 0 replies; 29+ messages in thread From: George Dunlap @ 2016-06-14 14:38 UTC (permalink / raw) To: Anthony PERARD; +Cc: xen-devel, Stefano Stabellini On 14/06/16 15:23, Anthony PERARD wrote: > On Tue, Jun 14, 2016 at 03:00:12PM +0100, George Dunlap wrote: >> On 14/06/16 14:53, Anthony PERARD wrote: >>> On Tue, Jun 14, 2016 at 11:34:43AM +0100, George Dunlap wrote: >>>> That code might be in qemu-upstream, but it's not in the qemu-xen trees; >>>> if you revert this bit of the patch and try to build with raisin it will >>>> fail. That's why I even bothered to add these in in the first place; >>>> and why IanC added these runes to tools/Makefile. >>>> >>>> Anthony, can you comment more authoritatively here on what's going on? >>> >>> Yes, easy, xen-4.7 introduce a new API but qemu-xen-4.7 does not know >>> about it, so qemu-xen will fail to compile. But there are some magic >>> flags that are use to provide an compatible API and can qemu-xen >>> compile. >>> >>> QEMU have been taught about the new API and will use it, without the >>> magic flags. QEMU will actually ignore the magic flags with a bunch of >>> #undef. >> >> So wait, I'm confused. Isn't qemu-xen supposed to be upstream qemu plus >> a handful of patches on top? Or has there not been a QEMU release since >> the time the new API? > > The first release to include the new stuff appear to be 2.6, which have > been released a month ago. So the answer is no. OK, but presumably you'll pull in / rebase qemu-unstable at some point in the next few weeks and then we can take the magic runes out of tools/Makefile. So back to raisin: I'll make a conditional to only add this for 4.7, with an explanation; and when it gets fixed for 4.7.1 I'll take it out. Thanks. -George _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH raisin 4/4] Update to 4.7, update qemu and qemu_traditional recipes 2016-06-14 13:53 ` Anthony PERARD 2016-06-14 14:00 ` George Dunlap @ 2016-06-15 10:39 ` Stefano Stabellini 2016-06-15 10:42 ` George Dunlap 1 sibling, 1 reply; 29+ messages in thread From: Stefano Stabellini @ 2016-06-15 10:39 UTC (permalink / raw) To: Anthony PERARD; +Cc: xen-devel, Stefano Stabellini, George Dunlap On Tue, 14 Jun 2016, Anthony PERARD wrote: > On Tue, Jun 14, 2016 at 11:34:43AM +0100, George Dunlap wrote: > > On 14/06/16 11:31, Stefano Stabellini wrote: > > > On Tue, 14 Jun 2016, George Dunlap wrote: > > >> On 14/06/16 11:08, Stefano Stabellini wrote: > > >>> On Tue, 14 Jun 2016, George Dunlap wrote: > > >>>> On 14/06/16 10:46, Stefano Stabellini wrote: > > >>>>> On Mon, 13 Jun 2016, George Dunlap wrote: > > >>>>>> Add a 4.7 config file, make it the default. > > >>>>>> > > >>>>>> Also update the qemu and qemu_traditional recipies after Ian Cambell's > > >>>>>> work to split off separate libraries. > > >>>>>> > > >>>>>> Signed-off-by: George Dunlap <george.dunlap@citrix.com> > > >>>>>> --- > > >>>>>> CC: Stefano Stabellini <sstabellini@kernel.org> > > >>>>>> --- > > >>>>>> components/qemu | 21 +++++++++++++-------- > > >>>>>> components/qemu_traditional | 2 +- > > >>>>>> configs/config-4.7 | 8 ++++++++ > > >>>>>> defconfig | 2 +- > > >>>>>> 4 files changed, 23 insertions(+), 10 deletions(-) > > >>>>>> > > >>>>>> diff --git a/components/qemu b/components/qemu > > >>>>>> index e0d92a5..8624b50 100644 > > >>>>>> --- a/components/qemu > > >>>>>> +++ b/components/qemu > > >>>>>> @@ -23,15 +23,20 @@ function qemu_build() { > > >>>>>> cd "$BASEDIR" > > >>>>>> git-checkout $QEMU_URL $QEMU_REVISION qemu-dir > > >>>>>> cd qemu-dir > > >>>>>> - ./configure --enable-xen --target-list=i386-softmmu --prefix=$PREFIX \ > > >>>>>> - --extra-cflags="-I$INST_DIR/$PREFIX/include" \ > > >>>>>> - --extra-ldflags="-L$INST_DIR/$PREFIX/lib -Wl,-rpath-link=$INST_DIR/$PREFIX/lib \ > > >>>>>> + > > >>>>>> + ./configure --enable-xen --target-list=i386-softmmu \ > > >>>>>> + --prefix=$PREFIX \ > > >>>>>> + --extra-cflags="-DXC_WANT_COMPAT_EVTCHN_API=1 \ > > >>>>>> + -DXC_WANT_COMPAT_GNTTAB_API=1 \ > > >>>>>> + -DXC_WANT_COMPAT_MAP_FOREIGN_API=1 \ > > >>>>>> + -I$INST_DIR/$PREFIX/include" \ > > >>>>>> + --extra-ldflags="-L$INST_DIR/$PREFIX/lib -Wl,-rpath-link=$INST_DIR/$PREFIX/lib \ > > >>>>>> -L$INST_DIR/$PREFIX/lib64 -Wl,-rpath-link=$INST_DIR/$PREFIX/lib64" \ > > >>>>>> - --disable-kvm \ > > >>>>>> - --disable-docs \ > > >>>>>> - --bindir=$PREFIX/lib/xen/bin \ > > >>>>>> - --datadir=$PREFIX/share/qemu-xen \ > > >>>>>> - --disable-guest-agent > > >>>>>> + --bindir=$PREFIX/lib/xen/bin \ > > >>>>>> + --datadir=$PREFIX/share/qemu-xen \ > > >>>>>> + --disable-kvm \ > > >>>>>> + --disable-docs \ > > >>>>>> + --disable-guest-agent > > >>>>> > > >>>>> This is adding XC_WANT_COMPAT_EVTCHN_API=1, etc, unconditionally. If we > > >>>>> make this change, will raisin still be able to buil an older qemu > > >>>>> against an older xen? > > >>>> > > >>>> I've tested this with XEN_RELEASE set to 4.5 and 4.6, and it builds just > > >>>> fine. The #define is just ignored, since neither Xen nor qemu in the > > >>>> 4.5 and 4.6 branches know anything about it. > > >>>> > > >>>> If part of our goal is to be a repository of the canonical way to build > > >>>> things, though, perhaps it would be better to have the extra flags be > > >>>> conditional on the value of XEN_RELEASE. > > >>> > > >>> If we are sure that it's ignored than it might be benign. Maybe we > > >>> should just add an in-code comment to explain why we added it though. > > >> > > >> Well really, I think that the detection should be put into the qemu > > >> configure scripts -- i.e., it should detect that there's the new API and > > >> add the appropriate #defines itself. But that's the sort of change I > > >> don't think there will be a lot of enthusiasm for. :-) > > > > > > Actually there is already code in QEMU configure script for the > > > detection. In fact I was wondering if explicitly passing XC_WANT_* is > > > actually necessary. I don't think it is from QEMU POV. > > > > That code might be in qemu-upstream, but it's not in the qemu-xen trees; > > if you revert this bit of the patch and try to build with raisin it will > > fail. That's why I even bothered to add these in in the first place; > > and why IanC added these runes to tools/Makefile. > > > > Anthony, can you comment more authoritatively here on what's going on? > > Yes, easy, xen-4.7 introduce a new API but qemu-xen-4.7 does not know > about it, so qemu-xen will fail to compile. But there are some magic > flags that are use to provide an compatible API and can qemu-xen > compile. > > QEMU have been taught about the new API and will use it, without the > magic flags. QEMU will actually ignore the magic flags with a bunch of > #undef. Looking a bit more closely, these flags are to enable backward compatibility in libxc, which of course is not supposed to break any QEMUs, old or new. To the contrary, it helps older QEMUs to build properly against newer Xen versions (qemu-xen-4.7 falls in this category). In addition new QEMU is able to ignore these flags when it knows that it can use the new interface. So overall I would just pass them unconditionally with a comment as explanation. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 29+ messages in thread
* Re: [PATCH raisin 4/4] Update to 4.7, update qemu and qemu_traditional recipes 2016-06-15 10:39 ` Stefano Stabellini @ 2016-06-15 10:42 ` George Dunlap 0 siblings, 0 replies; 29+ messages in thread From: George Dunlap @ 2016-06-15 10:42 UTC (permalink / raw) To: Stefano Stabellini, Anthony PERARD; +Cc: xen-devel On 15/06/16 11:39, Stefano Stabellini wrote: > On Tue, 14 Jun 2016, Anthony PERARD wrote: >> On Tue, Jun 14, 2016 at 11:34:43AM +0100, George Dunlap wrote: >>> On 14/06/16 11:31, Stefano Stabellini wrote: >>>> On Tue, 14 Jun 2016, George Dunlap wrote: >>>>> On 14/06/16 11:08, Stefano Stabellini wrote: >>>>>> On Tue, 14 Jun 2016, George Dunlap wrote: >>>>>>> On 14/06/16 10:46, Stefano Stabellini wrote: >>>>>>>> On Mon, 13 Jun 2016, George Dunlap wrote: >>>>>>>>> Add a 4.7 config file, make it the default. >>>>>>>>> >>>>>>>>> Also update the qemu and qemu_traditional recipies after Ian Cambell's >>>>>>>>> work to split off separate libraries. >>>>>>>>> >>>>>>>>> Signed-off-by: George Dunlap <george.dunlap@citrix.com> >>>>>>>>> --- >>>>>>>>> CC: Stefano Stabellini <sstabellini@kernel.org> >>>>>>>>> --- >>>>>>>>> components/qemu | 21 +++++++++++++-------- >>>>>>>>> components/qemu_traditional | 2 +- >>>>>>>>> configs/config-4.7 | 8 ++++++++ >>>>>>>>> defconfig | 2 +- >>>>>>>>> 4 files changed, 23 insertions(+), 10 deletions(-) >>>>>>>>> >>>>>>>>> diff --git a/components/qemu b/components/qemu >>>>>>>>> index e0d92a5..8624b50 100644 >>>>>>>>> --- a/components/qemu >>>>>>>>> +++ b/components/qemu >>>>>>>>> @@ -23,15 +23,20 @@ function qemu_build() { >>>>>>>>> cd "$BASEDIR" >>>>>>>>> git-checkout $QEMU_URL $QEMU_REVISION qemu-dir >>>>>>>>> cd qemu-dir >>>>>>>>> - ./configure --enable-xen --target-list=i386-softmmu --prefix=$PREFIX \ >>>>>>>>> - --extra-cflags="-I$INST_DIR/$PREFIX/include" \ >>>>>>>>> - --extra-ldflags="-L$INST_DIR/$PREFIX/lib -Wl,-rpath-link=$INST_DIR/$PREFIX/lib \ >>>>>>>>> + >>>>>>>>> + ./configure --enable-xen --target-list=i386-softmmu \ >>>>>>>>> + --prefix=$PREFIX \ >>>>>>>>> + --extra-cflags="-DXC_WANT_COMPAT_EVTCHN_API=1 \ >>>>>>>>> + -DXC_WANT_COMPAT_GNTTAB_API=1 \ >>>>>>>>> + -DXC_WANT_COMPAT_MAP_FOREIGN_API=1 \ >>>>>>>>> + -I$INST_DIR/$PREFIX/include" \ >>>>>>>>> + --extra-ldflags="-L$INST_DIR/$PREFIX/lib -Wl,-rpath-link=$INST_DIR/$PREFIX/lib \ >>>>>>>>> -L$INST_DIR/$PREFIX/lib64 -Wl,-rpath-link=$INST_DIR/$PREFIX/lib64" \ >>>>>>>>> - --disable-kvm \ >>>>>>>>> - --disable-docs \ >>>>>>>>> - --bindir=$PREFIX/lib/xen/bin \ >>>>>>>>> - --datadir=$PREFIX/share/qemu-xen \ >>>>>>>>> - --disable-guest-agent >>>>>>>>> + --bindir=$PREFIX/lib/xen/bin \ >>>>>>>>> + --datadir=$PREFIX/share/qemu-xen \ >>>>>>>>> + --disable-kvm \ >>>>>>>>> + --disable-docs \ >>>>>>>>> + --disable-guest-agent >>>>>>>> >>>>>>>> This is adding XC_WANT_COMPAT_EVTCHN_API=1, etc, unconditionally. If we >>>>>>>> make this change, will raisin still be able to buil an older qemu >>>>>>>> against an older xen? >>>>>>> >>>>>>> I've tested this with XEN_RELEASE set to 4.5 and 4.6, and it builds just >>>>>>> fine. The #define is just ignored, since neither Xen nor qemu in the >>>>>>> 4.5 and 4.6 branches know anything about it. >>>>>>> >>>>>>> If part of our goal is to be a repository of the canonical way to build >>>>>>> things, though, perhaps it would be better to have the extra flags be >>>>>>> conditional on the value of XEN_RELEASE. >>>>>> >>>>>> If we are sure that it's ignored than it might be benign. Maybe we >>>>>> should just add an in-code comment to explain why we added it though. >>>>> >>>>> Well really, I think that the detection should be put into the qemu >>>>> configure scripts -- i.e., it should detect that there's the new API and >>>>> add the appropriate #defines itself. But that's the sort of change I >>>>> don't think there will be a lot of enthusiasm for. :-) >>>> >>>> Actually there is already code in QEMU configure script for the >>>> detection. In fact I was wondering if explicitly passing XC_WANT_* is >>>> actually necessary. I don't think it is from QEMU POV. >>> >>> That code might be in qemu-upstream, but it's not in the qemu-xen trees; >>> if you revert this bit of the patch and try to build with raisin it will >>> fail. That's why I even bothered to add these in in the first place; >>> and why IanC added these runes to tools/Makefile. >>> >>> Anthony, can you comment more authoritatively here on what's going on? >> >> Yes, easy, xen-4.7 introduce a new API but qemu-xen-4.7 does not know >> about it, so qemu-xen will fail to compile. But there are some magic >> flags that are use to provide an compatible API and can qemu-xen >> compile. >> >> QEMU have been taught about the new API and will use it, without the >> magic flags. QEMU will actually ignore the magic flags with a bunch of >> #undef. > > Looking a bit more closely, these flags are to enable backward > compatibility in libxc, which of course is not supposed to break any > QEMUs, old or new. To the contrary, it helps older QEMUs to build > properly against newer Xen versions (qemu-xen-4.7 falls in this > category). In addition new QEMU is able to ignore these flags when it > knows that it can use the new interface. So overall I would just pass > them unconditionally with a comment as explanation. Well I've already updated the patch to enable them conditionally (with a comment), so I think I'll just use the new version. :-) -George _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 29+ messages in thread
end of thread, other threads:[~2016-06-15 16:12 UTC | newest] Thread overview: 29+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2016-06-13 14:44 [PATCH raisin 0/4] Fixes for 4.6, update to 4.7 George Dunlap 2016-06-13 14:44 ` [PATCH raisin 1/4] components/xen: Actually disable rombios George Dunlap 2016-06-14 9:32 ` Stefano Stabellini 2016-06-13 14:44 ` [PATCH raisin 2/4] config: Separate config urls into a separate file George Dunlap 2016-06-14 9:34 ` Stefano Stabellini 2016-06-13 14:44 ` [PATCH raisin 3/4] Update config-4.6 and config-4.5 to point to stable branches George Dunlap 2016-06-14 9:40 ` Stefano Stabellini 2016-06-14 9:57 ` George Dunlap 2016-06-14 10:01 ` Stefano Stabellini 2016-06-14 10:10 ` George Dunlap 2016-06-15 10:24 ` Stefano Stabellini 2016-06-15 10:38 ` George Dunlap 2016-06-15 10:53 ` Stefano Stabellini 2016-06-15 11:13 ` George Dunlap 2016-06-15 14:02 ` Stefano Stabellini 2016-06-15 16:11 ` George Dunlap 2016-06-13 14:44 ` [PATCH raisin 4/4] Update to 4.7, update qemu and qemu_traditional recipes George Dunlap 2016-06-14 9:46 ` Stefano Stabellini 2016-06-14 10:03 ` George Dunlap 2016-06-14 10:08 ` Stefano Stabellini 2016-06-14 10:17 ` George Dunlap 2016-06-14 10:31 ` Stefano Stabellini 2016-06-14 10:34 ` George Dunlap 2016-06-14 13:53 ` Anthony PERARD 2016-06-14 14:00 ` George Dunlap 2016-06-14 14:23 ` Anthony PERARD 2016-06-14 14:38 ` George Dunlap 2016-06-15 10:39 ` Stefano Stabellini 2016-06-15 10:42 ` George Dunlap
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).