| Message ID | 859a1c176765dbb9ea394dde387b8ed072365ca8.1689180344.git.pav@iki.fi (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [BlueZ,1/2] test-runner: set non-quiet printk before running tests | expand |
| Context | Check | Description |
|---|---|---|
| tedd_an/pre-ci_am | success | Success |
| tedd_an/CheckPatch | success | CheckPatch PASS |
| tedd_an/GitLint | success | Gitlint PASS |
| tedd_an/BuildEll | success | Build ELL PASS |
| tedd_an/BluezMake | success | Bluez Make PASS |
| tedd_an/MakeCheck | success | Bluez Make Check PASS |
| tedd_an/MakeDistcheck | success | Make Distcheck PASS |
| tedd_an/CheckValgrind | success | Check Valgrind PASS |
| tedd_an/CheckSmatch | success | CheckSparse PASS |
| tedd_an/bluezmakeextell | success | Make External ELL PASS |
| tedd_an/IncrementalBuild | success | Incremental Build PASS |
| tedd_an/ScanBuild | warning | ScanBuild: tools/test-runner.c:941:2: warning: 2nd function call argument is an uninitialized value printf("Running command %s\n", cmdname ? cmdname : argv[0]); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 1 warning generated. |
Hi Pauli, On Wed, Jul 12, 2023 at 10:00 AM Pauli Virtanen <pav@iki.fi> wrote: > > It is useful to see WARN_ON/bt_dev_err messages when running the tests. > Enable non-quiet printk levels after boot, so that it is on by default > and doesn't need to be handled in local test scripts. > --- > > Notes: > It could be useful to also check for BUG/WARNING in the bluez test bot. > > tools/test-runner.c | 16 ++++++++++++++++ > 1 file changed, 16 insertions(+) > > diff --git a/tools/test-runner.c b/tools/test-runner.c > index d74bb1087..288901a61 100644 > --- a/tools/test-runner.c > +++ b/tools/test-runner.c > @@ -136,6 +136,20 @@ static const char *config_table[] = { > NULL > }; > > +static void enable_printk(void) > +{ > + FILE *f; > + > + /* Set non-quiet printk level */ > + f = fopen("/proc/sys/kernel/printk", "w"); > + if (!f) { > + perror("Failed to set printk"); > + return; > + } > + fprintf(f, "7 4 1 7"); Can you have a comment on what this 7 4 1 7 is for? > + fclose(f); > +} > + > static void prepare_sandbox(void) > { > int i; > @@ -181,6 +195,8 @@ static void prepare_sandbox(void) > "mode=0755") < 0) > perror("Failed to create filesystem"); > } > + > + enable_printk(); > } > > static char *const qemu_argv[] = { > -- > 2.41.0 >
This is automated email and please do not reply to this email!
Dear submitter,
Thank you for submitting the patches to the linux bluetooth mailing list.
This is a CI test results with your patch series:
PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=764973
---Test result---
Test Summary:
CheckPatch PASS 0.97 seconds
GitLint PASS 0.65 seconds
BuildEll PASS 26.62 seconds
BluezMake PASS 771.59 seconds
MakeCheck PASS 11.94 seconds
MakeDistcheck PASS 155.03 seconds
CheckValgrind PASS 250.24 seconds
CheckSmatch PASS 335.90 seconds
bluezmakeextell PASS 101.43 seconds
IncrementalBuild PASS 1295.86 seconds
ScanBuild WARNING 1024.62 seconds
Details
##############################
Test: ScanBuild - WARNING
Desc: Run Scan Build
Output:
tools/test-runner.c:941:2: warning: 2nd function call argument is an uninitialized value
printf("Running command %s\n", cmdname ? cmdname : argv[0]);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 warning generated.
---
Regards,
Linux Bluetooth
diff --git a/tools/test-runner.c b/tools/test-runner.c index d74bb1087..288901a61 100644 --- a/tools/test-runner.c +++ b/tools/test-runner.c @@ -136,6 +136,20 @@ static const char *config_table[] = { NULL }; +static void enable_printk(void) +{ + FILE *f; + + /* Set non-quiet printk level */ + f = fopen("/proc/sys/kernel/printk", "w"); + if (!f) { + perror("Failed to set printk"); + return; + } + fprintf(f, "7 4 1 7"); + fclose(f); +} + static void prepare_sandbox(void) { int i; @@ -181,6 +195,8 @@ static void prepare_sandbox(void) "mode=0755") < 0) perror("Failed to create filesystem"); } + + enable_printk(); } static char *const qemu_argv[] = {
It is useful to see WARN_ON/bt_dev_err messages when running the tests. Enable non-quiet printk levels after boot, so that it is on by default and doesn't need to be handled in local test scripts. --- Notes: It could be useful to also check for BUG/WARNING in the bluez test bot. tools/test-runner.c | 16 ++++++++++++++++ 1 file changed, 16 insertions(+)