Patchwork qemux86: disable paravirt guest, causes test failures

login
register
mail settings
Submitter Ross Burton
Date May 7, 2013, 3:49 p.m.
Message ID <1367941787-8508-1-git-send-email-ross.burton@intel.com>
Download mbox | patch
Permalink /patch/49483/
State New
Headers show

Comments

Ross Burton - May 7, 2013, 3:49 p.m.
The shutdown sanity test that we previously saw on qemux86-64 in 2.6.37 have
started to re-appear reproducably on qemux86 in 3.4 in the Danny series.  Until
this can be root-caused and fixed in the kernel, disable paravirtualisation
across all kernels and all qemu machines.

[ YOCTO #4196 ]

Signed-off-by: Ross Burton <ross.burton@intel.com>
---
 meta/recipes-kernel/linux/linux-yocto-dev.bb    |    1 -
 meta/recipes-kernel/linux/linux-yocto-rt_3.8.bb |    2 +-
 meta/recipes-kernel/linux/linux-yocto_3.4.bb    |    1 -
 meta/recipes-kernel/linux/linux-yocto_3.8.bb    |    2 +-
 4 files changed, 2 insertions(+), 4 deletions(-)
Bruce Ashfield - May 7, 2013, 4:04 p.m.
On Tue, May 7, 2013 at 11:49 AM, Ross Burton <ross.burton@intel.com> wrote:
> The shutdown sanity test that we previously saw on qemux86-64 in 2.6.37 have
> started to re-appear reproducably on qemux86 in 3.4 in the Danny series.  Until
> this can be root-caused and fixed in the kernel, disable paravirtualisation
> across all kernels and all qemu machines.

Make sure to cc me on linux-yocto patches, I noticed this by chance.

>
> [ YOCTO #4196 ]
>
> Signed-off-by: Ross Burton <ross.burton@intel.com>
> ---
>  meta/recipes-kernel/linux/linux-yocto-dev.bb    |    1 -
>  meta/recipes-kernel/linux/linux-yocto-rt_3.8.bb |    2 +-
>  meta/recipes-kernel/linux/linux-yocto_3.4.bb    |    1 -
>  meta/recipes-kernel/linux/linux-yocto_3.8.bb    |    2 +-
>  4 files changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/meta/recipes-kernel/linux/linux-yocto-dev.bb b/meta/recipes-kernel/linux/linux-yocto-dev.bb
> index fdea085..fcf37cc 100644
> --- a/meta/recipes-kernel/linux/linux-yocto-dev.bb
> +++ b/meta/recipes-kernel/linux/linux-yocto-dev.bb
> @@ -44,5 +44,4 @@ COMPATIBLE_MACHINE = "(qemuarm|qemux86|qemuppc|qemumips|qemux86-64)"
>  KERNEL_FEATURES_append = " features/netfilter/netfilter.scc"
>  KERNEL_FEATURES_append_qemux86=" cfg/sound.scc"
>  KERNEL_FEATURES_append_qemux86-64=" cfg/sound.scc"
> -KERNEL_FEATURES_append_qemux86=" cfg/paravirt_kvm.scc"

I'd rather keep this on for the -dev kernel, it is development for a reason and
we should leave this on as a leading edge feature.

Bruce

>  KERNEL_FEATURES_append = " ${@bb.utils.contains("TUNE_FEATURES", "mx32", " cfg/x32.scc", "" ,d)}"
> diff --git a/meta/recipes-kernel/linux/linux-yocto-rt_3.8.bb b/meta/recipes-kernel/linux/linux-yocto-rt_3.8.bb
> index 980229c..38e35db 100644
> --- a/meta/recipes-kernel/linux/linux-yocto-rt_3.8.bb
> +++ b/meta/recipes-kernel/linux/linux-yocto-rt_3.8.bb
> @@ -25,6 +25,6 @@ COMPATIBLE_MACHINE = "(qemux86|qemux86-64|qemuarm)"
>  # Functionality flags
>  KERNEL_EXTRA_FEATURES ?= "features/netfilter/netfilter.scc features/taskstats/taskstats.scc"
>  KERNEL_FEATURES_append = " ${KERNEL_EXTRA_FEATURES}"
> -KERNEL_FEATURES_append_qemux86=" cfg/sound.scc cfg/paravirt_kvm.scc"
> +KERNEL_FEATURES_append_qemux86=" cfg/sound.scc"
>  KERNEL_FEATURES_append_qemux86-64=" cfg/sound.scc"
>  KERNEL_FEATURES_append = " ${@bb.utils.contains("TUNE_FEATURES", "mx32", " cfg/x32.scc", "" ,d)}"
> diff --git a/meta/recipes-kernel/linux/linux-yocto_3.4.bb b/meta/recipes-kernel/linux/linux-yocto_3.4.bb
> index b81b9ac..18d665a 100644
> --- a/meta/recipes-kernel/linux/linux-yocto_3.4.bb
> +++ b/meta/recipes-kernel/linux/linux-yocto_3.4.bb
> @@ -26,5 +26,4 @@ COMPATIBLE_MACHINE = "qemuarm|qemux86|qemuppc|qemumips|qemux86-64"
>  KERNEL_FEATURES_append = " features/netfilter/netfilter.scc"
>  KERNEL_FEATURES_append_qemux86=" cfg/sound.scc"
>  KERNEL_FEATURES_append_qemux86-64=" cfg/sound.scc"
> -KERNEL_FEATURES_append_qemux86=" cfg/paravirt_kvm.scc"
>  KERNEL_FEATURES_append = " ${@bb.utils.contains("TUNE_FEATURES", "mx32", " cfg/x32.scc", "" ,d)}"
> diff --git a/meta/recipes-kernel/linux/linux-yocto_3.8.bb b/meta/recipes-kernel/linux/linux-yocto_3.8.bb
> index 4bf6bf4..75e2dbb 100644
> --- a/meta/recipes-kernel/linux/linux-yocto_3.8.bb
> +++ b/meta/recipes-kernel/linux/linux-yocto_3.8.bb
> @@ -25,6 +25,6 @@ COMPATIBLE_MACHINE = "qemuarm|qemux86|qemuppc|qemumips|qemux86-64"
>  # Functionality flags
>  KERNEL_EXTRA_FEATURES ?= "features/netfilter/netfilter.scc"
>  KERNEL_FEATURES_append = " ${KERNEL_EXTRA_FEATURES}"
> -KERNEL_FEATURES_append_qemux86=" cfg/sound.scc cfg/paravirt_kvm.scc"
> +KERNEL_FEATURES_append_qemux86=" cfg/sound.scc"
>  KERNEL_FEATURES_append_qemux86-64=" cfg/sound.scc"
>  KERNEL_FEATURES_append = " ${@bb.utils.contains("TUNE_FEATURES", "mx32", " cfg/x32.scc", "" ,d)}"
> --
> 1.7.10.4
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core



--
"Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end"
Darren Hart - May 7, 2013, 4:21 p.m.
On 05/07/2013 09:04 AM, Bruce Ashfield wrote:
> On Tue, May 7, 2013 at 11:49 AM, Ross Burton <ross.burton@intel.com> wrote:
>> The shutdown sanity test that we previously saw on qemux86-64 in 2.6.37 have
>> started to re-appear reproducably on qemux86 in 3.4 in the Danny series.  Until
>> this can be root-caused and fixed in the kernel, disable paravirtualisation
>> across all kernels and all qemu machines.
> 
> Make sure to cc me on linux-yocto patches, I noticed this by chance.
> 
>>
>> [ YOCTO #4196 ]
>>
>> Signed-off-by: Ross Burton <ross.burton@intel.com>
>> ---
>>  meta/recipes-kernel/linux/linux-yocto-dev.bb    |    1 -
>>  meta/recipes-kernel/linux/linux-yocto-rt_3.8.bb |    2 +-
>>  meta/recipes-kernel/linux/linux-yocto_3.4.bb    |    1 -
>>  meta/recipes-kernel/linux/linux-yocto_3.8.bb    |    2 +-
>>  4 files changed, 2 insertions(+), 4 deletions(-)
>>
>> diff --git a/meta/recipes-kernel/linux/linux-yocto-dev.bb b/meta/recipes-kernel/linux/linux-yocto-dev.bb
>> index fdea085..fcf37cc 100644
>> --- a/meta/recipes-kernel/linux/linux-yocto-dev.bb
>> +++ b/meta/recipes-kernel/linux/linux-yocto-dev.bb
>> @@ -44,5 +44,4 @@ COMPATIBLE_MACHINE = "(qemuarm|qemux86|qemuppc|qemumips|qemux86-64)"
>>  KERNEL_FEATURES_append = " features/netfilter/netfilter.scc"
>>  KERNEL_FEATURES_append_qemux86=" cfg/sound.scc"
>>  KERNEL_FEATURES_append_qemux86-64=" cfg/sound.scc"
>> -KERNEL_FEATURES_append_qemux86=" cfg/paravirt_kvm.scc"
> 
> I'd rather keep this on for the -dev kernel, it is development for a reason and
> we should leave this on as a leading edge feature.

Agreed

> 
> Bruce
> 
>>  KERNEL_FEATURES_append = " ${@bb.utils.contains("TUNE_FEATURES", "mx32", " cfg/x32.scc", "" ,d)}"
>> diff --git a/meta/recipes-kernel/linux/linux-yocto-rt_3.8.bb b/meta/recipes-kernel/linux/linux-yocto-rt_3.8.bb
>> index 980229c..38e35db 100644
>> --- a/meta/recipes-kernel/linux/linux-yocto-rt_3.8.bb
>> +++ b/meta/recipes-kernel/linux/linux-yocto-rt_3.8.bb
>> @@ -25,6 +25,6 @@ COMPATIBLE_MACHINE = "(qemux86|qemux86-64|qemuarm)"
>>  # Functionality flags
>>  KERNEL_EXTRA_FEATURES ?= "features/netfilter/netfilter.scc features/taskstats/taskstats.scc"
>>  KERNEL_FEATURES_append = " ${KERNEL_EXTRA_FEATURES}"
>> -KERNEL_FEATURES_append_qemux86=" cfg/sound.scc cfg/paravirt_kvm.scc"
>> +KERNEL_FEATURES_append_qemux86=" cfg/sound.scc"
>>  KERNEL_FEATURES_append_qemux86-64=" cfg/sound.scc"
>>  KERNEL_FEATURES_append = " ${@bb.utils.contains("TUNE_FEATURES", "mx32", " cfg/x32.scc", "" ,d)}"
>> diff --git a/meta/recipes-kernel/linux/linux-yocto_3.4.bb b/meta/recipes-kernel/linux/linux-yocto_3.4.bb
>> index b81b9ac..18d665a 100644
>> --- a/meta/recipes-kernel/linux/linux-yocto_3.4.bb
>> +++ b/meta/recipes-kernel/linux/linux-yocto_3.4.bb
>> @@ -26,5 +26,4 @@ COMPATIBLE_MACHINE = "qemuarm|qemux86|qemuppc|qemumips|qemux86-64"
>>  KERNEL_FEATURES_append = " features/netfilter/netfilter.scc"
>>  KERNEL_FEATURES_append_qemux86=" cfg/sound.scc"
>>  KERNEL_FEATURES_append_qemux86-64=" cfg/sound.scc"
>> -KERNEL_FEATURES_append_qemux86=" cfg/paravirt_kvm.scc"
>>  KERNEL_FEATURES_append = " ${@bb.utils.contains("TUNE_FEATURES", "mx32", " cfg/x32.scc", "" ,d)}"
>> diff --git a/meta/recipes-kernel/linux/linux-yocto_3.8.bb b/meta/recipes-kernel/linux/linux-yocto_3.8.bb
>> index 4bf6bf4..75e2dbb 100644
>> --- a/meta/recipes-kernel/linux/linux-yocto_3.8.bb
>> +++ b/meta/recipes-kernel/linux/linux-yocto_3.8.bb
>> @@ -25,6 +25,6 @@ COMPATIBLE_MACHINE = "qemuarm|qemux86|qemuppc|qemumips|qemux86-64"
>>  # Functionality flags
>>  KERNEL_EXTRA_FEATURES ?= "features/netfilter/netfilter.scc"
>>  KERNEL_FEATURES_append = " ${KERNEL_EXTRA_FEATURES}"
>> -KERNEL_FEATURES_append_qemux86=" cfg/sound.scc cfg/paravirt_kvm.scc"
>> +KERNEL_FEATURES_append_qemux86=" cfg/sound.scc"


Have we seen the failure on 3.8? Or only on 3.4?
Ross Burton - May 7, 2013, 5:08 p.m.
On 7 May 2013 17:21, Darren Hart <dvhart@linux.intel.com> wrote:
> Have we seen the failure on 3.8? Or only on 3.4?

I've only seen it on 3.4 but it's only occasionally hitting so that
doesn't really say a huge amount.  I can re-send it for 3.4 and we can
continue to watch the AB  -- if it doesn't re-appear on 3.8 then we
can try re-enabling it for qemux86-64 too.

Ross

Patch

diff --git a/meta/recipes-kernel/linux/linux-yocto-dev.bb b/meta/recipes-kernel/linux/linux-yocto-dev.bb
index fdea085..fcf37cc 100644
--- a/meta/recipes-kernel/linux/linux-yocto-dev.bb
+++ b/meta/recipes-kernel/linux/linux-yocto-dev.bb
@@ -44,5 +44,4 @@  COMPATIBLE_MACHINE = "(qemuarm|qemux86|qemuppc|qemumips|qemux86-64)"
 KERNEL_FEATURES_append = " features/netfilter/netfilter.scc"
 KERNEL_FEATURES_append_qemux86=" cfg/sound.scc"
 KERNEL_FEATURES_append_qemux86-64=" cfg/sound.scc"
-KERNEL_FEATURES_append_qemux86=" cfg/paravirt_kvm.scc"
 KERNEL_FEATURES_append = " ${@bb.utils.contains("TUNE_FEATURES", "mx32", " cfg/x32.scc", "" ,d)}"
diff --git a/meta/recipes-kernel/linux/linux-yocto-rt_3.8.bb b/meta/recipes-kernel/linux/linux-yocto-rt_3.8.bb
index 980229c..38e35db 100644
--- a/meta/recipes-kernel/linux/linux-yocto-rt_3.8.bb
+++ b/meta/recipes-kernel/linux/linux-yocto-rt_3.8.bb
@@ -25,6 +25,6 @@  COMPATIBLE_MACHINE = "(qemux86|qemux86-64|qemuarm)"
 # Functionality flags
 KERNEL_EXTRA_FEATURES ?= "features/netfilter/netfilter.scc features/taskstats/taskstats.scc"
 KERNEL_FEATURES_append = " ${KERNEL_EXTRA_FEATURES}"
-KERNEL_FEATURES_append_qemux86=" cfg/sound.scc cfg/paravirt_kvm.scc"
+KERNEL_FEATURES_append_qemux86=" cfg/sound.scc"
 KERNEL_FEATURES_append_qemux86-64=" cfg/sound.scc"
 KERNEL_FEATURES_append = " ${@bb.utils.contains("TUNE_FEATURES", "mx32", " cfg/x32.scc", "" ,d)}"
diff --git a/meta/recipes-kernel/linux/linux-yocto_3.4.bb b/meta/recipes-kernel/linux/linux-yocto_3.4.bb
index b81b9ac..18d665a 100644
--- a/meta/recipes-kernel/linux/linux-yocto_3.4.bb
+++ b/meta/recipes-kernel/linux/linux-yocto_3.4.bb
@@ -26,5 +26,4 @@  COMPATIBLE_MACHINE = "qemuarm|qemux86|qemuppc|qemumips|qemux86-64"
 KERNEL_FEATURES_append = " features/netfilter/netfilter.scc"
 KERNEL_FEATURES_append_qemux86=" cfg/sound.scc"
 KERNEL_FEATURES_append_qemux86-64=" cfg/sound.scc"
-KERNEL_FEATURES_append_qemux86=" cfg/paravirt_kvm.scc"
 KERNEL_FEATURES_append = " ${@bb.utils.contains("TUNE_FEATURES", "mx32", " cfg/x32.scc", "" ,d)}"
diff --git a/meta/recipes-kernel/linux/linux-yocto_3.8.bb b/meta/recipes-kernel/linux/linux-yocto_3.8.bb
index 4bf6bf4..75e2dbb 100644
--- a/meta/recipes-kernel/linux/linux-yocto_3.8.bb
+++ b/meta/recipes-kernel/linux/linux-yocto_3.8.bb
@@ -25,6 +25,6 @@  COMPATIBLE_MACHINE = "qemuarm|qemux86|qemuppc|qemumips|qemux86-64"
 # Functionality flags
 KERNEL_EXTRA_FEATURES ?= "features/netfilter/netfilter.scc"
 KERNEL_FEATURES_append = " ${KERNEL_EXTRA_FEATURES}"
-KERNEL_FEATURES_append_qemux86=" cfg/sound.scc cfg/paravirt_kvm.scc"
+KERNEL_FEATURES_append_qemux86=" cfg/sound.scc"
 KERNEL_FEATURES_append_qemux86-64=" cfg/sound.scc"
 KERNEL_FEATURES_append = " ${@bb.utils.contains("TUNE_FEATURES", "mx32", " cfg/x32.scc", "" ,d)}"