Date: Sat, 21 Apr 2012 02:49:47 +0200 From: Grzegorz Bernacki <gjb@semihalf.com> To: Konstantin Belousov <kostikbel@gmail.com> Cc: svn-src-projects@FreeBSD.org, Grzegorz Bernacki <gber@FreeBSD.org>, Pawel Jakub Dawidek <pjd@FreeBSD.org>, src-committers@FreeBSD.org Subject: Re: svn commit: r233072 - projects/nand/sys/kern Message-ID: <4F92042B.4010606@semihalf.com> In-Reply-To: <20120317161050.GI75778@deviant.kiev.zoral.com.ua> References: <201203170318.q2H3ITdI047893@svn.freebsd.org> <20120317085116.GC1340@garage.freebsd.pl> <20120317161050.GI75778@deviant.kiev.zoral.com.ua>
next in thread | previous in thread | raw e-mail | index | archive | help
W dniu 2012-03-17 17:10, Konstantin Belousov pisze: > 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 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. Hi Kostik, We revert all unnecessary changes in sys/kern files. The only change which left is the one in vfs_bio.c file. I believe that this one is requested for managed buffer to work correctly. Let me verify it and I will let you know. regards, grzesiek
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4F92042B.4010606>