Messages in this thread | | | From | Chris Mason <> | Subject | Re: Boot failure on Arndale with next-20131105 | Date | Tue, 5 Nov 2013 19:04:32 -0500 |
| |
Quoting Olof Johansson (2013-11-05 17:41:42) > On Tue, Nov 5, 2013 at 2:06 PM, Stephen Warren <swarren@wwwdotorg.org> wrote: > > On 11/05/2013 01:56 PM, Chris Mason wrote: > >> Quoting Olof Johansson (2013-11-05 15:38:33) > >>> On Tue, Nov 5, 2013 at 12:33 PM, Chris Mason <chris.mason@fusionio.com> wrote: > >>>> Quoting Olof Johansson (2013-11-05 15:23:51) > >>>>> On Tue, Nov 5, 2013 at 11:33 AM, Jens Axboe <axboe@kernel.dk> wrote: > >>>> > >>>> [ horrible crashes fixed by removing my patch ] > >>>> > > ... > >> Ok, I think I see it. My guess is that you're hitting bounce buffers. > >> __blk_queue_bounce is the only caller of the bio splitting code I can > >> see that you might be hitting. > >> > >> My first patch exposed a lurking bug in bio_clone_biovec. Basically > >> bio->bi_vcnt is being doubled instead of initialized. > >> > >> This patch is only compile tested, but I think it'll fix it. > > > > Tested-by: Stephen Warren <swarren@nvidia.com> > > (this fixes the issue on Tegra30/Beaver at least) > > Tested-by: Olof Johansson <olof@lixom.net> > > This resolves boot failures on: > * Tegra30/beaver > * OMAP4/panda > * i.MX6/wandboard > > Thanks Chris!
Perfect, thanks for bisecting and trying the patches. Kent, if things get rebased, could you please fold this patch and my bi_vcnt patch in?
-chris
| |