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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.