Quoting Olof Johansson (2013-11-05 17:41:42) > On Tue, Nov 5, 2013 at 2:06 PM, Stephen Warren <swarren@xxxxxxxxxxxxx> 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@xxxxxxxxxxxx> wrote: > >>>> Quoting Olof Johansson (2013-11-05 15:23:51) > >>>>> On Tue, Nov 5, 2013 at 11:33 AM, Jens Axboe <axboe@xxxxxxxxx> 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@xxxxxxxxxx> > > (this fixes the issue on Tegra30/Beaver at least) > > Tested-by: Olof Johansson <olof@xxxxxxxxx> > > 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 -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
