[PATCH 1/3] linux-yocto: Enable debug/printk.scc KERNEL_FEATURE on qemuall machines.

[PATCH 1/3] linux-yocto: Enable debug/printk.scc KERNEL_FEATURE on qemuall machines.

[Aníbal Limón]

We are experimenting some issues in the Autobuilder infraestructure
possible due to high I/O loads, in order to provide more information
about intervals of times in printk enable by default debug/printk.scc
on qemu development images.

[YOCTO #9299]

Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
---
 meta/recipes-kernel/linux/linux-yocto.inc | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/recipes-kernel/linux/linux-yocto.inc b/meta/recipes-kernel/linux/linux-yocto.inc
index 0af9f8c..98a48ec 100644
--- a/meta/recipes-kernel/linux/linux-yocto.inc
+++ b/meta/recipes-kernel/linux/linux-yocto.inc
@@ -17,6 +17,8 @@ DEPENDS_append_nios2 = " libgcc"
 KERNEL_CC_append_nios2 = " ${TOOLCHAIN_OPTIONS}"
 KERNEL_LD_append_nios2 = " ${TOOLCHAIN_OPTIONS}"
 
+KERNEL_FEATURES_append_qemuall=" features/debug/printk.scc"
+
 # A KMACHINE is the mapping of a yocto $MACHINE to what is built
 # by the kernel. This is typically the branch that should be built,
 # and it can be specific to the machine or shared
-- 
2.1.4

[PATCH 2/3] utils/qemurunner.py: QemuRunner.start() add support for specify extra kernel cmdline

[Aníbal Limón]

Add ability to specify extra_bootargs (kernel cmdline) in order to enable systemd
debug log in images that enables systemd init.

[YOCTO #9299]

Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
---
 meta/lib/oeqa/targetcontrol.py    | 4 ++--
 meta/lib/oeqa/utils/qemurunner.py | 8 ++++++--
 2 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/meta/lib/oeqa/targetcontrol.py b/meta/lib/oeqa/targetcontrol.py
index 1c57efa..d3b49b7 100644
--- a/meta/lib/oeqa/targetcontrol.py
+++ b/meta/lib/oeqa/targetcontrol.py
@@ -174,8 +174,8 @@ class QemuTarget(BaseTarget):
         bb.note("Qemu log file: %s" % self.qemulog)
         super(QemuTarget, self).deploy()
 
-    def start(self, params=None, ssh=True):
-        if self.runner.start(params, get_ip=ssh):
+    def start(self, params=None, ssh=True, extra_bootparams=None):
+        if self.runner.start(params, get_ip=ssh, extra_bootparams=extra_bootparams):
             if ssh:
                 self.ip = self.runner.ip
                 self.server_ip = self.runner.server_ip
diff --git a/meta/lib/oeqa/utils/qemurunner.py b/meta/lib/oeqa/utils/qemurunner.py
index b8ac3f0..df73120 100644
--- a/meta/lib/oeqa/utils/qemurunner.py
+++ b/meta/lib/oeqa/utils/qemurunner.py
@@ -92,7 +92,7 @@ class QemuRunner:
                 self._dump_host()
                 raise SystemExit
 
-    def start(self, qemuparams = None, get_ip = True):
+    def start(self, qemuparams = None, get_ip = True, extra_bootparams = None):
         if self.display:
             os.environ["DISPLAY"] = self.display
             # Set this flag so that Qemu doesn't do any grabs as SDL grabs
@@ -120,7 +120,11 @@ class QemuRunner:
             return False
 
 
-        self.qemuparams = 'bootparams="console=tty1 console=ttyS0,115200n8 printk.time=1" qemuparams="-serial tcp:127.0.0.1:{}"'.format(threadport)
+        bootparams = 'console=tty1 console=ttyS0,115200n8 printk.time=1'
+        if extra_bootparams:
+            bootparams = bootparams + ' ' + extra_bootparams
+
+        self.qemuparams = 'bootparams="{0}" qemuparams="-serial tcp:127.0.0.1:{1}"'.format(bootparams, threadport)
         if not self.display:
             self.qemuparams = 'nographic ' + self.qemuparams
         if qemuparams:
-- 
2.1.4

[PATCH 3/3] classes/testimage: When image is systemd enable debug log level

[Aníbal Limón]

In order to get more information about systemd boot process to
be able to debug random failures due to high I/O.

[YOCTO #9299]

Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
---
 meta/classes/testimage.bbclass | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/meta/classes/testimage.bbclass b/meta/classes/testimage.bbclass
index 7fd0f62..f1b54d7 100644
--- a/meta/classes/testimage.bbclass
+++ b/meta/classes/testimage.bbclass
@@ -147,7 +147,10 @@ def testimage_main(d):
     tc.extract_packages()
     target.deploy()
     try:
-        target.start()
+        bootparams = None
+        if d.getVar('VIRTUAL-RUNTIME_init_manager', '') == 'systemd':
+            bootparams = 'systemd.log_level=debug systemd.log_target=console'
+        target.start(extra_bootparams=bootparams)
         starttime = time.time()
         result = tc.runTests()
         stoptime = time.time()
-- 
2.1.4

Re: [PATCH 3/3] classes/testimage: When image is systemd enable debug log level

[Burton, Ross]

[-- Attachment #1: Type: text/plain, Size: 596 bytes --]

On 12 July 2016 at 22:29, Aníbal Limón <anibal.limon@linux.intel.com> wrote:

> +            bootparams = 'systemd.log_level=debug systemd
> .log_target=console'
>

The problem with this is that it appears that logger output then goes
*only* to the console and not the journal or syslog, which breaks selftest
(and expected behaviour).

https://autobuilder.yoctoproject.org/main/builders/nightly-qa-systemd/builds/851
for example, which goes green if I revert this patch.

If this is the case then it would be great if there was a
journal-and-console log_target I guess.

Ross

[-- Attachment #2: Type: text/html, Size: 1318 bytes --]

Re: [PATCH 3/3] classes/testimage: When image is systemd enable debug log level

[Aníbal Limón]

[-- Attachment #1: Type: text/plain, Size: 922 bytes --]



On 07/15/2016 02:34 PM, Burton, Ross wrote:
> On 12 July 2016 at 22:29, Aníbal Limón <anibal.limon@linux.intel.com> wrote:
> 
>> +            bootparams = 'systemd.log_level=debug systemd
>> .log_target=console'
>>
> 
> The problem with this is that it appears that logger output then goes
> *only* to the console and not the journal or syslog, which breaks selftest
> (and expected behaviour).
> 
> https://autobuilder.yoctoproject.org/main/builders/nightly-qa-systemd/builds/851
> for example, which goes green if I revert this patch.
> 
> If this is the case then it would be great if there was a
> journal-and-console log_target I guess.

I'll review the issue i guess the journal test is failing for something
else but if we change the backend to journalctl systemd starts to fail
with timeout, seems that create a log of I/O writing in the journal
database.

	alimon

> 
> Ross
> 


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

Re: [PATCH 3/3] classes/testimage: When image is systemd enable debug log level

[Aníbal Limón]

[-- Attachment #1: Type: text/plain, Size: 1301 bytes --]



On 07/25/2016 12:21 PM, Aníbal Limón wrote:
> 
> 
> On 07/15/2016 02:34 PM, Burton, Ross wrote:
>> On 12 July 2016 at 22:29, Aníbal Limón <anibal.limon@linux.intel.com> wrote:
>>
>>> +            bootparams = 'systemd.log_level=debug systemd
>>> .log_target=console'
>>>
>>
>> The problem with this is that it appears that logger output then goes
>> *only* to the console and not the journal or syslog, which breaks selftest
>> (and expected behaviour).
>>
>> https://autobuilder.yoctoproject.org/main/builders/nightly-qa-systemd/builds/851
>> for example, which goes green if I revert this patch.
>>
>> If this is the case then it would be great if there was a
>> journal-and-console log_target I guess.
> 
> I'll review the issue i guess the journal test is failing for something
> else but if we change the backend to journalctl systemd starts to fail
> with timeout, seems that create a log of I/O writing in the journal
> database.
> 
> 	alimon

Reviewing the master branch without my patch seems that the issue for
don't exist /var/log/messages is an old one because the file isn't
created. The timeout issue was caused for the reasons that we now (I/O
load) but the syslog failure occurs without my patch also.

	alimon

> 
>>
>> Ross
>>
> 
> 
> 


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]