Re: [patch 06/14] hfsplus: remove hfsplus_permission()

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Miklos Szeredi
Date: Friday, May 23, 2008 - 12:05 pm

> >  a) it's nontrivial to fix (even understanding the problem is

Semi-concrete: link(2) locks the target's parent and the source.
Cross-directory rename(2) locks both parents.  If link's target is a
file which has children, this can result in an ABBA deadlock.  That's
_before_ the filesystem's ->link() or ->rename() function is called.


Dunno.  What's the difference between "it wasn't used" and "it didn't
work and nobody noticed"?  I think not much :)

Miklos
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[patch 06/14] hfsplus: remove hfsplus_permission(), Miklos Szeredi, (Wed May 21, 10:15 am)
Re: [patch 06/14] hfsplus: remove hfsplus_permission(), Roman Zippel, (Wed May 21, 1:35 pm)
Re: [patch 06/14] hfsplus: remove hfsplus_permission(), Miklos Szeredi, (Thu May 22, 12:58 am)
Re: [patch 06/14] hfsplus: remove hfsplus_permission(), Roman Zippel, (Thu May 22, 5:02 am)
Re: [patch 06/14] hfsplus: remove hfsplus_permission(), Miklos Szeredi, (Thu May 22, 5:17 am)
Re: [patch 06/14] hfsplus: remove hfsplus_permission(), Roman Zippel, (Thu May 22, 5:28 am)
Re: [patch 06/14] hfsplus: remove hfsplus_permission(), Miklos Szeredi, (Thu May 22, 5:37 am)
Re: [patch 06/14] hfsplus: remove hfsplus_permission(), Christoph Hellwig, (Fri May 23, 2:21 am)
Re: [patch 06/14] hfsplus: remove hfsplus_permission(), Roman Zippel, (Fri May 23, 8:11 am)
Re: [patch 06/14] hfsplus: remove hfsplus_permission(), Roman Zippel, (Fri May 23, 8:18 am)
Re: [patch 06/14] hfsplus: remove hfsplus_permission(), Miklos Szeredi, (Fri May 23, 8:31 am)
Re: [patch 06/14] hfsplus: remove hfsplus_permission(), Miklos Szeredi, (Fri May 23, 8:49 am)
Re: [patch 06/14] hfsplus: remove hfsplus_permission(), Roman Zippel, (Fri May 23, 9:30 am)
Re: [patch 06/14] hfsplus: remove hfsplus_permission(), Miklos Szeredi, (Fri May 23, 11:02 am)
Re: [patch 06/14] hfsplus: remove hfsplus_permission(), Roman Zippel, (Fri May 23, 11:33 am)
Re: [patch 06/14] hfsplus: remove hfsplus_permission(), Miklos Szeredi, (Fri May 23, 12:05 pm)
Re: [patch 06/14] hfsplus: remove hfsplus_permission(), Roman Zippel, (Fri May 23, 2:52 pm)
Re: [patch 06/14] hfsplus: remove hfsplus_permission(), Miklos Szeredi, (Fri May 23, 11:59 pm)