| Message ID | 20240515084601.3240503-6-libaokun@huaweicloud.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | cachefiles: some bugfixes and cleanups for ondemand requests | expand |
On 5/15/24 4:45 PM, libaokun@huaweicloud.com wrote: > From: Baokun Li <libaokun1@huawei.com> > > This lets us see the correct trace output. > > Fixes: c8383054506c ("cachefiles: notify the user daemon when looking up cookie") > Signed-off-by: Baokun Li <libaokun1@huawei.com> Could we move this simple fix to the beginning of the patch set? Reviewed-by: Jingbo Xu <jefflexu@linux.alibaba.com> > --- > include/trace/events/cachefiles.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/include/trace/events/cachefiles.h b/include/trace/events/cachefiles.h > index 119a823fb5a0..bb56e3104b12 100644 > --- a/include/trace/events/cachefiles.h > +++ b/include/trace/events/cachefiles.h > @@ -130,6 +130,8 @@ enum cachefiles_error_trace { > EM(cachefiles_obj_see_lookup_failed, "SEE lookup_failed") \ > EM(cachefiles_obj_see_withdraw_cookie, "SEE withdraw_cookie") \ > EM(cachefiles_obj_see_withdrawal, "SEE withdrawal") \ > + EM(cachefiles_obj_get_ondemand_fd, "GET ondemand_fd") \ > + EM(cachefiles_obj_put_ondemand_fd, "PUT ondemand_fd") \ > EM(cachefiles_obj_get_read_req, "GET read_req") \ > E_(cachefiles_obj_put_read_req, "PUT read_req") >
On 2024/5/20 15:40, Jingbo Xu wrote: > > On 5/15/24 4:45 PM, libaokun@huaweicloud.com wrote: >> From: Baokun Li <libaokun1@huawei.com> >> >> This lets us see the correct trace output. >> >> Fixes: c8383054506c ("cachefiles: notify the user daemon when looking up cookie") >> Signed-off-by: Baokun Li <libaokun1@huawei.com> > > Could we move this simple fix to the beginning of the patch set? > > Reviewed-by: Jingbo Xu <jefflexu@linux.alibaba.com> > Thanks for the review! This patch is here because when adding trace-related output in the last patch (path 4), it was found that cachefiles_obj_[get|put]_ondemand_fd did not have any relevant trace output, so it is added in this patch. Putting it in the first patch may confuse the reader as to why it was added, but it is easier to understand with cachefiles_obj_[get|put]_read_req in front of it. >> --- >> include/trace/events/cachefiles.h | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/include/trace/events/cachefiles.h b/include/trace/events/cachefiles.h >> index 119a823fb5a0..bb56e3104b12 100644 >> --- a/include/trace/events/cachefiles.h >> +++ b/include/trace/events/cachefiles.h >> @@ -130,6 +130,8 @@ enum cachefiles_error_trace { >> EM(cachefiles_obj_see_lookup_failed, "SEE lookup_failed") \ >> EM(cachefiles_obj_see_withdraw_cookie, "SEE withdraw_cookie") \ >> EM(cachefiles_obj_see_withdrawal, "SEE withdrawal") \ >> + EM(cachefiles_obj_get_ondemand_fd, "GET ondemand_fd") \ >> + EM(cachefiles_obj_put_ondemand_fd, "PUT ondemand_fd") \ >> EM(cachefiles_obj_get_read_req, "GET read_req") \ >> E_(cachefiles_obj_put_read_req, "PUT read_req") >>
diff --git a/include/trace/events/cachefiles.h b/include/trace/events/cachefiles.h index 119a823fb5a0..bb56e3104b12 100644 --- a/include/trace/events/cachefiles.h +++ b/include/trace/events/cachefiles.h @@ -130,6 +130,8 @@ enum cachefiles_error_trace { EM(cachefiles_obj_see_lookup_failed, "SEE lookup_failed") \ EM(cachefiles_obj_see_withdraw_cookie, "SEE withdraw_cookie") \ EM(cachefiles_obj_see_withdrawal, "SEE withdrawal") \ + EM(cachefiles_obj_get_ondemand_fd, "GET ondemand_fd") \ + EM(cachefiles_obj_put_ondemand_fd, "PUT ondemand_fd") \ EM(cachefiles_obj_get_read_req, "GET read_req") \ E_(cachefiles_obj_put_read_req, "PUT read_req")