summaryrefslogtreecommitdiffstats
path: root/include/linux/loop.h
diff options
context:
space:
mode:
authorChristoph Hellwig <hch@infradead.org>2011-10-17 12:57:20 +0200
committerJens Axboe <axboe@kernel.dk>2011-10-17 12:57:20 +0200
commit456be1484ffc72a24bdb4200b5847c4fa90139d9 (patch)
tree570f0818bd6cfa245ab23d0121853b7b1e5a649b /include/linux/loop.h
parent8bc03e8f3a334e09e89a7dffb486ee97a5ce84ae (diff)
downloadop-kernel-dev-456be1484ffc72a24bdb4200b5847c4fa90139d9.zip
op-kernel-dev-456be1484ffc72a24bdb4200b5847c4fa90139d9.tar.gz
loop: remove the incorrect write_begin/write_end shortcut
Currently the loop device tries to call directly into write_begin/write_end instead of going through ->write if it can. This is a fairly nasty shortcut as write_begin and write_end are only callbacks for the generic write code and expect to be called with filesystem specific locks held. This code currently causes various issues for clustered filesystems as it doesn't take the required cluster locks, and it also causes issues for XFS as it doesn't properly lock against the swapext ioctl as called by the defragmentation tools. This in case causes data corruption if defragmentation hits a busy loop device in the wrong time window, as reported by RH QA. The reason why we have this shortcut is that it saves a data copy when doing a transformation on the loop device, which is the technical term for using cryptoloop (or an XOR transformation). Given that cryptoloop has been deprecated in favour of dm-crypt my opinion is that we should simply drop this shortcut instead of finding complicated ways to to introduce a formal interface for this shortcut. Signed-off-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'include/linux/loop.h')
-rw-r--r--include/linux/loop.h1
1 files changed, 0 insertions, 1 deletions
diff --git a/include/linux/loop.h b/include/linux/loop.h
index 683d698..a068806 100644
--- a/include/linux/loop.h
+++ b/include/linux/loop.h
@@ -73,7 +73,6 @@ struct loop_device {
*/
enum {
LO_FLAGS_READ_ONLY = 1,
- LO_FLAGS_USE_AOPS = 2,
LO_FLAGS_AUTOCLEAR = 4,
};
OpenPOWER on IntegriCloud