From: Jaegeuk Kim <jaegeuk@kernel.org>
To: Steven Rostedt <rostedt@goodmis.org>
Cc: patchwork-bot+f2fs@kernel.org, linux-kernel@vger.kernel.org,
linux-f2fs-devel@lists.sourceforge.net, mhiramat@kernel.org,
Linus Torvalds <torvalds@linux-foundation.org>,
Douglas RAILLARD <douglas.raillard@arm.com>,
linux-trace-kernel@vger.kernel.org
Subject: Re: [f2fs-dev] [PATCH] f2fs: Fix f2fs_truncate_partial_nodes ftrace event
Date: Tue, 28 Mar 2023 17:36:06 -0700 [thread overview]
Message-ID: <ZCOH9pc3hBKU3G3Z@google.com> (raw)
In-Reply-To: <20230328190306.3337b7b0@gandalf.local.home>
On 03/28, Steven Rostedt wrote:
> On Tue, 07 Mar 2023 17:40:24 +0000
> patchwork-bot+f2fs@kernel.org wrote:
>
> > Hello:
> >
> > This patch was applied to jaegeuk/f2fs.git (dev)
> > by Jaegeuk Kim <jaegeuk@kernel.org>:
> >
> > On Mon, 6 Mar 2023 12:25:49 +0000 you wrote:
> > > From: Douglas Raillard <douglas.raillard@arm.com>
> > >
> > > Fix the nid_t field so that its size is correctly reported in the text
> > > format embedded in trace.dat files. As it stands, it is reported as
> > > being of size 4:
> > >
> > > field:nid_t nid[3]; offset:24; size:4; signed:0;
> > >
> > > [...]
> >
> > Here is the summary with links:
> > - [f2fs-dev] f2fs: Fix f2fs_truncate_partial_nodes ftrace event
> > https://git.kernel.org/jaegeuk/f2fs/c/3ccc99d5a4ea
> >
> > You are awesome, thank you!
>
> I'm hoping you are not waiting for the merge window to get this in. Please
> get it to Linus before rc5 is out, because I plan on adding my patch[1]
> which will cause this to fail the build at rc5.
>
> The above is a fix to bug that causes unwanted results. Please get it into
> mainline ASAP.
Thanks for heads-up. I sent a pull request having the above patch to Linus.
https://lore.kernel.org/linux-f2fs-devel/ZCOHd4jYn8zUCEZ0@google.com/T/#u
Thanks,
>
> [1] https://patchwork.kernel.org/project/linux-trace-kernel/patch/20230309221302.642e82d9@gandalf.local.home/
>
> -- Steve
_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
WARNING: multiple messages have this Message-ID (diff)
From: Jaegeuk Kim <jaegeuk@kernel.org>
To: Steven Rostedt <rostedt@goodmis.org>
Cc: patchwork-bot+f2fs@kernel.org,
Douglas RAILLARD <douglas.raillard@arm.com>,
chao@kernel.org, mhiramat@kernel.org,
linux-f2fs-devel@lists.sourceforge.net,
linux-kernel@vger.kernel.org, linux-trace-kernel@vger.kernel.org,
Linus Torvalds <torvalds@linux-foundation.org>
Subject: Re: [f2fs-dev] [PATCH] f2fs: Fix f2fs_truncate_partial_nodes ftrace event
Date: Tue, 28 Mar 2023 17:36:06 -0700 [thread overview]
Message-ID: <ZCOH9pc3hBKU3G3Z@google.com> (raw)
In-Reply-To: <20230328190306.3337b7b0@gandalf.local.home>
On 03/28, Steven Rostedt wrote:
> On Tue, 07 Mar 2023 17:40:24 +0000
> patchwork-bot+f2fs@kernel.org wrote:
>
> > Hello:
> >
> > This patch was applied to jaegeuk/f2fs.git (dev)
> > by Jaegeuk Kim <jaegeuk@kernel.org>:
> >
> > On Mon, 6 Mar 2023 12:25:49 +0000 you wrote:
> > > From: Douglas Raillard <douglas.raillard@arm.com>
> > >
> > > Fix the nid_t field so that its size is correctly reported in the text
> > > format embedded in trace.dat files. As it stands, it is reported as
> > > being of size 4:
> > >
> > > field:nid_t nid[3]; offset:24; size:4; signed:0;
> > >
> > > [...]
> >
> > Here is the summary with links:
> > - [f2fs-dev] f2fs: Fix f2fs_truncate_partial_nodes ftrace event
> > https://git.kernel.org/jaegeuk/f2fs/c/3ccc99d5a4ea
> >
> > You are awesome, thank you!
>
> I'm hoping you are not waiting for the merge window to get this in. Please
> get it to Linus before rc5 is out, because I plan on adding my patch[1]
> which will cause this to fail the build at rc5.
>
> The above is a fix to bug that causes unwanted results. Please get it into
> mainline ASAP.
Thanks for heads-up. I sent a pull request having the above patch to Linus.
https://lore.kernel.org/linux-f2fs-devel/ZCOHd4jYn8zUCEZ0@google.com/T/#u
Thanks,
>
> [1] https://patchwork.kernel.org/project/linux-trace-kernel/patch/20230309221302.642e82d9@gandalf.local.home/
>
> -- Steve
next prev parent reply other threads:[~2023-03-29 0:36 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-03-06 12:25 [f2fs-dev] [PATCH] f2fs: Fix f2fs_truncate_partial_nodes ftrace event Douglas RAILLARD
2023-03-06 12:25 ` Douglas RAILLARD
2023-03-06 12:51 ` [f2fs-dev] " Mukesh Ojha
2023-03-06 12:51 ` Mukesh Ojha
2023-03-06 16:15 ` [f2fs-dev] " Steven Rostedt
2023-03-06 16:15 ` Steven Rostedt
2023-03-06 17:25 ` [f2fs-dev] " Steven Rostedt
2023-03-06 17:25 ` Steven Rostedt
2023-03-07 6:01 ` [f2fs-dev] " Chao Yu
2023-03-07 6:01 ` Chao Yu
2023-03-07 17:40 ` [f2fs-dev] " patchwork-bot+f2fs
2023-03-07 17:40 ` patchwork-bot+f2fs
2023-03-28 23:03 ` Steven Rostedt
2023-03-28 23:03 ` Steven Rostedt
2023-03-29 0:36 ` Jaegeuk Kim [this message]
2023-03-29 0:36 ` Jaegeuk Kim
2023-03-29 1:01 ` Steven Rostedt
2023-03-29 1:01 ` Steven Rostedt
2023-03-29 17:41 ` Linus Torvalds
2023-03-29 17:41 ` Linus Torvalds
2023-08-14 13:32 ` Kajetan Puchalski
2023-08-14 13:32 ` Kajetan Puchalski
2023-08-14 14:39 ` [f2fs-dev] " Kajetan Puchalski
2023-08-14 14:39 ` Kajetan Puchalski
2023-08-14 20:37 ` [f2fs-dev] " Jaegeuk Kim
2023-08-14 20:37 ` Jaegeuk Kim
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ZCOH9pc3hBKU3G3Z@google.com \
--to=jaegeuk@kernel.org \
--cc=douglas.raillard@arm.com \
--cc=linux-f2fs-devel@lists.sourceforge.net \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-trace-kernel@vger.kernel.org \
--cc=mhiramat@kernel.org \
--cc=patchwork-bot+f2fs@kernel.org \
--cc=rostedt@goodmis.org \
--cc=torvalds@linux-foundation.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.