Messages in this thread Patch in this message | | | Date | Wed, 21 Sep 2022 08:08:55 -0000 | From | "tip-bot2 for Jules Irenge" <> | Subject | [tip: perf/core] perf/core: Convert snprintf() to scnprintf() |
| |
The following commit has been merged into the perf/core branch of tip:
Commit-ID: 678739d622ae7b75b62d550858b6bf104c43e2df Gitweb: https://git.kernel.org/tip/678739d622ae7b75b62d550858b6bf104c43e2df Author: Jules Irenge <jbi.octave@gmail.com> AuthorDate: Sun, 18 Sep 2022 00:41:08 +01:00 Committer: Ingo Molnar <mingo@kernel.org> CommitterDate: Wed, 21 Sep 2022 10:01:20 +02:00
perf/core: Convert snprintf() to scnprintf()
Coccinelle reports a warning:
WARNING: use scnprintf or sprintf
Adding to that, there has also been some slow migration from snprintf to scnprintf.
This LWN article explains the rationale for this change:
https: //lwn.net/Articles/69419/
No change in behavior.
[ mingo: Improved the changelog. ]
Signed-off-by: Jules Irenge <jbi.octave@gmail.com> Signed-off-by: Ingo Molnar <mingo@kernel.org> --- kernel/events/core.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/kernel/events/core.c b/kernel/events/core.c index 7da5515..c07e9a3 100644 --- a/kernel/events/core.c +++ b/kernel/events/core.c @@ -10952,7 +10952,7 @@ static ssize_t nr_addr_filters_show(struct device *dev, { struct pmu *pmu = dev_get_drvdata(dev); - return snprintf(page, PAGE_SIZE - 1, "%d\n", pmu->nr_addr_filters); + return scnprintf(page, PAGE_SIZE - 1, "%d\n", pmu->nr_addr_filters); } DEVICE_ATTR_RO(nr_addr_filters); @@ -10963,7 +10963,7 @@ type_show(struct device *dev, struct device_attribute *attr, char *page) { struct pmu *pmu = dev_get_drvdata(dev); - return snprintf(page, PAGE_SIZE-1, "%d\n", pmu->type); + return scnprintf(page, PAGE_SIZE - 1, "%d\n", pmu->type); } static DEVICE_ATTR_RO(type); @@ -10974,7 +10974,7 @@ perf_event_mux_interval_ms_show(struct device *dev, { struct pmu *pmu = dev_get_drvdata(dev); - return snprintf(page, PAGE_SIZE-1, "%d\n", pmu->hrtimer_interval_ms); + return scnprintf(page, PAGE_SIZE - 1, "%d\n", pmu->hrtimer_interval_ms); } static DEFINE_MUTEX(mux_interval_mutex);
| |