diff options
author | Sage Weil <sage@newdream.net> | 2010-11-09 15:43:12 -0500 |
---|---|---|
committer | Sage Weil <sage@newdream.net> | 2010-11-09 15:43:12 -0500 |
commit | b7495fc2ff941db6a118a93ab8d61149e3f4cef8 (patch) | |
tree | 231c339d74760e2fa13e5e6f41c10bc28cea51b3 /fs/ceph/inode.c | |
parent | e98b6fed84d0f0155d7b398e0dfeac74c792f2d0 (diff) |
ceph: make page alignment explicit in osd interface
We used to infer alignment of IOs within a page based on the file offset,
which assumed they matched. This broke with direct IO that was not aligned
to pages (e.g., 512-byte aligned IO). We were also trusting the alignment
specified in the OSD reply, which could have been adjusted by the server.
Explicitly specify the page alignment when setting up OSD IO requests.
Signed-off-by: Sage Weil <sage@newdream.net>
Diffstat (limited to 'fs/ceph/inode.c')
-rw-r--r-- | fs/ceph/inode.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/ceph/inode.c b/fs/ceph/inode.c index 7bc0fbd26af2..8153ee5a8d74 100644 --- a/fs/ceph/inode.c +++ b/fs/ceph/inode.c | |||
@@ -1752,7 +1752,7 @@ int ceph_do_getattr(struct inode *inode, int mask) | |||
1752 | return 0; | 1752 | return 0; |
1753 | } | 1753 | } |
1754 | 1754 | ||
1755 | dout("do_getattr inode %p mask %s\n", inode, ceph_cap_string(mask)); | 1755 | dout("do_getattr inode %p mask %s mode 0%o\n", inode, ceph_cap_string(mask), inode->i_mode); |
1756 | if (ceph_caps_issued_mask(ceph_inode(inode), mask, 1)) | 1756 | if (ceph_caps_issued_mask(ceph_inode(inode), mask, 1)) |
1757 | return 0; | 1757 | return 0; |
1758 | 1758 | ||