Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 5 Dec 2013 10:25:56 +0200
From:      Konstantin Belousov <kostikbel@gmail.com>
To:        Paolo Pinto <paolo.pinto@netasq.com>, nicolas dumont <nicolas.dumont@netasq.com>, freebsd-geom@freebsd.org
Subject:   Re: geom_uzip, panic: bio_length in mdstart_vnode()
Message-ID:  <20131205082556.GA59496@kib.kiev.ua>
In-Reply-To: <20131205020742.GU55638@funkthat.com>
References:  <529F2748.2060900@netasq.com> <20131204162029.GV59496@kib.kiev.ua> <20131205020742.GU55638@funkthat.com>

next in thread | previous in thread | raw e-mail | index | archive | help

[-- Attachment #1 --]
On Wed, Dec 04, 2013 at 06:07:43PM -0800, John-Mark Gurney wrote:
> Konstantin Belousov wrote this message on Wed, Dec 04, 2013 at 18:20 +0200:
> > On Wed, Dec 04, 2013 at 01:59:52PM +0100, Paolo Pinto wrote:
> > > Hi list!
> > > 
> > > My kernel is compiled with option INVARIANTS and I get a reproducible
> > > kernel panic when trying to read data from a GEOM based compressed
> > > memory disk:
> > > 
> > > Unread portion of the kernel message buffer:
> > > panic: bio_length 140288
> > > cpuid = 3
> > > KDB: stack backtrace:
> > > #0 0xffffffff80909726 at kdb_backtrace+0x66
> > > #1 0xffffffff808d0fa8 at panic+0x1d8
> > > #2 0xffffffff80595949 at mdstart_vnode+0x619
> > 
> > The issue is that geom_uzip creates bios which are larger than MAXPHYS.
> 
> Shouldn't geom_uzip be fixed to honor MAXPHYS?
By all means, fix it.

> 
> > As a workaround, the following patch should be enough.  It only fires
> > assert when md really uses pbuf, and since geom_uzip knows nothing
> > about unmapped bio, the assertion must not trigger.
> 
> Ummm... what's the point of MAXPHYS if we allow IO larger than it?

This is rhetorical question, right ?

[-- Attachment #2 --]
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.22 (FreeBSD)

iQIcBAEBAgAGBQJSoDiTAAoJEJDCuSvBvK1BGcwP/Rocdfagi0MjGmqknHJC3Xg2
X/nG0tr6nY/sga7Hbsip5v/jwp9wt3Pfd3v1q6UQ2qPQgNR+Gbs+4XHnLuFf/Ap1
8YZRn7lvVJ/jEZY8OeM/srOOlEddC9oOPTTQ0XIgpP5z3rlSHiy+8ab+s+suS1X1
H25Fhr9gkNFwNWl489+oitzJldfjp0/7oY2jIaIy4jzTAiwCUuz8hBUbeVZxXM0Y
K8KGlsWdgSRWh9JmPKKTr6ycGYxwKkysktwQ+fKXRtm8j/3pQ1fJ11vRLDopDM8d
+oj+s075rzylrcLlrrLiUtOZ6ykTZ7RQ3YPQkw5AikbBtFq2xP/jyQRpNKahzED4
vY5FhbH9OFuURb9n2AAtv0YWy/sCl2SmZLI3CkmmhVE8IBLdB/fWBUFGCwhNeyHZ
6J8ZU3jAXoFGcXfEzkJYULYnnPyrOeFp2TJuogjHRA/AUDwoWG5uDjtzjyKLJYBj
WY5NiDCn8CpLACX1xw3VjHUBP2Y2OTITUMl5gaicQWd7YzORae1Lf6y3YkzTj627
s0mHwnSGVYvODhuQTav3NwOTitwEuf49q8QrpQ3QDX2ylSnd/WMF3Vf3nuA8pxTk
jmNl58/mXSAl7i5ynv3b1+H3vbx2qYjny9nhvKFKwDb3VZVpJS3dUP8/QRK1/H2T
OJLBJlwJdEyaVZv0m/ST
=MGua
-----END PGP SIGNATURE-----

Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20131205082556.GA59496>