From: Vincent Whitchurch <vincent.whitchurch@axis.com>
To: Jonathan Cameron <jic23@kernel.org>
Cc: "linux-iio@vger.kernel.org" <linux-iio@vger.kernel.org>,
Andy Shevchenko <andy.shevchenko@gmail.com>,
Peter Rosin <peda@axentia.se>,
Michael Hennerich <michael.hennerich@analog.com>,
Lars-Peter Clausen <lars@metafoo.de>,
Jonathan Cameron <Jonathan.Cameron@huawei.com>
Subject: Re: [PATCH v2 00/17] staging/iio: Clean up AD7746 CDC driver and move from staging.
Date: Wed, 22 Jun 2022 15:07:54 +0200 [thread overview]
Message-ID: <20220622130754.GA3639@axis.com> (raw)
In-Reply-To: <20220619185839.1363503-1-jic23@kernel.org>
On Sun, Jun 19, 2022 at 08:58:22PM +0200, Jonathan Cameron wrote:
> Vincent: I'm getting some instability with roadtest after forwards porting to
> current mainline. Tests run 'sometimes'. Other times I get a crash
> in um_set_signal. Seems unrelated to the test this series adds.
My local work-in-progress version has a lot of changes since the RFC,
but yes, I'm seeing a splat too (below) if I use the RFC version on
v5.19-rc3. The problem goes away if I enable CONFIG_UML_RANDOM=y, as I
have done in my local branch:
diff --git a/tools/testing/roadtest/roadtest/tests/base/config b/tools/testing/roadtest/roadtest/tests/base/config
index c1952d047c8e..74b201c48d42 100644
--- a/tools/testing/roadtest/roadtest/tests/base/config
+++ b/tools/testing/roadtest/roadtest/tests/base/config
@@ -4,6 +4,7 @@ CONFIG_LOG_BUF_SHIFT=14
CONFIG_EXPERT=y
CONFIG_HOSTFS=y
CONFIG_UML_TIME_TRAVEL_SUPPORT=y
+CONFIG_UML_RANDOM=y
CONFIG_NULL_CHAN=y
CONFIG_PORT_CHAN=y
CONFIG_PTY_CHAN=y
[ 547.130000][ T21] BUG: failure at arch/um/kernel/time.c:415/time_travel_update_time()!
[ 547.130000][ T21] Kernel panic - not syncing: BUG!
[ 547.130000][ T21] CPU: 0 PID: 21 Comm: python3 Not tainted 5.19.0-rc3-00018-gcb2e63986f7f #77
[ 547.130000][ T21] Stack:
[ 547.130000][ T21] 605860dd 60290b07 60037130 00000000
[ 547.130000][ T21] 605860dd 00000000 607a39d0 60426253
[ 547.130000][ T21] 6064d1a8 00000000 604106ad 60557d16
[ 547.130000][ T21] Call Trace:
[ 547.130000][ T21] [<60290b07>] ? dump_stack_print_info+0xd7/0xf0
[ 547.130000][ T21] [<60037130>] ? um_set_signals+0x0/0x50
[ 547.130000][ T21] [<60426253>] ? dump_stack_lvl+0x5b/0x7a
[ 547.130000][ T21] [<604106ad>] ? _printk+0x0/0x9b
[ 547.130000][ T21] [<60025440>] ? time_travel_del_event+0x0/0xc0
[ 547.130000][ T21] [<6040b016>] ? panic+0x1a8/0x372
[ 547.130000][ T21] [<6040ae6e>] ? panic+0x0/0x372
[ 547.130000][ T21] [<604106ad>] ? _printk+0x0/0x9b
[ 547.130000][ T21] [<60037130>] ? um_set_signals+0x0/0x50
[ 547.130000][ T21] [<60037174>] ? um_set_signals+0x44/0x50
[ 547.130000][ T21] [<60025440>] ? time_travel_del_event+0x0/0xc0
[ 547.130000][ T21] [<60409c35>] ? time_travel_update_time.cold+0x6c/0x133
[ 547.130000][ T21] [<6043b5f0>] ? __schedule+0x780/0x880
[ 547.130000][ T21] [<6043ae70>] ? __schedule+0x0/0x880
[ 547.130000][ T21] [<600bd220>] ? random_get_entropy_fallback+0x0/0x30
[ 547.130000][ T21] [<6003be10>] ? get_fp_registers+0x0/0x80
[ 547.130000][ T21] [<600258fd>] ? timer_read+0xbd/0xf0
[ 547.130000][ T21] [<60305fb0>] ? mix_pool_bytes+0x0/0x60
[ 547.130000][ T21] [<60305fb0>] ? mix_pool_bytes+0x0/0x60
[ 547.130000][ T21] [<60430d32>] ? try_to_generate_entropy+0x14d/0x164
[ 547.130000][ T21] [<60430f98>] ? entropy_timer+0x0/0x48
[ 547.130000][ T21] [<60431049>] ? urandom_read_iter.cold+0xc/0x11
[ 547.130000][ T21] [<60176b52>] ? new_sync_read+0xe2/0x150
[ 547.130000][ T21] [<60178dc2>] ? vfs_read+0xf2/0x200
[ 547.130000][ T21] [<601a51d5>] ? __fdget_pos+0x15/0x60
[ 547.130000][ T21] [<601793a1>] ? ksys_read+0x61/0xf0
[ 547.130000][ T21] [<600280ca>] ? handle_syscall+0xaa/0xf0
[ 547.130000][ T21] [<60039236>] ? userspace+0x346/0x570
[ 547.130000][ T21] [<60037130>] ? um_set_signals+0x0/0x50
next prev parent reply other threads:[~2022-06-22 13:08 UTC|newest]
Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-06-19 18:58 [PATCH v2 00/17] staging/iio: Clean up AD7746 CDC driver and move from staging Jonathan Cameron
2022-06-19 18:58 ` [PATCH v2 01/17] iio: core: Increase precision of IIO_VAL_FRACTIONAL_LOG2 when possible Jonathan Cameron
2022-06-19 18:58 ` [PATCH v2 02/17] iio: ABI: Fix wrong format of differential capacitance channel ABI Jonathan Cameron
2022-06-19 22:58 ` Andy Shevchenko
2022-06-20 17:50 ` Jonathan Cameron
2022-06-19 18:58 ` [PATCH v2 03/17] staging: iio: cdc: ad7746: Use explicit be24 handling Jonathan Cameron
2022-06-19 18:58 ` [PATCH v2 04/17] staging: iio: cdc: ad7746: Push handling of supply voltage scale to userspace Jonathan Cameron
2022-06-19 18:58 ` [PATCH v2 05/17] staging: iio: cdc: ad7746: Use local buffer for multi byte reads Jonathan Cameron
2022-06-19 23:00 ` Andy Shevchenko
2022-06-20 16:42 ` Jonathan Cameron
2022-06-20 18:23 ` Jonathan Cameron
2022-06-19 18:58 ` [PATCH v2 06/17] staging: iio: cdc: ad7746: Factor out ad7746_read_channel() Jonathan Cameron
2022-06-19 18:58 ` [PATCH v2 07/17] staging: iio: cdc: ad7764: Push locking down into case statements in read/write_raw Jonathan Cameron
2022-06-19 18:58 ` [PATCH v2 08/17] staging: iio: cdc: ad7746: Break up use of chan->address and use FIELD_PREP etc Jonathan Cameron
2022-06-19 18:58 ` [PATCH v2 09/17] staging: iio: cdc: ad7746: Drop usused i2c_set_clientdata() Jonathan Cameron
2022-06-19 18:58 ` [PATCH v2 10/17] staging: iio: cdc: ad7746: Use _raw and _scale for temperature channels Jonathan Cameron
2022-06-19 18:58 ` [PATCH v2 11/17] iio: core: Introduce _inputoffset for differential channels Jonathan Cameron
2022-06-19 23:20 ` Andy Shevchenko
2022-06-20 18:13 ` Jonathan Cameron
2022-06-20 18:35 ` Andy Shevchenko
2022-06-20 20:01 ` Jonathan Cameron
2022-06-20 19:57 ` Jonathan Cameron
2022-06-19 18:58 ` [PATCH v2 12/17] staging: iio: cdc: ad7746: Switch from _offset to " Jonathan Cameron
2022-06-19 18:58 ` [PATCH v2 13/17] staging: iio: cdc: ad7746: Use read_avail() rather than opencoding Jonathan Cameron
2022-06-19 18:58 ` [PATCH v2 14/17] staging: iio: ad7746: White space cleanup Jonathan Cameron
2022-06-19 18:58 ` [PATCH v2 15/17] iio: cdc: ad7746: Add device specific ABI documentation Jonathan Cameron
2022-06-19 18:58 ` [PATCH v2 16/17] iio: cdc: ad7746: Move driver out of staging Jonathan Cameron
2022-06-19 23:11 ` Andy Shevchenko
2022-06-20 16:40 ` Jonathan Cameron
2022-06-20 18:30 ` Jonathan Cameron
2022-06-20 18:40 ` Andy Shevchenko
2022-06-20 19:53 ` Jonathan Cameron
2022-06-20 20:01 ` Andy Shevchenko
2022-06-19 18:58 ` [PATCH v2 17/17] RFC: iio: cdc: ad7746: Add roadtest Jonathan Cameron
2022-06-19 23:23 ` [PATCH v2 00/17] staging/iio: Clean up AD7746 CDC driver and move from staging Andy Shevchenko
2022-06-20 18:07 ` Jonathan Cameron
2022-06-21 14:34 ` Joe Simmons-Talbott
2022-06-22 13:07 ` Vincent Whitchurch [this message]
2022-06-26 11:07 ` Jonathan Cameron
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20220622130754.GA3639@axis.com \
--to=vincent.whitchurch@axis.com \
--cc=Jonathan.Cameron@huawei.com \
--cc=andy.shevchenko@gmail.com \
--cc=jic23@kernel.org \
--cc=lars@metafoo.de \
--cc=linux-iio@vger.kernel.org \
--cc=michael.hennerich@analog.com \
--cc=peda@axentia.se \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).