1. 24 Aug, 2009 1 commit
    • Miklos Szeredi's avatar
      vfs_rename_dir() doesn't properly account for filesystems with · f5dc5e07
      Miklos Szeredi authored
      FS_RENAME_DOES_D_MOVE.  If new_dentry has a target inode attached, it
      unhashes the new_dentry prior to the rename() iop and rehashes it after,
      but doesn't account for the possibility that rename() may have swapped
      {old,new}_dentry.  For FS_RENAME_DOES_D_MOVE filesystems, it rehashes
      new_dentry (now the old renamed-from name, which d_move() expected to go
      away), such that a subsequent lookup will find it.
      
      This was caught by the recently posted POSIX fstest suite, rename/10.t
      test 62 (and others) on ceph.
      
      The bug was introduced by: commit 349457cc
      "[PATCH] Allow file systems to manually d_move() inside of ->rename()"
      
      Fix by not rehashing the new dentry.  Rehashing used to be needed by
      d_move() but isn't anymore.
      Reported-by: default avatarSage Weil <sage@newdream.net>
      Cc: Zach Brown <zach.brown@oracle.com>
      Signed-off-by: default avatarMiklos Szeredi <mszeredi@suse.cz>
      Cc: Mark Fasheh <mark.fasheh@oracle.com>
      Cc: Trond Myklebust <trond.myklebust@fys.uio.no>
      Cc: Al Viro <viro@zeniv.linux.org.uk>
      Cc: Christoph Hellwig <hch@lst.de>
      Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
      f5dc5e07
  2. 10 Sep, 2009 1 commit
  3. 11 Sep, 2009 1 commit
  4. 03 Sep, 2009 3 commits
  5. 22 Aug, 2009 1 commit
  6. 13 Aug, 2009 1 commit
  7. 03 Sep, 2009 1 commit
  8. 02 Sep, 2009 2 commits
  9. 13 Sep, 2009 1 commit
  10. 31 Aug, 2009 1 commit
  11. 25 Aug, 2009 1 commit
  12. 24 Aug, 2009 2 commits
  13. 20 Aug, 2009 1 commit
  14. 04 Aug, 2009 2 commits
  15. 29 Jul, 2009 4 commits
  16. 17 Jul, 2009 1 commit
  17. 11 Aug, 2009 1 commit
  18. 02 Oct, 2008 2 commits
    • Harvey Harrison's avatar
      It would be easy to get rid of the typedefs too, but I kept this · 57e43875
      Harvey Harrison authored
      minimal.
      
      Noticed by sparse:
      drivers/scsi/gdth.c:2395:31: warning: incorrect type in assignment (different base types)
      drivers/scsi/gdth.c:2395:31:    expected unsigned int [unsigned] [assigned] [usertype] last_block_no
      drivers/scsi/gdth.c:2395:31:    got restricted __be32 [usertype] <noident>
      drivers/scsi/gdth.c:2396:27: warning: incorrect type in assignment (different base types)
      drivers/scsi/gdth.c:2396:27:    expected unsigned int [unsigned] [assigned] [usertype] block_length
      drivers/scsi/gdth.c:2396:27:    got restricted __be32 [usertype] <noident>
      drivers/scsi/gdth.c:2406:33: warning: incorrect type in assignment (different base types)
      drivers/scsi/gdth.c:2406:33:    expected unsigned long long [unsigned] [usertype] last_block_no
      drivers/scsi/gdth.c:2406:33:    got restricted __be64 [usertype] <noident>
      drivers/scsi/gdth.c:2407:33: warning: incorrect type in assignment (different base types)
      drivers/scsi/gdth.c:2407:33:    expected unsigned int [unsigned] [usertype] block_length
      drivers/scsi/gdth.c:2407:33:    got restricted __be32 [usertype] <noident>
      Signed-off-by: default avatarHarvey Harrison <harvey.harrison@gmail.com>
      Cc: Boaz Harrosh <bharrosh@panasas.com>
      Cc: James Bottomley <James.Bottomley@HansenPartnership.com>
      Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
      57e43875
    • Harvey Harrison's avatar
      Remove two now unnecessary local vars. · 2f299076
      Harvey Harrison authored
      Signed-off-by: default avatarHarvey Harrison <harvey.harrison@gmail.com>
      Cc: Boaz Harrosh <bharrosh@panasas.com>
      Cc: James Bottomley <James.Bottomley@HansenPartnership.com>
      Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
      2f299076
  19. 13 May, 2008 1 commit
  20. 02 Sep, 2009 1 commit
  21. 12 Aug, 2009 1 commit
    • Andrew Morton's avatar
      WARNING: line over 80 characters · 68646226
      Andrew Morton authored
      #29: FILE: include/linux/wait.h:80:
      +extern void __init_waitqueue_head(wait_queue_head_t *q, struct lock_class_key *);
      
      WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable
      #56: FILE: kernel/wait.c:20:
      +EXPORT_SYMBOL(__init_waitqueue_head);
      
      total: 0 errors, 2 warnings, 31 lines checked
      
      ./patches/waitqueues-give-waitqueue-spinlocks-their-own-lockdep-classes.patch has style problems, please review.  If any of these errors
      are false positives report them to the maintainer, see
      CHECKPATCH in MAINTAINERS.
      
      Please run checkpatch prior to sending patches
      
      Cc: David Howells <dhowells@redhat.com>
      Cc: Ingo Molnar <mingo@elte.hu>
      Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
      Cc: Peter Zijlstra <peterz@infradead.org>
      Cc: Takashi Iwai <tiwai@suse.de>
      Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
      68646226
  22. 04 Aug, 2009 1 commit
  23. 24 Jul, 2009 1 commit
  24. 25 Aug, 2009 1 commit
  25. 04 Aug, 2009 1 commit
  26. 12 Sep, 2009 2 commits
  27. 13 Jul, 2009 1 commit
  28. 07 Aug, 2009 1 commit
  29. 24 Aug, 2009 1 commit
  30. 05 Jun, 2009 1 commit