remove lock and unlock super
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest8 patches 16 repliesIn review
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/8 | vfs: remove lock and unlock functions for super blockfs/ include/ | +0 | 2 replies | |
| 2/8 | exofs: remove lock/unlock superfs/ | +0 | 9 replies | |
| 3/8 | ext3: remove lock/unlock superfs/ fs/ | +7 | 5 replies | |
| 4/8 | ext4: remove lock/unlock superfs/ fs/ | +11 | 0 replies | |
| 5/8 | fat: remove lock/unlock superfs/ fs/ fs/ +2 more | +36 | 0 replies | |
| 6/8 | hpfs: remove lock/unlock superfs/ | +0 | 0 replies | |
| 7/8 | sysv: remove lock/unlock superfs/ fs/ fs/ +2 more | +20 | 0 replies | |
| 8/8 | ufs: remove lock/unlock superfs/ fs/ fs/ +1 more | +35 | 0 replies |
Reviews trailers per patch
1 of 8 patches has a Reviewed-by or Acked-by.
| Patch | Artem Bityutskiy | Boaz Harrosh |
|---|---|---|
| 1/8 | ||
| 2/8 | A-b | ACK-by |
| 3/8 | ||
| 4/8 | ||
| 5/8 | ||
| 6/8 | ||
| 7/8 | ||
| 8/8 |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH 0/8] remove lock and unlock super
Hi all,
I'm trying to remove the functions lock_super/unlock_super and to push the lock into each single fs. Currently these fs use these functions: ext3, ext4, fat, hpfs, exofs, sysv, ufs. At the moment I used the more conservative approach, I created a new mutex s_lock in the private sb info for each fs, so nothing change but a couple of notes:
1) exofs/hpfs: they use lock_super only in one function so the lock seems completely not needed and I removed it, do you see collateral effect?
2) fat/ufs: they have already got functions to lock the fs with a mutex, I don't know at the moment if a general review of the code can give us the possibility to "merge" the locks.
Bugs, comments, review are welcome especially from fs maintainers. Maybe this work can be a first cleaning, after that each fs can adjust its lock policy.
The patch is against 3.6-rc1.
Marco