| Message ID | 20211104123147.1632-1-lhenriques@suse.de (mailing list archive) |
|---|---|
| Headers | show |
| Series | ceph: metrics for remote object copies | expand |
On Thu, 2021-11-04 at 12:31 +0000, Luís Henriques wrote: > Hi! > > Here's v2 of this patchset. The differences from v1: > > * Instead of changing ceph_osdc_copy_from() in libceph.ko to return an > osd request, move that function into the cephfs code instead. > > Other than that, the 2nd patch is quite similar to the one from v1: it > effectively hooks the 'copyfrom' metrics infrastructure. > > Luís Henriques (2): > ceph: libceph: move ceph_osdc_copy_from() into cephfs code > ceph: add a new metric to keep track of remote object copies > > fs/ceph/debugfs.c | 3 +- > fs/ceph/file.c | 78 ++++++++++++++++++++++++++++----- > fs/ceph/metric.h | 8 ++++ > include/linux/ceph/osd_client.h | 19 ++++---- > net/ceph/osd_client.c | 60 ++++--------------------- > 5 files changed, 94 insertions(+), 74 deletions(-) > Looks good. Thanks, Luis. Merged into testing branch.
On Thu, Nov 04, 2021 at 11:09:32AM -0400, Jeff Layton wrote: > On Thu, 2021-11-04 at 12:31 +0000, Luís Henriques wrote: > > Hi! > > > > Here's v2 of this patchset. The differences from v1: > > > > * Instead of changing ceph_osdc_copy_from() in libceph.ko to return an > > osd request, move that function into the cephfs code instead. > > > > Other than that, the 2nd patch is quite similar to the one from v1: it > > effectively hooks the 'copyfrom' metrics infrastructure. > > > > Luís Henriques (2): > > ceph: libceph: move ceph_osdc_copy_from() into cephfs code > > ceph: add a new metric to keep track of remote object copies > > > > fs/ceph/debugfs.c | 3 +- > > fs/ceph/file.c | 78 ++++++++++++++++++++++++++++----- > > fs/ceph/metric.h | 8 ++++ > > include/linux/ceph/osd_client.h | 19 ++++---- > > net/ceph/osd_client.c | 60 ++++--------------------- > > 5 files changed, 94 insertions(+), 74 deletions(-) > > > > Looks good. Thanks, Luis. Merged into testing branch. Awesome, thanks. I'll wait until this is merged into mainline (which will take a while, of course) before I push changes to the fstests. Adding further checks to the tests that use remote copies was the drive for these new metrics, and I've already some patches for doing that. They'll need some cleanup but it doesn't make sense to push them before this is available upstream. Cheers, -- Luís
Hi! Here's v2 of this patchset. The differences from v1: * Instead of changing ceph_osdc_copy_from() in libceph.ko to return an osd request, move that function into the cephfs code instead. Other than that, the 2nd patch is quite similar to the one from v1: it effectively hooks the 'copyfrom' metrics infrastructure. Luís Henriques (2): ceph: libceph: move ceph_osdc_copy_from() into cephfs code ceph: add a new metric to keep track of remote object copies fs/ceph/debugfs.c | 3 +- fs/ceph/file.c | 78 ++++++++++++++++++++++++++++----- fs/ceph/metric.h | 8 ++++ include/linux/ceph/osd_client.h | 19 ++++---- net/ceph/osd_client.c | 60 ++++--------------------- 5 files changed, 94 insertions(+), 74 deletions(-)