diff options
Diffstat (limited to 'Documentation/filesystems/porting')
| -rw-r--r-- | Documentation/filesystems/porting | 26 |
1 files changed, 17 insertions, 9 deletions
diff --git a/Documentation/filesystems/porting b/Documentation/filesystems/porting index 8c91d1057d9a..0742feebc6e2 100644 --- a/Documentation/filesystems/porting +++ b/Documentation/filesystems/porting | |||
| @@ -94,9 +94,8 @@ protected. | |||
| 94 | --- | 94 | --- |
| 95 | [mandatory] | 95 | [mandatory] |
| 96 | 96 | ||
| 97 | BKL is also moved from around sb operations. ->write_super() Is now called | 97 | BKL is also moved from around sb operations. BKL should have been shifted into |
| 98 | without BKL held. BKL should have been shifted into individual fs sb_op | 98 | individual fs sb_op functions. If you don't need it, remove it. |
| 99 | functions. If you don't need it, remove it. | ||
| 100 | 99 | ||
| 101 | --- | 100 | --- |
| 102 | [informational] | 101 | [informational] |
| @@ -355,12 +354,10 @@ protects *all* the dcache state of a given dentry. | |||
| 355 | via rcu-walk path walk (basically, if the file can have had a path name in the | 354 | via rcu-walk path walk (basically, if the file can have had a path name in the |
| 356 | vfs namespace). | 355 | vfs namespace). |
| 357 | 356 | ||
| 358 | i_dentry and i_rcu share storage in a union, and the vfs expects | 357 | Even though i_dentry and i_rcu share storage in a union, we will |
| 359 | i_dentry to be reinitialized before it is freed, so an: | 358 | initialize the former in inode_init_always(), so just leave it alone in |
| 360 | 359 | the callback. It used to be necessary to clean it there, but not anymore | |
| 361 | INIT_LIST_HEAD(&inode->i_dentry); | 360 | (starting at 3.2). |
| 362 | |||
| 363 | must be done in the RCU callback. | ||
| 364 | 361 | ||
| 365 | -- | 362 | -- |
| 366 | [recommended] | 363 | [recommended] |
| @@ -433,3 +430,14 @@ release it yourself. | |||
| 433 | d_alloc_root() is gone, along with a lot of bugs caused by code | 430 | d_alloc_root() is gone, along with a lot of bugs caused by code |
| 434 | misusing it. Replacement: d_make_root(inode). The difference is, | 431 | misusing it. Replacement: d_make_root(inode). The difference is, |
| 435 | d_make_root() drops the reference to inode if dentry allocation fails. | 432 | d_make_root() drops the reference to inode if dentry allocation fails. |
| 433 | |||
| 434 | -- | ||
| 435 | [mandatory] | ||
| 436 | The witch is dead! Well, 2/3 of it, anyway. ->d_revalidate() and | ||
| 437 | ->lookup() do *not* take struct nameidata anymore; just the flags. | ||
| 438 | -- | ||
| 439 | [mandatory] | ||
| 440 | ->create() doesn't take struct nameidata *; unlike the previous | ||
| 441 | two, it gets "is it an O_EXCL or equivalent?" boolean argument. Note that | ||
| 442 | local filesystems can ignore tha argument - they are guaranteed that the | ||
| 443 | object doesn't exist. It's remote/distributed ones that might care... | ||
