• Linus Torvalds's avatar
    Merge tag 'afs-fixes-20190620' of git://git.kernel.org/pub/scm/linux/kernel/git/dhowells/linux-fs · cd0f3aae
    Linus Torvalds authored
    Pull AFS fixes from David Howells:
     "The in-kernel AFS client has been undergoing testing on opendev.org on
      one of their mirror machines. They are using AFS to hold data that is
      then served via apache, and Ian Wienand had reported seeing oopses,
      spontaneous machine reboots and updates to volumes going missing. This
      patch series appears to have fixed the problem, very probably due to
      patch (2), but it's not 100% certain.
    
      (1) Fix the printing of the "vnode modified" warning to exclude checks
          on files for which we don't have a callback promise from the
          server (and so don't expect the server to tell us when it
          changes).
    
          Without this, for every file or directory for which we still have
          an in-core inode that gets changed on the server, we may get a
          message logged when we next look at it. This can happen in bulk
          if, for instance, someone does "vos release" to update a R/O
          volume from a R/W volume and a whole set of files are all changed
          together.
    
          We only really want to log a message if the file changed and the
          server didn't tell us about it or we failed to track the state
          internally.
    
      (2) Fix accidental corruption of either afs_vlserver struct objects or
          the the following memory locations (which could hold anything).
          The issue is caused by a union that points to two different
          structs in struct afs_call (to save space in the struct). The call
          cleanup code assumes that it can simply call the cleanup for one
          of those structs if not NULL - when it might be actually pointing
          to the other struct.
    
          This means that every Volume Location RPC op is going to corrupt
          something.
    
      (3) Fix an uninitialised spinlock. This isn't too bad, it just causes
          a one-off warning if lockdep is enabled when "vos release" is
          called, but the spinlock still behaves correctly.
    
      (4) Fix the setting of i_block in the inode. This causes du, for
          example, to produce incorrect results, but otherwise should not be
          dangerous to the kernel"
    
    * tag 'afs-fixes-20190620' of git://git.kernel.org/pub/scm/linux/kernel/git/dhowells/linux-fs:
      afs: Fix setting of i_blocks
      afs: Fix uninitialised spinlock afs_volume::cb_break_lock
      afs: Fix vlserver record corruption
      afs: Fix over zealous "vnode modified" warnings
    cd0f3aae
volume.c 8 KB