2021-09-13 10:06:44

by Rolf Eike Beer

[permalink] [raw]
Subject: [PATCH] workqueue: annotate alloc_workqueue() as printf

This also enables checking of allows alloc_ordered_workqueue().

Signed-off-by: Rolf Eike Beer <[email protected]>
---
include/linux/workqueue.h | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/include/linux/workqueue.h b/include/linux/workqueue.h
index 2ebef6b1a3d6..49e93f5b46ad 100644
--- a/include/linux/workqueue.h
+++ b/include/linux/workqueue.h
@@ -399,9 +399,9 @@ extern struct workqueue_struct *system_freezable_power_efficient_wq;
* RETURNS:
* Pointer to the allocated workqueue on success, %NULL on failure.
*/
-struct workqueue_struct *alloc_workqueue(const char *fmt,
- unsigned int flags,
- int max_active, ...);
+__printf(1, 4) struct workqueue_struct *alloc_workqueue(const char *fmt,
+ unsigned int flags,
+ int max_active, ...);

/**
* alloc_ordered_workqueue - allocate an ordered workqueue
--
2.33.0


--
Rolf Eike Beer, emlix GmbH, https://www.emlix.com
Fon +49 551 30664-0, Fax +49 551 30664-11
Gothaer Platz 3, 37083 Göttingen, Germany
Sitz der Gesellschaft: Göttingen, Amtsgericht Göttingen HR B 3160
Geschäftsführung: Heike Jordan, Dr. Uwe Kracke – Ust-IdNr.: DE 205 198 055

emlix - smart embedded open source



2021-09-14 00:47:38

by Tejun Heo

[permalink] [raw]
Subject: Re: [PATCH] workqueue: annotate alloc_workqueue() as printf

On Mon, Sep 13, 2021 at 12:02:56PM +0200, Rolf Eike Beer wrote:
> This also enables checking of allows alloc_ordered_workqueue().
>
> Signed-off-by: Rolf Eike Beer <[email protected]>

Applied to wq/for-5.16.

> +__printf(1, 4) struct workqueue_struct *alloc_workqueue(const char *fmt,
> + unsigned int flags,
> + int max_active, ...);

I broke the line after the return type while applying.

Thanks.

--
tejun