Whamcloud - gitweb
LU-10994 osc: remove oap_cli 78/59078/4
authorJohn L. Hammond <jhammond@whamcloud.com>
Sun, 4 May 2025 07:49:11 +0000 (00:49 -0700)
committerOleg Drokin <green@whamcloud.com>
Thu, 15 May 2025 07:30:44 +0000 (07:30 +0000)
commit7d7bc36eb4da15c8f87d9381f61bf1cc0e5263c6
tree9ddd5ea2f7a6549856ae04e4bf36ee47492ec8c0
parent45b717e92e7b81e594ab6365c4da95034a6f6511
LU-10994 osc: remove oap_cli

Remove the redundant oap_cli member from struct osc_async_page.

...:(cl_page.c:216:__cl_page_alloc()) slab-alloced 'cl_page': 256 at 000000009ab84b37.

Lustre-change: https://review.whamcloud.com/47403
Lustre-commit: 9b4fb6e65414fa75a23970863e8b0cc34d621db7

Was-Change-Id: Idd088f0906a10773568495933592ac5e755dc047

LU-10994 clio: remove cpl_obj

Remove cpl_obj from struct cl_page_slice. This member is only used in
the osc layer and struct osc_page already contains a pointer to the
osc_object.

Lustre-change: https://review.whamcloud.com/47402
Lustre-commit: 10da530a2411f28988ab5287e73ccbcfb6100605

Was-Change-Id: I6451aa50ff0e8db67f1c6f4f7edbde4fa8d36c5b

LU-10994 clio: remove unused convenience functions

Remove the unused convenience functions cl_page_top(), cl_page_at(),
cl_page_at_trusted(), and cl2vm_page().

Lustre-change: https://review.whamcloud.com/47401
Lustre-commit: 27e19a5420ae3ec22d8f3fe7f0e7794676479540

Was-Change-Id: I9c994d8f4c81bc93383a9eb46def514685a27690

LU-10994 clio: remove struct vvp_page

Remove struct vvp_page and use struct cl_page_slice in its place. Use
cp_vmpage in place of vpg_page and cl_page_index() in place of
vvp_index().

Lustre-change: https://review.whamcloud.com/47400
Lustre-commit: 127570b3e5a2ff018323724d2c060ccda1fc5e3d

Was-Change-Id: I2cd408f08e6ff9f7686b591c02ea95e31ad2b2ae

LU-10994 clio: remove cpo_prep and cpo_make_ready

Remove the cpo_prep and cpo_make_ready methods from struct
cl_page_operations. These methods were only implemented by the vvp
layer and so they can be easily inlined into cl_page_prep() and
cl_page_make_ready().

Lustre-change: https://review.whamcloud.com/47399
Lustre-commit: ca161cfbcbf04bffb33a678d3a744a9276b4b78a

Was-Change-Id: I177fd8d3c3832bcc8f06ed98cdf9d30f18d49e88

LU-10994 clio: remove vvp_page_print()

Remove vvp_page_print() by placing equivalent code in cl_page_print().

Lustre-change: https://review.whamcloud.com/47398
Lustre-commit: bf1d1b0e41ff245f59280a0783e80dc81278e730

Was-Change-Id: I815c4f63dc6fe57eec0987f209a2f34d3ff58146

LU-10994 clio: Remove cl_2queue_add wrapper

Remove the wrapper function cl_2queue_add() and replace all its calls in
different files with the function it wrapped. Also, comments are added
wherever necessary to make the working of function clear. Prototype of
the function is also removed from the header file as it is no longer
needed.

Linux-commit: 53f1a12768a55e53b2c40e00a8804b1edfa739b3

Lustre-change: https://review.whamcloud.com/47651
Lustre-commit: 5038bf8db83d4cb409b7563f028f48cca0385c19

Was-Change-Id: Ic746c45e3dda9fdf3f1d2f8c6204d80fec5c058f

LU-10994 clio: remove cpo_assume, cpo_unassume, cpo_fini

Remove the cl_page methods cpo_assume, cpo_unassume, and
cpo_fini. These methods were only implemented by the vvp layer and so
they can be easily inlined into cl_page_assume() and
cl_page_unassume(). Remove vvp_page_delete() by inlining its contents
to cl_page_delete0().

Lustre-change: https://review.whamcloud.com/47373
Lustre-commit: 9045894fe0f5033334a39a35a6332dab4498e21e

Was-Change-Id: I260c5593983bac6742cf7577c26a4903e95ceb7c

LU-10994 clio: remove cpo_own and cpo_disown

Remove the cpo_own and cpo_disown methods from struct
cl_page_operations. These methods were only implemented by the vvp
layer so they can be inlined into cl_page_own0() and
cl_page_disown(). Move most of vvp_page_discard() and all of
vvp_transient_page_discard() into cl_page_discard().

Lustre-change: https://review.whamcloud.com/47372
Lustre-commit: 81c6dc423ce4c62a64d328e49697d26194177f9f

Was-Change-Id: I3f156d6ca3e4ea11c050b2addda38e84a84634b9

LU-10994 clio: remove cl_page_export() and cl_page_is_vmlocked()

Remove cl_page_export() and cl_page_is_vmlocked(), replacing them with
direct calls to PageSetUptodate() and PageLoecked().

Lustre-change: https://review.whamcloud.com/47241
Lustre-commit: 3d52a7c5753e80e78c3b6f6bb7a0b66b37f4849b

Was-Change-Id: I883d1664f4afc7a1d4006f9f4833db8125c0e8f5

LU-10994 echo: remove client operations from echo objects

Remove the client (io, page, lock) operations from echo_client
objects. This will facilitate the simplification of CLIO.

Lustre-change: https://review.whamcloud.com/47240
Lustre-commit: 6060ee55b194e37e87031c40e9d48f967eabe314

Was-Change-Id: If9e55c7d54c171aa2e1bcf272641c2bd6be8ad48

LU-10994 lov: remove lov_page

Remove the lov page layer since it does nothing but costs 24 bytes per
page plus pointer chases.

Lustre-change: https://review.whamcloud.com/47221
Lustre-commit: 56f520b1a4c9ae64caa235e9ce7699e7fb627f0c

Was-Change-Id: Icd7b4b0041e0fe414a3a4143179f45845177960e

LU-6142 clio: make cp_ref in cl_page a refcount_t

As this is used as a refcount, it should be declared
as one.

Lustre-change: https://review.whamcloud.com/49072
Lustre-commit: e19804a3b7e793a11b1c8b5e0db9f6315f243b8c

Was-Change-Id: I8108e14e545bb56aae34a0f6ae9d5a04227fc067

LU-6142 obdclass: make ccc_users in cl_client_cache a refcount_t

As this is used as a refcount, it should be declared
as one.

Lustre-change: https://review.whamcloud.com/48881
Lustre-commit: ed0f74cd635fa110c1896981d58c389826041808

Was-Change-Id: I5af513ccb2b706a398e647ce0427affa4516a9b5

LU-6142 obdclass: change some foo0() to __foo()

Change:
  cl_io_init0 -> __cl_io_init
  cl_lock_trace0 -> __cl_lock_trace
  cl_page_delete0 -> __cl_page_delete
  cl_page_state_set0 -> __cl_page_state_set
  cl_page_own0 -> __cl_page_own
  cl_page_disown0 -> __cl_page_disown
  cl_page_delete0 -> __cl_page_delete

This is more consistent with Linux naming style.

Lustre-change: https://review.whamcloud.com/48803
Lustre-commit: 8828aa8e75f28808da42f521662a245c6b8a9896

Was-Change-Id: If38b52465d42ac425d47c1e9ded62bd7f013e0eb

Signed-off-by: John L. Hammond <jhammond@whamcloud.com>
Change-Id: Idd088f0906a10773568495933592ac5e755dc047
Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/59078
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
24 files changed:
lustre/include/cl_object.h
lustre/include/lustre_osc.h
lustre/llite/file.c
lustre/llite/llite_lib.c
lustre/llite/lproc_llite.c
lustre/llite/rw.c
lustre/llite/rw26.c
lustre/llite/vvp_dev.c
lustre/llite/vvp_internal.h
lustre/llite/vvp_io.c
lustre/llite/vvp_object.c
lustre/llite/vvp_page.c
lustre/lov/lov_cl_internal.h
lustre/lov/lov_object.c
lustre/lov/lov_page.c
lustre/obdclass/cl_internal.h
lustre/obdclass/cl_io.c
lustre/obdclass/cl_lock.c
lustre/obdclass/cl_page.c
lustre/obdecho/echo_client.c
lustre/osc/osc_cache.c
lustre/osc/osc_lock.c
lustre/osc/osc_page.c
lustre/osc/osc_request.c