[PATCH v3 0/2]: x86/arm64: add xenconfig

[PATCH v3 0/2]: x86/arm64: add xenconfig

[Luis R. Rodriguez]

From: "Luis R. Rodriguez" <mcgrof@suse.com>

This is based on some old set I had lying around. The virtconfig
changes I had proposed a while ago got merged and reused for
tinyconfig, this adapts my original set to use the new mergeconfig.

Not sure who's tree this should go through, last time these were
lost in space and only the non-xen things got cherry picked later,
who's tree should this go through?

Luis R. Rodriguez (2):
  x86, platform, xen, kconfig: clarify kvmconfig is for kvm
  x86, arm, platform, xen, kconfig: add xen defconfig helper

 arch/x86/configs/xen.config |  6 ++++++
 kernel/configs/xen.config   | 32 ++++++++++++++++++++++++++++++++
 scripts/kconfig/Makefile    |  7 ++++++-
 3 files changed, 44 insertions(+), 1 deletion(-)
 create mode 100644 arch/x86/configs/xen.config
 create mode 100644 kernel/configs/xen.config

-- 
2.1.1

[PATCH v3 1/2] x86, platform, kconfig: clarify kvmconfig is for kvm

[Luis R. Rodriguez]

From: "Luis R. Rodriguez" <mcgrof@suse.com>

We'll be adding options for xen as well.

Cc: Josh Triplett <josh@joshtriplett.org>
Cc: Borislav Petkov <bp@suse.de>
Cc: Pekka Enberg <penberg@kernel.org>
Cc: David Rientjes <rientjes@google.com>
Cc: Michal Marek <mmarek@suse.cz>
Cc: Randy Dunlap <rdunlap@infradead.org>
Cc: penberg@kernel.org
Cc: levinsasha928@gmail.com
Cc: mtosatti@redhat.com
Cc: fengguang.wu@intel.com
Cc: David Vrabel <david.vrabel@citrix.com>
Cc: Ian Campbell <Ian.Campbell@citrix.com>
Cc: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Cc: xen-devel@lists.xenproject.org
Acked-by: David Rientjes <rientjes@google.com>
Signed-off-by: Luis R. Rodriguez <mcgrof@suse.com>
---
 scripts/kconfig/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/kconfig/Makefile b/scripts/kconfig/Makefile
index 9645c07..ff612b0 100644
--- a/scripts/kconfig/Makefile
+++ b/scripts/kconfig/Makefile
@@ -141,7 +141,7 @@ help:
 	@echo  '  randconfig	  - New config with random answer to all options'
 	@echo  '  listnewconfig   - List new options'
 	@echo  '  olddefconfig	  - Same as silentoldconfig but sets new symbols to their default value'
-	@echo  '  kvmconfig	  - Enable additional options for guest kernel support'
+	@echo  '  kvmconfig	  - Enable additional options for kvm guest kernel support'
 	@echo  '  tinyconfig	  - Configure the tiniest possible kernel'
 
 # lxdialog stuff
-- 
2.1.1

Re: [PATCH v3 1/2] x86, platform, kconfig: clarify kvmconfig is for kvm

[Borislav Petkov]

On Mon, Dec 08, 2014 at 03:04:59PM -0800, Luis R. Rodriguez wrote:
> From: "Luis R. Rodriguez" <mcgrof@suse.com>
> 
> We'll be adding options for xen as well.
> 
> Cc: Josh Triplett <josh@joshtriplett.org>
> Cc: Borislav Petkov <bp@suse.de>
> Cc: Pekka Enberg <penberg@kernel.org>
> Cc: David Rientjes <rientjes@google.com>
> Cc: Michal Marek <mmarek@suse.cz>
> Cc: Randy Dunlap <rdunlap@infradead.org>
> Cc: penberg@kernel.org
> Cc: levinsasha928@gmail.com
> Cc: mtosatti@redhat.com
> Cc: fengguang.wu@intel.com
> Cc: David Vrabel <david.vrabel@citrix.com>
> Cc: Ian Campbell <Ian.Campbell@citrix.com>
> Cc: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
> Cc: xen-devel@lists.xenproject.org
> Acked-by: David Rientjes <rientjes@google.com>
> Signed-off-by: Luis R. Rodriguez <mcgrof@suse.com>
> ---
>  scripts/kconfig/Makefile | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/scripts/kconfig/Makefile b/scripts/kconfig/Makefile
> index 9645c07..ff612b0 100644
> --- a/scripts/kconfig/Makefile
> +++ b/scripts/kconfig/Makefile
> @@ -141,7 +141,7 @@ help:
>  	@echo  '  randconfig	  - New config with random answer to all options'
>  	@echo  '  listnewconfig   - List new options'
>  	@echo  '  olddefconfig	  - Same as silentoldconfig but sets new symbols to their default value'
> -	@echo  '  kvmconfig	  - Enable additional options for guest kernel support'
> +	@echo  '  kvmconfig	  - Enable additional options for kvm guest kernel support'
>  	@echo  '  tinyconfig	  - Configure the tiniest possible kernel'
>  
>  # lxdialog stuff

Acked-by: Borislav Petkov <bp@suse.de>

-- 
Regards/Gruss,
    Boris.

Sent from a fat crate under my desk. Formatting is fine.
--

[PATCH v3 2/2] x86, arm64, platform, xen, kconfig: add xen defconfig helper

[Luis R. Rodriguez]

From: "Luis R. Rodriguez" <mcgrof@suse.com>

This lets you build a kernel which can support xen dom0
or xen guests by just using:

   make xenconfig

on both x86 and arm64 kernels. This also splits out the
options which are available currently to be built with x86
and 'make ARCH=arm64' under a shared config.

Technically xen supports a dom0 kernel and also a guest
kernel configuration but upon review with the xen team
since we don't have many dom0 options its best to just
combine these two into one.

Cc: Josh Triplett <josh@joshtriplett.org>
Cc: Borislav Petkov <bp@suse.de>
Cc: Pekka Enberg <penberg@kernel.org>
Cc: David Rientjes <rientjes@google.com>
Cc: Michal Marek <mmarek@suse.cz>
Cc: Randy Dunlap <rdunlap@infradead.org>
Cc: penberg@kernel.org
Cc: levinsasha928@gmail.com
Cc: mtosatti@redhat.com
Cc: fengguang.wu@intel.com
Cc: David Vrabel <david.vrabel@citrix.com>
Cc: Ian Campbell <Ian.Campbell@citrix.com>
Cc: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Cc: xen-devel@lists.xenproject.org
Signed-off-by: Luis R. Rodriguez <mcgrof@suse.com>
---
 arch/x86/configs/xen.config |  6 ++++++
 kernel/configs/xen.config   | 32 ++++++++++++++++++++++++++++++++
 scripts/kconfig/Makefile    |  5 +++++
 3 files changed, 43 insertions(+)
 create mode 100644 arch/x86/configs/xen.config
 create mode 100644 kernel/configs/xen.config

diff --git a/arch/x86/configs/xen.config b/arch/x86/configs/xen.config
new file mode 100644
index 0000000..b97e893
--- /dev/null
+++ b/arch/x86/configs/xen.config
@@ -0,0 +1,6 @@
+# x86 xen specific config options
+CONFIG_XEN_PVHVM=y
+CONFIG_XEN_MAX_DOMAIN_MEMORY=500
+CONFIG_XEN_SAVE_RESTORE=y
+# CONFIG_XEN_DEBUG_FS is not set
+CONFIG_XEN_PVH=y
diff --git a/kernel/configs/xen.config b/kernel/configs/xen.config
new file mode 100644
index 0000000..0d0eb6d
--- /dev/null
+++ b/kernel/configs/xen.config
@@ -0,0 +1,32 @@
+# generic config
+CONFIG_XEN=y
+CONFIG_XEN_DOM0=y
+CONFIG_PCI_XEN=y
+CONFIG_XEN_PCIDEV_FRONTEND=m
+CONFIG_XEN_BLKDEV_FRONTEND=m
+CONFIG_XEN_BLKDEV_BACKEND=m
+CONFIG_XEN_NETDEV_FRONTEND=m
+CONFIG_XEN_NETDEV_BACKEND=m
+CONFIG_INPUT_XEN_KBDDEV_FRONTEND=y
+CONFIG_HVC_XEN=y
+CONFIG_HVC_XEN_FRONTEND=y
+CONFIG_TCG_XEN=m
+CONFIG_XEN_WDT=m
+CONFIG_XEN_FBDEV_FRONTEND=y
+CONFIG_XEN_BALLOON=y
+CONFIG_XEN_BALLOON_MEMORY_HOTPLUG=y
+CONFIG_XEN_SCRUB_PAGES=y
+CONFIG_XEN_DEV_EVTCHN=m
+CONFIG_XEN_BACKEND=y
+CONFIG_XENFS=m
+CONFIG_XEN_COMPAT_XENFS=y
+CONFIG_XEN_SYS_HYPERVISOR=y
+CONFIG_XEN_XENBUS_FRONTEND=y
+CONFIG_XEN_GNTDEV=m
+CONFIG_XEN_GRANT_DEV_ALLOC=m
+CONFIG_SWIOTLB_XEN=y
+CONFIG_XEN_PCIDEV_BACKEND=m
+CONFIG_XEN_PRIVCMD=m
+CONFIG_XEN_ACPI_PROCESSOR=m
+CONFIG_XEN_MCE_LOG=y
+CONFIG_XEN_HAVE_PVMMU=y
diff --git a/scripts/kconfig/Makefile b/scripts/kconfig/Makefile
index ff612b0..f4a8f89 100644
--- a/scripts/kconfig/Makefile
+++ b/scripts/kconfig/Makefile
@@ -117,6 +117,10 @@ PHONY += kvmconfig
 kvmconfig:
 	$(call mergeconfig,kvm_guest)
 
+PHONY += xenconfig
+xenconfig:
+	$(call mergeconfig,xen)
+
 PHONY += tinyconfig
 tinyconfig: allnoconfig
 	$(call mergeconfig,tiny)
@@ -142,6 +146,7 @@ help:
 	@echo  '  listnewconfig   - List new options'
 	@echo  '  olddefconfig	  - Same as silentoldconfig but sets new symbols to their default value'
 	@echo  '  kvmconfig	  - Enable additional options for kvm guest kernel support'
+	@echo  '  xenconfig       - Enable additional options for xen dom0 and guest kernel support'
 	@echo  '  tinyconfig	  - Configure the tiniest possible kernel'
 
 # lxdialog stuff
-- 
2.1.1

Re: [Xen-devel] [PATCH v3 2/2] x86, arm64, platform, xen, kconfig: add xen defconfig helper

[Julien Grall]

Hello Luis,

On 08/12/2014 23:05, Luis R. Rodriguez wrote:
> diff --git a/kernel/configs/xen.config b/kernel/configs/xen.config
> new file mode 100644
> index 0000000..0d0eb6d
> --- /dev/null
> +++ b/kernel/configs/xen.config
> +CONFIG_XEN_MCE_LOG=y

MCE is x86 specific.

> +CONFIG_XEN_HAVE_PVMMU=y

We don't have PVMMU support on ARM. Shouldn't you move this config in 
architecture specific code?

Regards

-- 
Julien Grall

Re: [Xen-devel] [PATCH v3 2/2] x86, arm64, platform, xen, kconfig: add xen defconfig helper

[Luis R. Rodriguez]

On Tue, Dec 9, 2014 at 1:06 AM, Julien Grall <julien.grall@linaro.org> wrote:
> Hello Luis,
>
> On 08/12/2014 23:05, Luis R. Rodriguez wrote:
>>
>> diff --git a/kernel/configs/xen.config b/kernel/configs/xen.config
>> new file mode 100644
>> index 0000000..0d0eb6d
>> --- /dev/null
>> +++ b/kernel/configs/xen.config
>> +CONFIG_XEN_MCE_LOG=y
>
>
> MCE is x86 specific.

That's what I thought too but its available for arm64, so should we
fix that Kconfig to depend on x86?

>> +CONFIG_XEN_HAVE_PVMMU=y
>
>
> We don't have PVMMU support on ARM. Shouldn't you move this config in
> architecture specific code?

If you are sure then yes.

 Luis

Re: [Xen-devel] [PATCH v3 2/2] x86, arm64, platform, xen, kconfig: add xen defconfig helper

[Luis R. Rodriguez]

On Tue, Dec 9, 2014 at 12:22 PM, Luis R. Rodriguez
<mcgrof@do-not-panic.com> wrote:
> If you are sure then yes.

Likewise here.

 Luis

Re: [Xen-devel] [PATCH v3 2/2] x86, arm64, platform, xen, kconfig: add xen defconfig helper

[Julien Grall]

On 09/12/14 20:22, Luis R. Rodriguez wrote:
> On Tue, Dec 9, 2014 at 1:06 AM, Julien Grall <julien.grall@linaro.org> wrote:
>> Hello Luis,
>>
>> On 08/12/2014 23:05, Luis R. Rodriguez wrote:
>>>
>>> diff --git a/kernel/configs/xen.config b/kernel/configs/xen.config
>>> new file mode 100644
>>> index 0000000..0d0eb6d
>>> --- /dev/null
>>> +++ b/kernel/configs/xen.config
>>> +CONFIG_XEN_MCE_LOG=y
>>
>>
>> MCE is x86 specific.
> 
> That's what I thought too but its available for arm64, so should we
> fix that Kconfig to depend on x86?

Are you sure? On the Linus's repo I have:

config XEN_MCE_LOG
        bool "Xen platform mcelog"
        depends on XEN_DOM0 && X86_64 && X86_MCE

Anyway, the MCE interface in the hypervisor is implemented in arch/x86
not in common code.

>>> +CONFIG_XEN_HAVE_PVMMU=y
>>
>>
>> We don't have PVMMU support on ARM. Shouldn't you move this config in
>> architecture specific code?
> 
> If you are sure then yes.

I'm 100% sure. MMU is handled by the hardware on ARM.

Thinking a bit more about this option. CONFIG_XEN_HAVE_PVMMU can't be
selected by the user. It's automatically added per platform (for
instance see arch/x86/xen/Kconfig).

So maybe it should not even appear in the one of the fragment configs?

Regards,

-- 
Julien Grall

Re: [Xen-devel] [PATCH v3 2/2] x86, arm64, platform, xen, kconfig: add xen defconfig helper

[Luis R. Rodriguez]

On Tue, Dec 9, 2014 at 12:37 PM, Julien Grall <julien.grall@linaro.org> wrote:
> On 09/12/14 20:22, Luis R. Rodriguez wrote:
>> On Tue, Dec 9, 2014 at 1:06 AM, Julien Grall <julien.grall@linaro.org> wrote:
>>> Hello Luis,
>>>
>>> On 08/12/2014 23:05, Luis R. Rodriguez wrote:
>>>>
>>>> diff --git a/kernel/configs/xen.config b/kernel/configs/xen.config
>>>> new file mode 100644
>>>> index 0000000..0d0eb6d
>>>> --- /dev/null
>>>> +++ b/kernel/configs/xen.config
>>>> +CONFIG_XEN_MCE_LOG=y
>>>
>>>
>>> MCE is x86 specific.
>>
>> That's what I thought too but its available for arm64, so should we
>> fix that Kconfig to depend on x86?
>
> Are you sure? On the Linus's repo I have:
>
> config XEN_MCE_LOG
>         bool "Xen platform mcelog"
>         depends on XEN_DOM0 && X86_64 && X86_MCE
>
> Anyway, the MCE interface in the hypervisor is implemented in arch/x86
> not in common code.

OK I'll move to x86.

>>>> +CONFIG_XEN_HAVE_PVMMU=y
>>>
>>>
>>> We don't have PVMMU support on ARM. Shouldn't you move this config in
>>> architecture specific code?
>>
>> If you are sure then yes.
>
> I'm 100% sure. MMU is handled by the hardware on ARM.
>
> Thinking a bit more about this option. CONFIG_XEN_HAVE_PVMMU can't be
> selected by the user. It's automatically added per platform (for
> instance see arch/x86/xen/Kconfig).
>
> So maybe it should not even appear in the one of the fragment configs?

And I'll remove this one.

 Luis

Re: [PATCH v3 0/2]: x86/arm64: add xenconfig

[Josh Triplett]

On Mon, Dec 08, 2014 at 03:04:58PM -0800, Luis R. Rodriguez wrote:
> From: "Luis R. Rodriguez" <mcgrof@suse.com>
> 
> This is based on some old set I had lying around. The virtconfig
> changes I had proposed a while ago got merged and reused for
> tinyconfig, this adapts my original set to use the new mergeconfig.
> 
> Not sure who's tree this should go through, last time these were
> lost in space and only the non-xen things got cherry picked later,
> who's tree should this go through?
> 
> Luis R. Rodriguez (2):
>   x86, platform, xen, kconfig: clarify kvmconfig is for kvm
>   x86, arm, platform, xen, kconfig: add xen defconfig helper

For both:
Reviewed-by: Josh Triplett <josh@joshtriplett.org>

>  arch/x86/configs/xen.config |  6 ++++++
>  kernel/configs/xen.config   | 32 ++++++++++++++++++++++++++++++++
>  scripts/kconfig/Makefile    |  7 ++++++-
>  3 files changed, 44 insertions(+), 1 deletion(-)
>  create mode 100644 arch/x86/configs/xen.config
>  create mode 100644 kernel/configs/xen.config
> 
> -- 
> 2.1.1
> 

[PATCH v3 0/2] x86/arm64: add xenconfig

[Luis R. Rodriguez]

From: "Luis R. Rodriguez" <mcgrof@suse.com>

This v3 addresses Stefano's feedback from the v2 series, namely
moving PCI stuff to x86 as its all x86 specific and also just
removing the CONFIG_TCG_XEN=m from the general config. To be
clear the changes from the v2 series are below.

Luis R. Rodriguez (2):
  x86, platform, xen, kconfig: clarify kvmconfig is for kvm
  x86, arm, platform, xen, kconfig: add xen defconfig helper

 arch/x86/configs/xen.config | 10 ++++++++++
 kernel/configs/xen.config   | 26 ++++++++++++++++++++++++++
 scripts/kconfig/Makefile    |  7 ++++++-
 3 files changed, 42 insertions(+), 1 deletion(-)
 create mode 100644 arch/x86/configs/xen.config
 create mode 100644 kernel/configs/xen.config

-- 
2.1.1

diff --git a/arch/x86/configs/xen.config b/arch/x86/configs/xen.config
index 92b8587f..fa7d214 100644
--- a/arch/x86/configs/xen.config
+++ b/arch/x86/configs/xen.config
@@ -5,3 +5,6 @@ CONFIG_XEN_SAVE_RESTORE=y
 # CONFIG_XEN_DEBUG_FS is not set
 CONFIG_XEN_PVH=y
 CONFIG_XEN_MCE_LOG=y
+CONFIG_PCI_XEN=y
+CONFIG_XEN_PCIDEV_FRONTEND=m
+CONFIG_XEN_ACPI_PROCESSOR=m
diff --git a/kernel/configs/xen.config b/kernel/configs/xen.config
index d2ec010..476fcf0 100644
--- a/kernel/configs/xen.config
+++ b/kernel/configs/xen.config
@@ -1,8 +1,6 @@
 # generic config
 CONFIG_XEN=y
 CONFIG_XEN_DOM0=y
-CONFIG_PCI_XEN=y
-CONFIG_XEN_PCIDEV_FRONTEND=m
 CONFIG_XEN_BLKDEV_FRONTEND=m
 CONFIG_XEN_BLKDEV_BACKEND=m
 CONFIG_XEN_NETDEV_FRONTEND=m
@@ -10,7 +8,6 @@ CONFIG_XEN_NETDEV_BACKEND=m
 CONFIG_INPUT_XEN_KBDDEV_FRONTEND=y
 CONFIG_HVC_XEN=y
 CONFIG_HVC_XEN_FRONTEND=y
-CONFIG_TCG_XEN=m
 CONFIG_XEN_WDT=m
 CONFIG_XEN_FBDEV_FRONTEND=y
 CONFIG_XEN_BALLOON=y
@@ -27,4 +24,3 @@ CONFIG_XEN_GRANT_DEV_ALLOC=m
 CONFIG_SWIOTLB_XEN=y
 CONFIG_XEN_PCIDEV_BACKEND=m
 CONFIG_XEN_PRIVCMD=m
-CONFIG_XEN_ACPI_PROCESSOR=m

Re: [PATCH v3 0/2] x86/arm64: add xenconfig

[Luis R. Rodriguez]

On Wed, Jan 14, 2015 at 11:33:45AM -0800, Luis R. Rodriguez wrote:
> From: "Luis R. Rodriguez" <mcgrof@suse.com>
> 
> This v3 addresses Stefano's feedback from the v2 series, namely
> moving PCI stuff to x86 as its all x86 specific and also just
> removing the CONFIG_TCG_XEN=m from the general config. To be
> clear the changes from the v2 series are below.
> 
> Luis R. Rodriguez (2):
>   x86, platform, xen, kconfig: clarify kvmconfig is for kvm
>   x86, arm, platform, xen, kconfig: add xen defconfig helper
> 
>  arch/x86/configs/xen.config | 10 ++++++++++
>  kernel/configs/xen.config   | 26 ++++++++++++++++++++++++++
>  scripts/kconfig/Makefile    |  7 ++++++-
>  3 files changed, 42 insertions(+), 1 deletion(-)
>  create mode 100644 arch/x86/configs/xen.config
>  create mode 100644 kernel/configs/xen.config

Who could these changes go through?

  Luis

Re: [PATCH v3 0/2] x86/arm64: add xenconfig

[Stefano Stabellini]

On Fri, 23 Jan 2015, Luis R. Rodriguez wrote:
> On Wed, Jan 14, 2015 at 11:33:45AM -0800, Luis R. Rodriguez wrote:
> > From: "Luis R. Rodriguez" <mcgrof@suse.com>
> > 
> > This v3 addresses Stefano's feedback from the v2 series, namely
> > moving PCI stuff to x86 as its all x86 specific and also just
> > removing the CONFIG_TCG_XEN=m from the general config. To be
> > clear the changes from the v2 series are below.
> > 
> > Luis R. Rodriguez (2):
> >   x86, platform, xen, kconfig: clarify kvmconfig is for kvm
> >   x86, arm, platform, xen, kconfig: add xen defconfig helper
> > 
> >  arch/x86/configs/xen.config | 10 ++++++++++
> >  kernel/configs/xen.config   | 26 ++++++++++++++++++++++++++
> >  scripts/kconfig/Makefile    |  7 ++++++-
> >  3 files changed, 42 insertions(+), 1 deletion(-)
> >  create mode 100644 arch/x86/configs/xen.config
> >  create mode 100644 kernel/configs/xen.config
> 
> Who could these changes go through?

I would be OK with taking it in the Xen tree, but I would feel more
comfortable doing that if you had an ack from Yann Morin (CC'ed).

Re: [PATCH v3 0/2] x86/arm64: add xenconfig

[Luis R. Rodriguez]

On Fri, Jan 23, 2015 at 03:19:25PM +0000, Stefano Stabellini wrote:
> On Fri, 23 Jan 2015, Luis R. Rodriguez wrote:
> > On Wed, Jan 14, 2015 at 11:33:45AM -0800, Luis R. Rodriguez wrote:
> > > From: "Luis R. Rodriguez" <mcgrof@suse.com>
> > > 
> > > This v3 addresses Stefano's feedback from the v2 series, namely
> > > moving PCI stuff to x86 as its all x86 specific and also just
> > > removing the CONFIG_TCG_XEN=m from the general config. To be
> > > clear the changes from the v2 series are below.
> > > 
> > > Luis R. Rodriguez (2):
> > >   x86, platform, xen, kconfig: clarify kvmconfig is for kvm
> > >   x86, arm, platform, xen, kconfig: add xen defconfig helper
> > > 
> > >  arch/x86/configs/xen.config | 10 ++++++++++
> > >  kernel/configs/xen.config   | 26 ++++++++++++++++++++++++++
> > >  scripts/kconfig/Makefile    |  7 ++++++-
> > >  3 files changed, 42 insertions(+), 1 deletion(-)
> > >  create mode 100644 arch/x86/configs/xen.config
> > >  create mode 100644 kernel/configs/xen.config
> > 
> > Who could these changes go through?
> 
> I would be OK with taking it in the Xen tree, but I would feel more
> comfortable doing that if you had an ack from Yann Morin (CC'ed).

*Poke*

  Luis

Re: [PATCH v3 0/2] x86/arm64: add xenconfig

[Luis R. Rodriguez]

On Tue, Jan 27, 2015 at 12:00 PM, Luis R. Rodriguez <mcgrof@suse.com> wrote:
> On Fri, Jan 23, 2015 at 03:19:25PM +0000, Stefano Stabellini wrote:
>> On Fri, 23 Jan 2015, Luis R. Rodriguez wrote:
>> > On Wed, Jan 14, 2015 at 11:33:45AM -0800, Luis R. Rodriguez wrote:
>> > > From: "Luis R. Rodriguez" <mcgrof@suse.com>
>> > >
>> > > This v3 addresses Stefano's feedback from the v2 series, namely
>> > > moving PCI stuff to x86 as its all x86 specific and also just
>> > > removing the CONFIG_TCG_XEN=m from the general config. To be
>> > > clear the changes from the v2 series are below.
>> > >
>> > > Luis R. Rodriguez (2):
>> > >   x86, platform, xen, kconfig: clarify kvmconfig is for kvm
>> > >   x86, arm, platform, xen, kconfig: add xen defconfig helper
>> > >
>> > >  arch/x86/configs/xen.config | 10 ++++++++++
>> > >  kernel/configs/xen.config   | 26 ++++++++++++++++++++++++++
>> > >  scripts/kconfig/Makefile    |  7 ++++++-
>> > >  3 files changed, 42 insertions(+), 1 deletion(-)
>> > >  create mode 100644 arch/x86/configs/xen.config
>> > >  create mode 100644 kernel/configs/xen.config
>> >
>> > Who could these changes go through?
>>
>> I would be OK with taking it in the Xen tree, but I would feel more
>> comfortable doing that if you had an ack from Yann Morin (CC'ed).
>
> *Poke*

Hey Yann, wondering if you had any feedback. Thanks.

 Luis

Re: [PATCH v3 0/2] x86/arm64: add xenconfig

[Paul Bolle]

[Added Michal. Removed Yann.]

On Thu, 2015-01-29 at 12:38 -0800, Luis R. Rodriguez wrote:
> On Tue, Jan 27, 2015 at 12:00 PM, Luis R. Rodriguez <mcgrof@suse.com> wrote:
> > On Fri, Jan 23, 2015 at 03:19:25PM +0000, Stefano Stabellini wrote:
> >> On Fri, 23 Jan 2015, Luis R. Rodriguez wrote:
> >> > On Wed, Jan 14, 2015 at 11:33:45AM -0800, Luis R. Rodriguez wrote:
> >> > > From: "Luis R. Rodriguez" <mcgrof@suse.com>
> >> > >
> >> > > This v3 addresses Stefano's feedback from the v2 series, namely
> >> > > moving PCI stuff to x86 as its all x86 specific and also just
> >> > > removing the CONFIG_TCG_XEN=m from the general config. To be
> >> > > clear the changes from the v2 series are below.
> >> > >
> >> > > Luis R. Rodriguez (2):
> >> > >   x86, platform, xen, kconfig: clarify kvmconfig is for kvm
> >> > >   x86, arm, platform, xen, kconfig: add xen defconfig helper
> >> > >
> >> > >  arch/x86/configs/xen.config | 10 ++++++++++
> >> > >  kernel/configs/xen.config   | 26 ++++++++++++++++++++++++++
> >> > >  scripts/kconfig/Makefile    |  7 ++++++-
> >> > >  3 files changed, 42 insertions(+), 1 deletion(-)
> >> > >  create mode 100644 arch/x86/configs/xen.config
> >> > >  create mode 100644 kernel/configs/xen.config
> >> >
> >> > Who could these changes go through?
> >>
> >> I would be OK with taking it in the Xen tree, but I would feel more
> >> comfortable doing that if you had an ack from Yann Morin (CC'ed).
> >
> > *Poke*
> 
> Hey Yann, wondering if you had any feedback. Thanks.

Yann has disappeared a year ago. Michal now, informally, keeps an eye on
kconfig related patches.


Paul Bolle

Re: [PATCH v3 0/2] x86/arm64: add xenconfig

[Michal Marek]

On 2015-01-29 21:47, Paul Bolle wrote:
> [Added Michal. Removed Yann.]
> 
> On Thu, 2015-01-29 at 12:38 -0800, Luis R. Rodriguez wrote:
>> On Tue, Jan 27, 2015 at 12:00 PM, Luis R. Rodriguez <mcgrof@suse.com> wrote:
>>> On Fri, Jan 23, 2015 at 03:19:25PM +0000, Stefano Stabellini wrote:
>>>> On Fri, 23 Jan 2015, Luis R. Rodriguez wrote:
>>>>> On Wed, Jan 14, 2015 at 11:33:45AM -0800, Luis R. Rodriguez wrote:
>>>>>> From: "Luis R. Rodriguez" <mcgrof@suse.com>
>>>>>>
>>>>>> This v3 addresses Stefano's feedback from the v2 series, namely
>>>>>> moving PCI stuff to x86 as its all x86 specific and also just
>>>>>> removing the CONFIG_TCG_XEN=m from the general config. To be
>>>>>> clear the changes from the v2 series are below.
>>>>>>
>>>>>> Luis R. Rodriguez (2):
>>>>>>   x86, platform, xen, kconfig: clarify kvmconfig is for kvm
>>>>>>   x86, arm, platform, xen, kconfig: add xen defconfig helper
>>>>>>
>>>>>>  arch/x86/configs/xen.config | 10 ++++++++++
>>>>>>  kernel/configs/xen.config   | 26 ++++++++++++++++++++++++++
>>>>>>  scripts/kconfig/Makefile    |  7 ++++++-
>>>>>>  3 files changed, 42 insertions(+), 1 deletion(-)
>>>>>>  create mode 100644 arch/x86/configs/xen.config
>>>>>>  create mode 100644 kernel/configs/xen.config
>>>>>
>>>>> Who could these changes go through?
>>>>
>>>> I would be OK with taking it in the Xen tree, but I would feel more
>>>> comfortable doing that if you had an ack from Yann Morin (CC'ed).
>>>
>>> *Poke*
>>
>> Hey Yann, wondering if you had any feedback. Thanks.
> 
> Yann has disappeared a year ago. Michal now, informally, keeps an eye on
> kconfig related patches.

I did not see this. Can you please bounce me the original series?

Thanks,
Michal

Re: [PATCH v3 0/2] x86/arm64: add xenconfig

[Luis R. Rodriguez]

On Fri, Jan 30, 2015 at 2:49 AM, Michal Marek <mmarek@suse.cz> wrote:
> On 2015-01-29 21:47, Paul Bolle wrote:
>> [Added Michal. Removed Yann.]
>>
>> On Thu, 2015-01-29 at 12:38 -0800, Luis R. Rodriguez wrote:
>>> On Tue, Jan 27, 2015 at 12:00 PM, Luis R. Rodriguez <mcgrof@suse.com> wrote:
>>>> On Fri, Jan 23, 2015 at 03:19:25PM +0000, Stefano Stabellini wrote:
>>>>> On Fri, 23 Jan 2015, Luis R. Rodriguez wrote:
>>>>>> On Wed, Jan 14, 2015 at 11:33:45AM -0800, Luis R. Rodriguez wrote:
>>>>>>> From: "Luis R. Rodriguez" <mcgrof@suse.com>
>>>>>>>
>>>>>>> This v3 addresses Stefano's feedback from the v2 series, namely
>>>>>>> moving PCI stuff to x86 as its all x86 specific and also just
>>>>>>> removing the CONFIG_TCG_XEN=m from the general config. To be
>>>>>>> clear the changes from the v2 series are below.
>>>>>>>
>>>>>>> Luis R. Rodriguez (2):
>>>>>>>   x86, platform, xen, kconfig: clarify kvmconfig is for kvm
>>>>>>>   x86, arm, platform, xen, kconfig: add xen defconfig helper
>>>>>>>
>>>>>>>  arch/x86/configs/xen.config | 10 ++++++++++
>>>>>>>  kernel/configs/xen.config   | 26 ++++++++++++++++++++++++++
>>>>>>>  scripts/kconfig/Makefile    |  7 ++++++-
>>>>>>>  3 files changed, 42 insertions(+), 1 deletion(-)
>>>>>>>  create mode 100644 arch/x86/configs/xen.config
>>>>>>>  create mode 100644 kernel/configs/xen.config
>>>>>>
>>>>>> Who could these changes go through?
>>>>>
>>>>> I would be OK with taking it in the Xen tree, but I would feel more
>>>>> comfortable doing that if you had an ack from Yann Morin (CC'ed).
>>>>
>>>> *Poke*
>>>
>>> Hey Yann, wondering if you had any feedback. Thanks.
>>
>> Yann has disappeared a year ago. Michal now, informally, keeps an eye on
>> kconfig related patches.
>
> I did not see this. Can you please bounce me the original series?

Done, let me know if you did not get them, and sorry for not adding
you on the original To list, in no way did I consider this had to go
through you as it didn't include any major kconfig changes and though
this was rather more of an x86 / xen tree thing.

 Luis

Re: [PATCH v3 0/2] x86/arm64: add xenconfig

[Michal Marek]

Dne 30.1.2015 v 19:25 Luis R. Rodriguez napsal(a):
> On Fri, Jan 30, 2015 at 2:49 AM, Michal Marek <mmarek@suse.cz> wrote:
>> On 2015-01-29 21:47, Paul Bolle wrote:
>>> [Added Michal. Removed Yann.]
>>>
>>> On Thu, 2015-01-29 at 12:38 -0800, Luis R. Rodriguez wrote:
>>>> On Tue, Jan 27, 2015 at 12:00 PM, Luis R. Rodriguez <mcgrof@suse.com> wrote:
>>>>> On Fri, Jan 23, 2015 at 03:19:25PM +0000, Stefano Stabellini wrote:
>>>>>> On Fri, 23 Jan 2015, Luis R. Rodriguez wrote:
>>>>>>> On Wed, Jan 14, 2015 at 11:33:45AM -0800, Luis R. Rodriguez wrote:
>>>>>>>> From: "Luis R. Rodriguez" <mcgrof@suse.com>
>>>>>>>>
>>>>>>>> This v3 addresses Stefano's feedback from the v2 series, namely
>>>>>>>> moving PCI stuff to x86 as its all x86 specific and also just
>>>>>>>> removing the CONFIG_TCG_XEN=m from the general config. To be
>>>>>>>> clear the changes from the v2 series are below.
>>>>>>>>
>>>>>>>> Luis R. Rodriguez (2):
>>>>>>>>   x86, platform, xen, kconfig: clarify kvmconfig is for kvm
>>>>>>>>   x86, arm, platform, xen, kconfig: add xen defconfig helper
>>>>>>>>
>>>>>>>>  arch/x86/configs/xen.config | 10 ++++++++++
>>>>>>>>  kernel/configs/xen.config   | 26 ++++++++++++++++++++++++++
>>>>>>>>  scripts/kconfig/Makefile    |  7 ++++++-
>>>>>>>>  3 files changed, 42 insertions(+), 1 deletion(-)
>>>>>>>>  create mode 100644 arch/x86/configs/xen.config
>>>>>>>>  create mode 100644 kernel/configs/xen.config
>>>>>>>
>>>>>>> Who could these changes go through?
>>>>>>
>>>>>> I would be OK with taking it in the Xen tree, but I would feel more
>>>>>> comfortable doing that if you had an ack from Yann Morin (CC'ed).
>>>>>
>>>>> *Poke*
>>>>
>>>> Hey Yann, wondering if you had any feedback. Thanks.
>>>
>>> Yann has disappeared a year ago. Michal now, informally, keeps an eye on
>>> kconfig related patches.
>>
>> I did not see this. Can you please bounce me the original series?
> 
> Done, let me know if you did not get them

I got them, sorry for the delay. You can add Acked-by: Michal Marek
<mmarek@suse.cz> if you want.

Michal

Re: [PATCH v3 0/2] x86/arm64: add xenconfig

[Luis R. Rodriguez]

On Mon, Feb 2, 2015 at 1:13 PM, Michal Marek <mmarek@suse.cz> wrote:
> Dne 30.1.2015 v 19:25 Luis R. Rodriguez napsal(a):
>> On Fri, Jan 30, 2015 at 2:49 AM, Michal Marek <mmarek@suse.cz> wrote:
>>> On 2015-01-29 21:47, Paul Bolle wrote:
>>>> [Added Michal. Removed Yann.]
>>>>
>>>> On Thu, 2015-01-29 at 12:38 -0800, Luis R. Rodriguez wrote:
>>>>> On Tue, Jan 27, 2015 at 12:00 PM, Luis R. Rodriguez <mcgrof@suse.com> wrote:
>>>>>> On Fri, Jan 23, 2015 at 03:19:25PM +0000, Stefano Stabellini wrote:
>>>>>>> On Fri, 23 Jan 2015, Luis R. Rodriguez wrote:
>>>>>>>> On Wed, Jan 14, 2015 at 11:33:45AM -0800, Luis R. Rodriguez wrote:
>>>>>>>>> From: "Luis R. Rodriguez" <mcgrof@suse.com>
>>>>>>>>>
>>>>>>>>> This v3 addresses Stefano's feedback from the v2 series, namely
>>>>>>>>> moving PCI stuff to x86 as its all x86 specific and also just
>>>>>>>>> removing the CONFIG_TCG_XEN=m from the general config. To be
>>>>>>>>> clear the changes from the v2 series are below.
>>>>>>>>>
>>>>>>>>> Luis R. Rodriguez (2):
>>>>>>>>>   x86, platform, xen, kconfig: clarify kvmconfig is for kvm
>>>>>>>>>   x86, arm, platform, xen, kconfig: add xen defconfig helper
>>>>>>>>>
>>>>>>>>>  arch/x86/configs/xen.config | 10 ++++++++++
>>>>>>>>>  kernel/configs/xen.config   | 26 ++++++++++++++++++++++++++
>>>>>>>>>  scripts/kconfig/Makefile    |  7 ++++++-
>>>>>>>>>  3 files changed, 42 insertions(+), 1 deletion(-)
>>>>>>>>>  create mode 100644 arch/x86/configs/xen.config
>>>>>>>>>  create mode 100644 kernel/configs/xen.config
>>>>>>>>
>>>>>>>> Who could these changes go through?
>>>>>>>
>>>>>>> I would be OK with taking it in the Xen tree, but I would feel more
>>>>>>> comfortable doing that if you had an ack from Yann Morin (CC'ed).
>>>>>>
>>>>>> *Poke*
>>>>>
>>>>> Hey Yann, wondering if you had any feedback. Thanks.
>>>>
>>>> Yann has disappeared a year ago. Michal now, informally, keeps an eye on
>>>> kconfig related patches.
>>>
>>> I did not see this. Can you please bounce me the original series?
>>
>> Done, let me know if you did not get them
>
> I got them, sorry for the delay. You can add Acked-by: Michal Marek
> <mmarek@suse.cz> if you want.

Thanks Michal, what tree should this go through though?

 Luis

Re: [PATCH v3 0/2] x86/arm64: add xenconfig

[Luis R. Rodriguez]

On Mon, Feb 2, 2015 at 1:32 PM, Luis R. Rodriguez
<mcgrof@do-not-panic.com> wrote:
> On Mon, Feb 2, 2015 at 1:13 PM, Michal Marek <mmarek@suse.cz> wrote:
>> Dne 30.1.2015 v 19:25 Luis R. Rodriguez napsal(a):
>>> On Fri, Jan 30, 2015 at 2:49 AM, Michal Marek <mmarek@suse.cz> wrote:
>>>> On 2015-01-29 21:47, Paul Bolle wrote:
>>>>> [Added Michal. Removed Yann.]
>>>>>
>>>>> On Thu, 2015-01-29 at 12:38 -0800, Luis R. Rodriguez wrote:
>>>>>> On Tue, Jan 27, 2015 at 12:00 PM, Luis R. Rodriguez <mcgrof@suse.com> wrote:
>>>>>>> On Fri, Jan 23, 2015 at 03:19:25PM +0000, Stefano Stabellini wrote:
>>>>>>>> On Fri, 23 Jan 2015, Luis R. Rodriguez wrote:
>>>>>>>>> On Wed, Jan 14, 2015 at 11:33:45AM -0800, Luis R. Rodriguez wrote:
>>>>>>>>>> From: "Luis R. Rodriguez" <mcgrof@suse.com>
>>>>>>>>>>
>>>>>>>>>> This v3 addresses Stefano's feedback from the v2 series, namely
>>>>>>>>>> moving PCI stuff to x86 as its all x86 specific and also just
>>>>>>>>>> removing the CONFIG_TCG_XEN=m from the general config. To be
>>>>>>>>>> clear the changes from the v2 series are below.
>>>>>>>>>>
>>>>>>>>>> Luis R. Rodriguez (2):
>>>>>>>>>>   x86, platform, xen, kconfig: clarify kvmconfig is for kvm
>>>>>>>>>>   x86, arm, platform, xen, kconfig: add xen defconfig helper
>>>>>>>>>>
>>>>>>>>>>  arch/x86/configs/xen.config | 10 ++++++++++
>>>>>>>>>>  kernel/configs/xen.config   | 26 ++++++++++++++++++++++++++
>>>>>>>>>>  scripts/kconfig/Makefile    |  7 ++++++-
>>>>>>>>>>  3 files changed, 42 insertions(+), 1 deletion(-)
>>>>>>>>>>  create mode 100644 arch/x86/configs/xen.config
>>>>>>>>>>  create mode 100644 kernel/configs/xen.config
>>>>>>>>>
>>>>>>>>> Who could these changes go through?
>>>>>>>>
>>>>>>>> I would be OK with taking it in the Xen tree, but I would feel more
>>>>>>>> comfortable doing that if you had an ack from Yann Morin (CC'ed).
>>>>>>>
>>>>>>> *Poke*
>>>>>>
>>>>>> Hey Yann, wondering if you had any feedback. Thanks.
>>>>>
>>>>> Yann has disappeared a year ago. Michal now, informally, keeps an eye on
>>>>> kconfig related patches.
>>>>
>>>> I did not see this. Can you please bounce me the original series?
>>>
>>> Done, let me know if you did not get them
>>
>> I got them, sorry for the delay. You can add Acked-by: Michal Marek
>> <mmarek@suse.cz> if you want.
>
> Thanks Michal, what tree should this go through though?

David should this go through your tree? Can anyone provide pointers?

 Luis

Re: [PATCH v3 0/2] x86/arm64: add xenconfig

[David Rientjes]

On Fri, 6 Feb 2015, Luis R. Rodriguez wrote:

> David should this go through your tree? Can anyone provide pointers?
> 

I think it should probably go through Michal's tree?

It would help to have an ack from at least one of the x86 maintainers, so 
maybe add Ingo Molnar <mingo@redhat.com> and Thomas Gleixner 
<tglx@linutronix.de>.

We need an update to the MAINTAINERS file if "Yann E. MORIN" 
<yann.morin.1998@free.fr> isn't the active Kconfig maintainer anymore.

If all else fails, I'd try to ask Andrew Morton 
<akpm@linux-foundation.org> to route it.

Re: [PATCH v3 0/2] x86/arm64: add xenconfig

[Paul Bolle]

On Tue, 2015-02-10 at 14:21 -0800, David Rientjes wrote:
> We need an update to the MAINTAINERS file if "Yann E. MORIN" 
> <yann.morin.1998@free.fr> isn't the active Kconfig maintainer anymore.

Yes, we do. Michal, what update would you suggest?

Thanks,


Paul Bolle

Re: [PATCH v3 0/2] x86/arm64: add xenconfig

[Michal Marek]

On 2015-02-10 23:32, Paul Bolle wrote:
> On Tue, 2015-02-10 at 14:21 -0800, David Rientjes wrote:
>> We need an update to the MAINTAINERS file if "Yann E. MORIN" 
>> <yann.morin.1998@free.fr> isn't the active Kconfig maintainer anymore.
> 
> Yes, we do. Michal, what update would you suggest?

I'll revert the patch that added Yann. That is, change the contact back
to me and the status to "Odd fixes".

Michal

Re: [PATCH v3 0/2] x86/arm64: add xenconfig

[David Rientjes]

On Wed, 25 Feb 2015, Michal Marek wrote:

> > On Tue, 2015-02-10 at 14:21 -0800, David Rientjes wrote:
> >> We need an update to the MAINTAINERS file if "Yann E. MORIN" 
> >> <yann.morin.1998@free.fr> isn't the active Kconfig maintainer anymore.
> > 
> > Yes, we do. Michal, what update would you suggest?
> 
> I'll revert the patch that added Yann. That is, change the contact back
> to me and the status to "Odd fixes".
> 

Woohoo, so does this mean that Luis's series will finally be merged into a 
tree somewhere?   It's been a lengthy wait to try to get this merged.