Skip to content

Commit 305ff6b

Browse files
ethanwu-synoidryomov
authored andcommitted
ceph: supply snapshot context in ceph_uninline_data()
The ceph_uninline_data function was missing proper snapshot context handling for its OSD write operations. Both CEPH_OSD_OP_CREATE and CEPH_OSD_OP_WRITE requests were passing NULL instead of the appropriate snapshot context, which could lead to unnecessary object clone. Reproducer: ../src/vstart.sh --new -x --localhost --bluestore // turn on cephfs inline data ./bin/ceph fs set a inline_data true --yes-i-really-really-mean-it // allow fs_a client to take snapshot ./bin/ceph auth caps client.fs_a mds 'allow rwps fsname=a' mon 'allow r fsname=a' osd 'allow rw tag cephfs data=a' // mount cephfs with fuse, since kernel cephfs doesn't support inline write ceph-fuse --id fs_a -m 127.0.0.1:40318 --conf ceph.conf -d /mnt/mycephfs/ // bump snapshot seq mkdir /mnt/mycephfs/.snap/snap1 echo "foo" > /mnt/mycephfs/test // umount and mount it again using kernel cephfs client umount /mnt/mycephfs mount -t ceph fs_a@.a=/ /mnt/mycephfs/ -o conf=./ceph.conf echo "bar" >> /mnt/mycephfs/test ./bin/rados listsnaps -p cephfs.a.data $(printf "%x\n" $(stat -c %i /mnt/mycephfs/test)).00000000 will see this object does unnecessary clone 1000000000a.00000000 (seq:2): cloneid snaps size overlap 2 2 4 [] head - 8 but it's expected to see 10000000000.00000000 (seq:2): cloneid snaps size overlap head - 8 since there's no snapshot between these 2 writes clone happened because the first osd request CEPH_OSD_OP_CREATE doesn't pass snap context so object is created with snap seq 0, but later data writeback is equipped with snapshot context. snap.seq(1) > object snap seq(0), so osd does object clone. This fix properly acquiring the snapshot context before performing write operations. Signed-off-by: ethanwu <ethanwu@synology.com> Reviewed-by: Viacheslav Dubeyko <Slava.Dubeyko@ibm.com> Tested-by: Viacheslav Dubeyko <Slava.Dubeyko@ibm.com> Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
1 parent f16bd3f commit 305ff6b

1 file changed

Lines changed: 22 additions & 2 deletions

File tree

fs/ceph/addr.c

Lines changed: 22 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2199,6 +2199,7 @@ int ceph_uninline_data(struct file *file)
21992199
struct ceph_osd_request *req = NULL;
22002200
struct ceph_cap_flush *prealloc_cf = NULL;
22012201
struct folio *folio = NULL;
2202+
struct ceph_snap_context *snapc = NULL;
22022203
u64 inline_version = CEPH_INLINE_NONE;
22032204
struct page *pages[1];
22042205
int err = 0;
@@ -2226,6 +2227,24 @@ int ceph_uninline_data(struct file *file)
22262227
if (inline_version == 1) /* initial version, no data */
22272228
goto out_uninline;
22282229

2230+
down_read(&fsc->mdsc->snap_rwsem);
2231+
spin_lock(&ci->i_ceph_lock);
2232+
if (__ceph_have_pending_cap_snap(ci)) {
2233+
struct ceph_cap_snap *capsnap =
2234+
list_last_entry(&ci->i_cap_snaps,
2235+
struct ceph_cap_snap,
2236+
ci_item);
2237+
snapc = ceph_get_snap_context(capsnap->context);
2238+
} else {
2239+
if (!ci->i_head_snapc) {
2240+
ci->i_head_snapc = ceph_get_snap_context(
2241+
ci->i_snap_realm->cached_context);
2242+
}
2243+
snapc = ceph_get_snap_context(ci->i_head_snapc);
2244+
}
2245+
spin_unlock(&ci->i_ceph_lock);
2246+
up_read(&fsc->mdsc->snap_rwsem);
2247+
22292248
folio = read_mapping_folio(inode->i_mapping, 0, file);
22302249
if (IS_ERR(folio)) {
22312250
err = PTR_ERR(folio);
@@ -2241,7 +2260,7 @@ int ceph_uninline_data(struct file *file)
22412260
req = ceph_osdc_new_request(&fsc->client->osdc, &ci->i_layout,
22422261
ceph_vino(inode), 0, &len, 0, 1,
22432262
CEPH_OSD_OP_CREATE, CEPH_OSD_FLAG_WRITE,
2244-
NULL, 0, 0, false);
2263+
snapc, 0, 0, false);
22452264
if (IS_ERR(req)) {
22462265
err = PTR_ERR(req);
22472266
goto out_unlock;
@@ -2257,7 +2276,7 @@ int ceph_uninline_data(struct file *file)
22572276
req = ceph_osdc_new_request(&fsc->client->osdc, &ci->i_layout,
22582277
ceph_vino(inode), 0, &len, 1, 3,
22592278
CEPH_OSD_OP_WRITE, CEPH_OSD_FLAG_WRITE,
2260-
NULL, ci->i_truncate_seq,
2279+
snapc, ci->i_truncate_seq,
22612280
ci->i_truncate_size, false);
22622281
if (IS_ERR(req)) {
22632282
err = PTR_ERR(req);
@@ -2320,6 +2339,7 @@ int ceph_uninline_data(struct file *file)
23202339
folio_put(folio);
23212340
}
23222341
out:
2342+
ceph_put_snap_context(snapc);
23232343
ceph_free_cap_flush(prealloc_cf);
23242344
doutc(cl, "%llx.%llx inline_version %llu = %d\n",
23252345
ceph_vinop(inode), inline_version, err);

0 commit comments

Comments
 (0)