xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/2] Open Xen 4.9-unstable, reenable debug
@ 2016-12-02 12:22 Ian Jackson
  2016-12-02 12:23 ` [PATCH 1/2] Open Xen 4.9-unstable Ian Jackson
                   ` (2 more replies)
  0 siblings, 3 replies; 7+ messages in thread
From: Ian Jackson @ 2016-12-02 12:22 UTC (permalink / raw)
  To: xen-devel

I have pushed the first of these two patches to new-staging.  It is
very like all previous such commits.  The second, to reenable
hypervisor debug, I decided to get some review on.

When staging is properly open, I or the release manager will send an
announcement about its status and what should and should not be
committed where.

Thanks,
Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH 1/2] Open Xen 4.9-unstable
  2016-12-02 12:22 [PATCH 0/2] Open Xen 4.9-unstable, reenable debug Ian Jackson
@ 2016-12-02 12:23 ` Ian Jackson
  2016-12-02 12:53   ` Andrew Cooper
  2016-12-02 12:23 ` [PATCH 2/2] Re-enable hypervisor debug as part of opening 4.9 Ian Jackson
  2016-12-02 13:18 ` [PATCH 0/2] Open Xen 4.9-unstable, reenable debug Olaf Hering
  2 siblings, 1 reply; 7+ messages in thread
From: Ian Jackson @ 2016-12-02 12:23 UTC (permalink / raw)
  To: xen-devel; +Cc: Ian Jackson

* Change version number in README and xen/Makefile to `4.8-unstable'.

* Set `debug ?= y' in Config.mk.

* Set QEMU_UPSTREAM_REVISION to track qemu-xen.git `master'.

* Set MINIOS_UPSTREAM_REVISION and QEMU_TRADITIONAL_REVISION back to
  commit hashes, rather than 4.8 tags.

Hypervisor debug enablement is left aside for now.

Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
---
 Config.mk    | 10 ++++++----
 README       | 10 +++++-----
 xen/Makefile |  4 ++--
 3 files changed, 13 insertions(+), 11 deletions(-)

diff --git a/Config.mk b/Config.mk
index 1de7f36..1e02f62 100644
--- a/Config.mk
+++ b/Config.mk
@@ -18,7 +18,7 @@ or       = $(if $(strip $(1)),$(1),$(if $(strip $(2)),$(2),$(if $(strip $(3)),$(
 
 # A debug build of tools?
 # Hypervisor debug build is controlled by Kconfig.
-debug ?= n
+debug ?= y
 debug_symbols ?= $(debug)
 
 XEN_COMPILE_ARCH    ?= $(shell uname -m | sed -e s/i.86/x86_32/ \
@@ -277,8 +277,8 @@ SEABIOS_UPSTREAM_URL ?= git://xenbits.xen.org/seabios.git
 MINIOS_UPSTREAM_URL ?= git://xenbits.xen.org/mini-os.git
 endif
 OVMF_UPSTREAM_REVISION ?= bc54e50e0fe03c570014f363b547426913e92449
-QEMU_UPSTREAM_REVISION ?= qemu-xen-4.8.0-rc7
-MINIOS_UPSTREAM_REVISION ?= xen-4.8.0-rc1
+QEMU_UPSTREAM_REVISION ?= master
+MINIOS_UPSTREAM_REVISION ?= e20998fbec0af4d783abb1a0695ab4614064c520
 # Wed Sep 28 11:50:04 2016 +0200
 # minios: fix build issue with xen_*mb defines
 
@@ -289,7 +289,9 @@ SEABIOS_UPSTREAM_REVISION ?= rel-1.10.0
 ETHERBOOT_NICS ?= rtl8139 8086100e
 
 
-QEMU_TRADITIONAL_REVISION ?= xen-4.8.0-rc7
+QEMU_TRADITIONAL_REVISION ?= 89c4cbe8d234049b0145e4dc5e5d19d626250b57
+# Tue Nov 29 16:36:38 2016 +0000
+# xen: fix ioreq handling
 
 # Specify which qemu-dm to use. This may be `ioemu' to use the old
 # Mercurial in-tree version, or a local directory, or a git URL.
diff --git a/README b/README
index b370ce2..76433a9 100644
--- a/README
+++ b/README
@@ -1,9 +1,9 @@
 #################################
-__  __            _  _    ___   ___
-\ \/ /___ _ __   | || |  ( _ ) / _ \       _ __ ___
- \  // _ \ '_ \  | || |_ / _ \| | | |_____| '__/ __|
- /  \  __/ | | | |__   _| (_) | |_| |_____| | | (__
-/_/\_\___|_| |_|    |_|(_)___(_)___/      |_|  \___|
+__  __            _  _   ___                        _        _     _
+\ \/ /___ _ __   | || | / _ \       _   _ _ __  ___| |_ __ _| |__ | | ___
+ \  // _ \ '_ \  | || || (_) |_____| | | | '_ \/ __| __/ _` | '_ \| |/ _ \
+ /  \  __/ | | | |__   _\__, |_____| |_| | | | \__ \ || (_| | |_) | |  __/
+/_/\_\___|_| |_|    |_|(_)/_/       \__,_|_| |_|___/\__\__,_|_.__/|_|\___|
 
 #################################
 
diff --git a/xen/Makefile b/xen/Makefile
index b8df99f..ddac7e0 100644
--- a/xen/Makefile
+++ b/xen/Makefile
@@ -1,8 +1,8 @@
 # This is the correct place to edit the build version.
 # All other places this is stored (eg. compile.h) should be autogenerated.
 export XEN_VERSION       = 4
-export XEN_SUBVERSION    = 8
-export XEN_EXTRAVERSION ?= .0-rc$(XEN_VENDORVERSION)
+export XEN_SUBVERSION    = 9
+export XEN_EXTRAVERSION ?= -unstable$(XEN_VENDORVERSION)
 export XEN_FULLVERSION   = $(XEN_VERSION).$(XEN_SUBVERSION)$(XEN_EXTRAVERSION)
 -include xen-version
 
-- 
2.1.4


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 2/2] Re-enable hypervisor debug as part of opening 4.9
  2016-12-02 12:22 [PATCH 0/2] Open Xen 4.9-unstable, reenable debug Ian Jackson
  2016-12-02 12:23 ` [PATCH 1/2] Open Xen 4.9-unstable Ian Jackson
@ 2016-12-02 12:23 ` Ian Jackson
  2016-12-02 12:46   ` Jan Beulich
  2016-12-02 13:18 ` [PATCH 0/2] Open Xen 4.9-unstable, reenable debug Olaf Hering
  2 siblings, 1 reply; 7+ messages in thread
From: Ian Jackson @ 2016-12-02 12:23 UTC (permalink / raw)
  To: xen-devel
  Cc: Stefano Stabellini, Wei Liu, George Dunlap, Andrew Cooper,
	Ian Jackson, Tim Deegan, Jan Beulich

AFAICT following bacbf0cb7349 "build: convert debug to Kconfig"
hypervisor debug enablement is controlled here, rather than in
Config.mk.

The release checklist says that when branching, the new staging should
have debug enabled.  It seems to me that I should be changing this
here, therefore.

As additional evidence, I offer e1d1c68ea8a3 "xen: disable debug
build" which went in between 4.8.0 RC5 and RC6.  It does not explain
why this was done but it does STM that reverting that change is right.

CC: Andrew Cooper <andrew.cooper3@citrix.com>
CC: George Dunlap <George.Dunlap@eu.citrix.com>
CC: Jan Beulich <jbeulich@suse.com>
CC: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
CC: Stefano Stabellini <sstabellini@kernel.org>
CC: Tim Deegan <tim@xen.org>
CC: Wei Liu <wei.liu2@citrix.com>
Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
---
 xen/Kconfig.debug | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/xen/Kconfig.debug b/xen/Kconfig.debug
index b3bb085..e9f7dcd 100644
--- a/xen/Kconfig.debug
+++ b/xen/Kconfig.debug
@@ -3,7 +3,7 @@ menu "Debugging Options"
 
 config DEBUG
 	bool "Developer Checks"
-	default n
+	default y
 	---help---
 	  If you say Y here this will enable developer checks such as asserts
 	  and extra printks. This option is intended for development purposes
-- 
2.1.4


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH 2/2] Re-enable hypervisor debug as part of opening 4.9
  2016-12-02 12:23 ` [PATCH 2/2] Re-enable hypervisor debug as part of opening 4.9 Ian Jackson
@ 2016-12-02 12:46   ` Jan Beulich
  2016-12-02 12:51     ` Ian Jackson
  0 siblings, 1 reply; 7+ messages in thread
From: Jan Beulich @ 2016-12-02 12:46 UTC (permalink / raw)
  To: Ian Jackson
  Cc: Stefano Stabellini, Wei Liu, George Dunlap, Andrew Cooper,
	TimDeegan, xen-devel

>>> On 02.12.16 at 13:23, <ian.jackson@eu.citrix.com> wrote:
> AFAICT following bacbf0cb7349 "build: convert debug to Kconfig"
> hypervisor debug enablement is controlled here, rather than in
> Config.mk.
> 
> The release checklist says that when branching, the new staging should
> have debug enabled.  It seems to me that I should be changing this
> here, therefore.
> 
> As additional evidence, I offer e1d1c68ea8a3 "xen: disable debug
> build" which went in between 4.8.0 RC5 and RC6.  It does not explain
> why this was done but it does STM that reverting that change is right.
> 
> CC: Andrew Cooper <andrew.cooper3@citrix.com>
> CC: George Dunlap <George.Dunlap@eu.citrix.com>
> CC: Jan Beulich <jbeulich@suse.com>
> CC: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
> CC: Stefano Stabellini <sstabellini@kernel.org>
> CC: Tim Deegan <tim@xen.org>
> CC: Wei Liu <wei.liu2@citrix.com>
> Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>

Acked-by: Jan Beulich <jbeulich@suse.com>


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 2/2] Re-enable hypervisor debug as part of opening 4.9
  2016-12-02 12:46   ` Jan Beulich
@ 2016-12-02 12:51     ` Ian Jackson
  0 siblings, 0 replies; 7+ messages in thread
From: Ian Jackson @ 2016-12-02 12:51 UTC (permalink / raw)
  To: Jan Beulich
  Cc: Stefano Stabellini, Wei Liu, George Dunlap, Andrew Cooper,
	TimDeegan, xen-devel

Jan Beulich writes ("Re: [PATCH 2/2] Re-enable hypervisor debug as part of opening 4.9"):
> Acked-by: Jan Beulich <jbeulich@suse.com>

Thanks, pushed.  I have also updated the checklist.

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 1/2] Open Xen 4.9-unstable
  2016-12-02 12:23 ` [PATCH 1/2] Open Xen 4.9-unstable Ian Jackson
@ 2016-12-02 12:53   ` Andrew Cooper
  0 siblings, 0 replies; 7+ messages in thread
From: Andrew Cooper @ 2016-12-02 12:53 UTC (permalink / raw)
  To: Ian Jackson, xen-devel

On 02/12/16 12:23, Ian Jackson wrote:
> * Change version number in README and xen/Makefile to `4.8-unstable'.

ITYM 4.9 here, but it looks like the content of the patch was correct.

~Andrew

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 0/2] Open Xen 4.9-unstable, reenable debug
  2016-12-02 12:22 [PATCH 0/2] Open Xen 4.9-unstable, reenable debug Ian Jackson
  2016-12-02 12:23 ` [PATCH 1/2] Open Xen 4.9-unstable Ian Jackson
  2016-12-02 12:23 ` [PATCH 2/2] Re-enable hypervisor debug as part of opening 4.9 Ian Jackson
@ 2016-12-02 13:18 ` Olaf Hering
  2 siblings, 0 replies; 7+ messages in thread
From: Olaf Hering @ 2016-12-02 13:18 UTC (permalink / raw)
  To: Ian Jackson; +Cc: xen-devel


[-- Attachment #1.1: Type: text/plain, Size: 347 bytes --]

On Fri, Dec 02, Ian Jackson wrote:

> I have pushed the first of these two patches to new-staging.  It is
> very like all previous such commits.  The second, to reenable
> hypervisor debug, I decided to get some review on.

Please also bump the SONAMEs as it was done in commit
d337764d9b8e89eb9cb9d5be509823d9286f00c4 shortly after 4.7.0.

Olaf


[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

[-- Attachment #2: Type: text/plain, Size: 127 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2016-12-02 13:18 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-12-02 12:22 [PATCH 0/2] Open Xen 4.9-unstable, reenable debug Ian Jackson
2016-12-02 12:23 ` [PATCH 1/2] Open Xen 4.9-unstable Ian Jackson
2016-12-02 12:53   ` Andrew Cooper
2016-12-02 12:23 ` [PATCH 2/2] Re-enable hypervisor debug as part of opening 4.9 Ian Jackson
2016-12-02 12:46   ` Jan Beulich
2016-12-02 12:51     ` Ian Jackson
2016-12-02 13:18 ` [PATCH 0/2] Open Xen 4.9-unstable, reenable debug Olaf Hering

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).