| Message ID | 20240925085524.3525-1-zhujun2@cmss.chinamobile.com (mailing list archive) |
|---|---|
| State | Deferred |
| Headers | show |
| Series | tools/net:Fix the wrong format specifier | expand |
On Wed, Sep 25, 2024 at 01:55:24AM -0700, Zhu Jun wrote: > The format specifier of "unsigned int" in printf() should be "%u", not > "%d". > > Signed-off-by: Zhu Jun <zhujun2@cmss.chinamobile.com> Hi Zhu Jun, Thanks for your patch. While this change looks good to me it looks like it is for net-next. Currently net-next is closed for the v6.12 merge window. So please repost this patch once it reopens, after v6.12-rc1 has been released. I expect that to happen early next week. Also, please explicitly target net-next patches for that tree like this: Subject: [PATCH net-next] For reference, netdev processes are documented here: https://docs.kernel.org/process/maintainer-netdev.html
On Wed, Sep 25, 2024 at 08:53:54PM +0100, Simon Horman wrote: > On Wed, Sep 25, 2024 at 01:55:24AM -0700, Zhu Jun wrote: > > The format specifier of "unsigned int" in printf() should be "%u", not > > "%d". > > > > Signed-off-by: Zhu Jun <zhujun2@cmss.chinamobile.com> > > Hi Zhu Jun, > > Thanks for your patch. > > While this change looks good to me it looks like it is for net-next. > Currently net-next is closed for the v6.12 merge window. So please > repost this patch once it reopens, after v6.12-rc1 has been released. > I expect that to happen early next week. > > Also, please explicitly target net-next patches for that tree like this: > > Subject: [PATCH net-next] > > For reference, netdev processes are documented here: > https://docs.kernel.org/process/maintainer-netdev.html Sorry, one more thing, looking at git history I think the prefix for the patch should be 'tools: ynl: '. So perhaps: Subject: [PATCH net-next] tools: ynl: correct page-pool stat format specifier
diff --git a/tools/net/ynl/samples/page-pool.c b/tools/net/ynl/samples/page-pool.c index 332f281ee5cb..e5d521320fbf 100644 --- a/tools/net/ynl/samples/page-pool.c +++ b/tools/net/ynl/samples/page-pool.c @@ -118,7 +118,7 @@ int main(int argc, char **argv) name = if_indextoname(s->ifc, ifname); if (name) printf("%8s", name); - printf("[%d]\t", s->ifc); + printf("[%u]\t", s->ifc); } printf("page pools: %u (zombies: %u)\n",
The format specifier of "unsigned int" in printf() should be "%u", not "%d". Signed-off-by: Zhu Jun <zhujun2@cmss.chinamobile.com> --- tools/net/ynl/samples/page-pool.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)