On Mon, Jun 12, 2023 at 1:46 PM <xiubli@xxxxxxxxxx> wrote:
>
> From: Xiubo Li <xiubli@xxxxxxxxxx>
>
> This will help print the client's global_id in debug logs.
Hi Xiubo,
There is a related concern that clients can belong to different
clusters, in which case their global IDs might clash. If you chose
to disregard that as an unlikely scenario, it's probably fine, but
it would be nice to make that explicit in the commit message.
If account for that, the identifier block could look like:
[<cluster fsid> <gid>]
instead of:
[client.<gid>]
The "client." string prefix seems a bit redundant since the kernel
client's entity is always CEPH_ENTITY_TYPE_CLIENT. If you like it
anyway, I would at least get rid of the dot at the end to align with
how it's presented elsewhere (e.g. debugfs directory name or
"ceph.client_id" xattr).
Thanks,
Ilya
