git: DragonFly_RELEASE_5_6 hammer2 - Finally fix the 'file winds up with some zeros' bug

Matthew Dillon dillon at crater.dragonflybsd.org
Tue Nov 19 12:07:11 PST 2019


commit 9baa033a4a3953ca0f66542bc828184317627503
Author: Matthew Dillon <dillon at apollo.backplane.com>
Date:   Tue Nov 19 12:02:39 2019 -0800

    hammer2 - Finally fix the 'file winds up with some zeros' bug
    
    * Fix this bug, which has been reported several times this year
      but until nikita's hands-on I couldn't find it.
    
    * The bug occurs when H2 must transition from the 512 bytes of
      file data directly embedded in the inode to a block table as
      a file grows past 512 bytes.
    
      Basically what happens is that the buffer cache buffer holding
      the data in-transition can get lost.
    
      The result is that the first write() that overlaps seek offset
      512 in a small file can sometimes cause the prior data at the
      start of the file to be lost.
    
    * To solve the problem the buffer is now held across the transition
      and explicitly dirtied to force it to be flushed to the now-
      realized block table.
    
    Reported-by: nikita and several others this year

Summary of changes:
 sys/vfs/hammer2/hammer2_vnops.c | 30 +++++++++++++++++++++++++-----
 1 file changed, 25 insertions(+), 5 deletions(-)

http://gitweb.dragonflybsd.org/dragonfly.git/commitdiff/9baa033a4a3953ca0f66542bc828184317627503


-- 
DragonFly BSD source repository


More information about the Commits mailing list