svn commit: r233072 - projects/nand/sys/kern

Grzegorz Bernacki gjb at semihalf.com
Fri Mar 23 14:38:49 UTC 2012


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 at . 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,

Sorry for late response.  Yes, it is workaround. We are going to remove 
it, when we are finished with other tasks.

regards,
grzesiek


More information about the svn-src-projects mailing list