[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <9C317446-0259-445B-8AE7-D91E1E5ABF93@gmail.com>
Date: Fri, 24 May 2024 16:12:57 +0800
From: zhang warden <zhangwarden@...il.com>
To: Dan Carpenter <dan.carpenter@...aro.org>
Cc: Josh Poimboeuf <jpoimboe@...nel.org>,
Miroslav Benes <mbenes@...e.cz>,
Jiri Kosina <jikos@...nel.org>,
Petr Mladek <pmladek@...e.com>,
Joe Lawrence <joe.lawrence@...hat.com>,
live-patching@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH] livepatch: introduce klp_func called interface
> On May 23, 2024, at 22:22, Dan Carpenter <dan.carpenter@...aro.org> wrote:
>
> Always run your patches through checkpatch.
>
> So this patch is so that testers can see if a function has been called?
> Can you not get the same information from gcov or ftrace?
>
> There are style issues with the patch, but it's not so important until
> the design is agreed on.
>
> regards,
> dan carpenter
Hi, Dan.
This patch have format issues as Markus said. A newer version of this patch is sent which is checked by ./scripts/checkpatch.pl
Thanks for your suggestions.
Regards,
Wardenjohn
Powered by blists - more mailing lists