Re: 9pfs: Twalk crash

2021-09-01 Thread Greg Kurz
On Wed, 01 Sep 2021 18:07:39 +0200 Christian Schoenebeck wrote: > On Mittwoch, 1. September 2021 17:41:02 CEST Greg Kurz wrote: > > On Wed, 01 Sep 2021 16:21:06 +0200 > > > > Christian Schoenebeck wrote: > > > On Mittwoch, 1. September 2021 14:49:37 CEST Christian Schoenebeck wrote: > > > > > >

Re: 9pfs: Twalk crash

2021-09-01 Thread Christian Schoenebeck
On Mittwoch, 1. September 2021 17:41:02 CEST Greg Kurz wrote: > On Wed, 01 Sep 2021 16:21:06 +0200 > > Christian Schoenebeck wrote: > > On Mittwoch, 1. September 2021 14:49:37 CEST Christian Schoenebeck wrote: > > > > > And it triggered, however I am not sure if some of those functions I > > > >

Re: 9pfs: Twalk crash

2021-09-01 Thread Greg Kurz
On Wed, 01 Sep 2021 16:21:06 +0200 Christian Schoenebeck wrote: > On Mittwoch, 1. September 2021 14:49:37 CEST Christian Schoenebeck wrote: > > > > And it triggered, however I am not sure if some of those functions I > > > > asserted above are indeed allowed to be executed on a different thread >

Re: 9pfs: Twalk crash

2021-09-01 Thread Christian Schoenebeck
On Mittwoch, 1. September 2021 14:49:37 CEST Christian Schoenebeck wrote: > > > And it triggered, however I am not sure if some of those functions I > > > asserted above are indeed allowed to be executed on a different thread > > > than main thread: > > > > > > Program terminated with signal SIGAB

Re: 9pfs: Twalk crash

2021-09-01 Thread Christian Schoenebeck
On Dienstag, 31. August 2021 19:04:54 CEST Greg Kurz wrote: > > Ok, I made a quick "is same thread" assertion check: > > > > diff --git a/hw/9pfs/virtio-9p-device.c b/hw/9pfs/virtio-9p-device.c > > index 54ee93b71f..bb6ebd16aa 100644 > > --- a/hw/9pfs/virtio-9p-device.c > > +++ b/hw/9pfs/virtio-9p

Re: 9pfs: Twalk crash

2021-08-31 Thread Greg Kurz
On Tue, 31 Aug 2021 17:00:32 +0200 Christian Schoenebeck wrote: > On Dienstag, 31. August 2021 12:57:49 CEST Greg Kurz wrote: > > On Mon, 30 Aug 2021 17:55:04 +0200 > > > > Christian Schoenebeck wrote: > > > Apparently commit 8d6cb100731c4d28535adbf2a3c2d1f29be3fef4 '9pfs: reduce > > > latency

Re: 9pfs: Twalk crash

2021-08-31 Thread Christian Schoenebeck
On Dienstag, 31. August 2021 12:57:49 CEST Greg Kurz wrote: > On Mon, 30 Aug 2021 17:55:04 +0200 > > Christian Schoenebeck wrote: > > Apparently commit 8d6cb100731c4d28535adbf2a3c2d1f29be3fef4 '9pfs: reduce > > latency of Twalk' has introduced occasional crashes. > > > > My first impression afte

Re: 9pfs: Twalk crash

2021-08-31 Thread Greg Kurz
On Mon, 30 Aug 2021 17:55:04 +0200 Christian Schoenebeck wrote: > Apparently commit 8d6cb100731c4d28535adbf2a3c2d1f29be3fef4 '9pfs: reduce > latency of Twalk' has introduced occasional crashes. > > My first impression after looking at the backtrace: looks like the patch > itself is probably no

9pfs: Twalk crash

2021-08-30 Thread Christian Schoenebeck
Apparently commit 8d6cb100731c4d28535adbf2a3c2d1f29be3fef4 '9pfs: reduce latency of Twalk' has introduced occasional crashes. My first impression after looking at the backtrace: looks like the patch itself is probably not causing this, but rather unmasked this issue (i.e. increased the chance t