[PATCH v3 0/4] relayfs: misc changes

Jason Xing posted 4 patches 7 months ago
There is a newer version of this series
include/linux/relay.h   | 19 ++++++++++++++-
kernel/relay.c          | 52 +++++++++++++++++++++++++++++++++++------
kernel/trace/blktrace.c | 22 ++---------------
3 files changed, 65 insertions(+), 28 deletions(-)
[PATCH v3 0/4] relayfs: misc changes
Posted by Jason Xing 7 months ago
From: Jason Xing <kernelxing@tencent.com>

The series mostly focuses on the error counters which helps every user
debug their own kernel module.

---
Note: this series is made on top of this cleanup[1] and unmerged commit[2]
[1]: https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git/log/?h=mm-nonmm-unstable
[2]: https://lore.kernel.org/all/20250507134225.63248-1-kerneljasonxing@gmail.com/

Jason Xing (4):
  relayfs: support a counter tracking if per-cpu buffers is full
  relayfs: introduce getting relayfs statistics function
  blktrace: use rbuf->stats.full as a drop indicator in relayfs
  relayfs: support a counter tracking if data is too big to write

 include/linux/relay.h   | 19 ++++++++++++++-
 kernel/relay.c          | 52 +++++++++++++++++++++++++++++++++++------
 kernel/trace/blktrace.c | 22 ++---------------
 3 files changed, 65 insertions(+), 28 deletions(-)

-- 
2.43.5
Re: [PATCH v3 0/4] relayfs: misc changes
Posted by Masami Hiramatsu (Google) 7 months ago
On Sun, 18 May 2025 10:57:30 +0800
Jason Xing <kerneljasonxing@gmail.com> wrote:

> From: Jason Xing <kernelxing@tencent.com>
> 
> The series mostly focuses on the error counters which helps every user
> debug their own kernel module.
> 

The series looks good to me.

Reviewed-by: Masami Hiramatsu (Google) <mhiramat@kernel.org>

Thanks,

> ---
> Note: this series is made on top of this cleanup[1] and unmerged commit[2]
> [1]: https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git/log/?h=mm-nonmm-unstable
> [2]: https://lore.kernel.org/all/20250507134225.63248-1-kerneljasonxing@gmail.com/
> 
> Jason Xing (4):
>   relayfs: support a counter tracking if per-cpu buffers is full
>   relayfs: introduce getting relayfs statistics function
>   blktrace: use rbuf->stats.full as a drop indicator in relayfs
>   relayfs: support a counter tracking if data is too big to write
> 
>  include/linux/relay.h   | 19 ++++++++++++++-
>  kernel/relay.c          | 52 +++++++++++++++++++++++++++++++++++------
>  kernel/trace/blktrace.c | 22 ++---------------
>  3 files changed, 65 insertions(+), 28 deletions(-)
> 
> -- 
> 2.43.5
> 


-- 
Masami Hiramatsu (Google) <mhiramat@kernel.org>
Re: [PATCH v3 0/4] relayfs: misc changes
Posted by Jason Xing 7 months ago
On Fri, May 23, 2025 at 10:39 AM Masami Hiramatsu <mhiramat@kernel.org> wrote:
>
> On Sun, 18 May 2025 10:57:30 +0800
> Jason Xing <kerneljasonxing@gmail.com> wrote:
>
> > From: Jason Xing <kernelxing@tencent.com>
> >
> > The series mostly focuses on the error counters which helps every user
> > debug their own kernel module.
> >
>
> The series looks good to me.
>
> Reviewed-by: Masami Hiramatsu (Google) <mhiramat@kernel.org>

Thanks for the review.

Thanks,
Jason
Re: [PATCH v3 0/4] relayfs: misc changes
Posted by Jason Xing 7 months ago
On Sun, May 18, 2025 at 10:57 AM Jason Xing <kerneljasonxing@gmail.com> wrote:
>
> From: Jason Xing <kernelxing@tencent.com>
>
> The series mostly focuses on the error counters which helps every user
> debug their own kernel module.
>
> ---
> Note: this series is made on top of this cleanup[1] and unmerged commit[2]
> [1]: https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git/log/?h=mm-nonmm-unstable
> [2]: https://lore.kernel.org/all/20250507134225.63248-1-kerneljasonxing@gmail.com/
>
> Jason Xing (4):
>   relayfs: support a counter tracking if per-cpu buffers is full
>   relayfs: introduce getting relayfs statistics function
>   blktrace: use rbuf->stats.full as a drop indicator in relayfs
>   relayfs: support a counter tracking if data is too big to write
>
>  include/linux/relay.h   | 19 ++++++++++++++-
>  kernel/relay.c          | 52 +++++++++++++++++++++++++++++++++++------
>  kernel/trace/blktrace.c | 22 ++---------------
>  3 files changed, 65 insertions(+), 28 deletions(-)

Hi Andrew,

Any comments on the series?

Thanks,
Jason

>
> --
> 2.43.5
>
Re: [PATCH v3 0/4] relayfs: misc changes
Posted by Andrew Morton 7 months ago
On Fri, 23 May 2025 07:16:35 +0800 Jason Xing <kerneljasonxing@gmail.com> wrote:

> On Sun, May 18, 2025 at 10:57 AM Jason Xing <kerneljasonxing@gmail.com> wrote:
> >
> > From: Jason Xing <kernelxing@tencent.com>
> >
> > The series mostly focuses on the error counters which helps every user
> > debug their own kernel module.
> >
> > ---
> > Note: this series is made on top of this cleanup[1] and unmerged commit[2]
> > [1]: https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git/log/?h=mm-nonmm-unstable
> > [2]: https://lore.kernel.org/all/20250507134225.63248-1-kerneljasonxing@gmail.com/
> >
> > Jason Xing (4):
> >   relayfs: support a counter tracking if per-cpu buffers is full
> >   relayfs: introduce getting relayfs statistics function
> >   blktrace: use rbuf->stats.full as a drop indicator in relayfs
> >   relayfs: support a counter tracking if data is too big to write
> >
> >  include/linux/relay.h   | 19 ++++++++++++++-
> >  kernel/relay.c          | 52 +++++++++++++++++++++++++++++++++++------
> >  kernel/trace/blktrace.c | 22 ++---------------
> >  3 files changed, 65 insertions(+), 28 deletions(-)
> 
> Hi Andrew,
> 
> Any comments on the series?
> 

We're at -rc7 now, so I'll park this for consideration for -rc1.

Re: [PATCH v3 0/4] relayfs: misc changes
Posted by Jason Xing 7 months ago
On Fri, May 23, 2025 at 7:34 AM Andrew Morton <akpm@linux-foundation.org> wrote:
>
> On Fri, 23 May 2025 07:16:35 +0800 Jason Xing <kerneljasonxing@gmail.com> wrote:
>
> > On Sun, May 18, 2025 at 10:57 AM Jason Xing <kerneljasonxing@gmail.com> wrote:
> > >
> > > From: Jason Xing <kernelxing@tencent.com>
> > >
> > > The series mostly focuses on the error counters which helps every user
> > > debug their own kernel module.
> > >
> > > ---
> > > Note: this series is made on top of this cleanup[1] and unmerged commit[2]
> > > [1]: https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git/log/?h=mm-nonmm-unstable
> > > [2]: https://lore.kernel.org/all/20250507134225.63248-1-kerneljasonxing@gmail.com/
> > >
> > > Jason Xing (4):
> > >   relayfs: support a counter tracking if per-cpu buffers is full
> > >   relayfs: introduce getting relayfs statistics function
> > >   blktrace: use rbuf->stats.full as a drop indicator in relayfs
> > >   relayfs: support a counter tracking if data is too big to write
> > >
> > >  include/linux/relay.h   | 19 ++++++++++++++-
> > >  kernel/relay.c          | 52 +++++++++++++++++++++++++++++++++++------
> > >  kernel/trace/blktrace.c | 22 ++---------------
> > >  3 files changed, 65 insertions(+), 28 deletions(-)
> >
> > Hi Andrew,
> >
> > Any comments on the series?
> >
>
> We're at -rc7 now, so I'll park this for consideration for -rc1.

Okay. IIUC, it seems probably two or three weeks later. I think I'll
keep track of the series and remind you at that time just in case you
might forget it :)

Thanks,
Jason