Date: Tue, 8 May 2012 12:56:31 +0300 From: Konstantin Belousov <kostikbel@gmail.com> To: Grzegorz Bernacki <gber@freebsd.org> Cc: svn-src-projects@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r233072 - projects/nand/sys/kern Message-ID: <20120508095631.GV2358@deviant.kiev.zoral.com.ua> In-Reply-To: <4FA8FFB9.7090002@freebsd.org> References: <201203170318.q2H3ITdI047893@svn.freebsd.org> <20120317085116.GC1340@garage.freebsd.pl> <20120317161050.GI75778@deviant.kiev.zoral.com.ua> <4FA8FFB9.7090002@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--G+2xtMkjVKN5mon6 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, May 08, 2012 at 01:12:57PM +0200, Grzegorz Bernacki wrote: > On 03/17/12 17:10, Konstantin Belousov wrote: > >On Sat, Mar 17, 2012 at 09:51:16AM +0100, Pawel Jakub Dawidek wrote: > >>On Sat, Mar 17, 2012 at 03:18:29AM +0000, Grzegorz Bernacki wrote: > >>>Author: gber > >>>Date: Sat Mar 17 03:18:28 2012 > >>>New Revision: 233072 > >>>URL: http://svn.freebsd.org/changeset/base/233072 > >>> > >>>Log: > >>> Add VFS changes necessary for NANDFS to work. > >>> > >>> Ignore B_MANAGED buffer by syncer and ignore signal when msleep as = it > >>> can cause file system inconsistency. > >> > >>I'd suggest running these changes through kib@. Especially=20 > >>vn_start_write() > >>change below looks ugly, but maybe it is only temporary? > >It is not only ugly (and object against it). > > > >If the change makes any difference for the filesystem, then I just argue > >that the filesystem is broken. The vn_start_write() is done on the > >VFS entry peripheral, long before filesystem code is hit. > > > >I did not looked at the managed changes, you would need to describe > >what is wrong with current code and what is the purpose of the changes. > >B_MANAGED came from xfs, it seems, or at least xfs is the only current > >consumer of B_MANAGED buffers. >=20 > Hi Kostik, >=20 > Without our change in getblk() whenewer we allocate new block we get pani= c: >=20 > panic: bremfree: buffer 0xffffff807bf86080 not on a queue. >=20 > It is because blocks with B_MANAGED flag are not queued on any queue in= =20 > brelse() function. Could you look at it and give us approval to merge=20 > this change into HEAD? Right, but this is in fact the only function of the B_MANAGED flag. So the question is, what are you trying to accomplish. --G+2xtMkjVKN5mon6 Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.12 (FreeBSD) iEYEARECAAYFAk+o7c8ACgkQC3+MBN1Mb4iE4gCfb0+uywOSiHpdcv3pYxQ6rlY/ iFcAnim67HzUPhyScxHoenoiaG15TyrY =nIwi -----END PGP SIGNATURE----- --G+2xtMkjVKN5mon6--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20120508095631.GV2358>