From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f41.google.com (mail-wm1-f41.google.com [209.85.128.41]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id BFCA430FC2E for ; Fri, 8 May 2026 18:47:06 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.41 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778266028; cv=none; b=QQVSpqZ85UArEBXh5MnGwhYih40vWheUHLeB6hD1EVEKvp7Rqyo4CQNOgsqHlADnWL5PIRpb4Od19Somt/SirMlPKEaRptlVzUCWCd2KK9yfZP3P1D50ZJAw5csXqjcTfv3zAgzkAzMOBmgJq7aS4JJpnQAl6fMXJgxAJR50ViQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778266028; c=relaxed/simple; bh=eaqpf2ivuTcwfivhoAiZkqb796nN4j9k5Bv6JOQRQXY=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=XLVZvtevUL8WmtMWTc/Z3/HO2yIWTyJoaOG2b0J2puoFBk7QRRRL8JTDUX5FwvCvWCb6e0SfWozFyEJvvYgZAB+dR6QCgKKUNMhaXoTsIlWBVdqqKH8gjtSRkixSgEnskfLb28DAH8riCQanZOzjiIOttyQ6jDDokUhrRXTbxXE= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=jBL9NaRg; arc=none smtp.client-ip=209.85.128.41 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="jBL9NaRg" Received: by mail-wm1-f41.google.com with SMTP id 5b1f17b1804b1-488a14c31eeso16139895e9.0 for ; Fri, 08 May 2026 11:47:06 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1778266025; x=1778870825; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:from:to:cc:subject:date :message-id:reply-to; bh=EoLj3O/cwM27sEMxcTUy7jHQ6GFmhl6pvmSQAeSJBkg=; b=jBL9NaRgp2YhfIs8L5slZQZoM8CmLA9+diokmlLeAfBOzQEMad151VPH3uyRD0mbbX pNw28DmDSrZ+YbSZ/ncBMUgMtygw8NawCvBqIpkiQhDmMrJwGFdQWtEYW9JrRUd1ECxD +HMLd8eOxcN9Do5BI5yRNWxy6+glfjVnGns+onIAb9drPyTvLjda3Qzu+3lIszaEXDok xoq4o4OieH+MrGYYuRDOFAtjY1RzjjyOH5dQT5rV+IGmxryi2ykV5jA5hHXdBuDuWb9A RiZxhFNyyuZzXieTx315zPhxHsiUDpMjX9GQ5Dt2cvRie5A5o4/yxIUVmY0B25gEIJPq b/Lg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1778266025; x=1778870825; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=EoLj3O/cwM27sEMxcTUy7jHQ6GFmhl6pvmSQAeSJBkg=; b=rgpPdGDVv343Lr/PuJkMeDnEQLNu/uw2Ux4CDV9oou4LQmHkKUcKucMjAgIVY5fMNu LS+7HXPqEHJEYPxDs1lEcYqq0vhjp70I8ekT8AaOJSo+P8iamBBeWDPrTkLbkj0I0i+a 9HYSEXV7Btr3Um8ijSotEeowhztoMtKgL1Nl+/VWqI/C5DnKFpHReCyWMYPTJEKtl2W0 ab6FwiLezANKYy2pqmHGUppkfw1b4TDlSRaB+Mryt5Do7PSqdr4dhYaws3sG716PsWBE 437jzk0MG1xoRiJvNo1uRvKzsa3z/almPrikMOqN1XlAliIMTWjlKgI8ZPo9f7vMeSCX ye8Q== X-Forwarded-Encrypted: i=1; AFNElJ+WQHCM9NCn21rD0LYXhFVN+tuHi8pC6PhRJrJ2DI+D6mLJc5mpfKfLh33oLqmv+qFnmUd3ImQbZr3pHqU=@vger.kernel.org X-Gm-Message-State: AOJu0YyvA7Q07WFtbulEVDw+CsHfGijvx6mCLNuFte+MU1hoJVRuufxo WgHTOgoLf7lE6MsjYT4y4wY5VRoQTME8qPiT9zJMCbxH8xazSTO+sea/ X-Gm-Gg: AeBDiethAa9kmnZcxCUsPT8UsacAG1ZH8yigqXD6MPMiY6RuyKCRP43MUZtyRtmr0KC uD/V8ZBwEwPLZz397vO4SlxxBBAO+UJKF8kembOeMCLmHVPk7I9f5/CjS37wbGkX3CpyumY7jNn 0cx85co7bgM0qtN0e9VnoSsw0OPoNGPDdpeKPfR59Wggy1GAQ1KvePDwI5U3GSHYnSmS+5NVelt 578g0XsolNWwwTC5XnZhRXprzRNvc3rRuTc4iiFOMaz+Datb12MGOi9bTfG90CEuYZUwxmDZkWl r4/0oCZArQY9GxxnXTfIRsim8yvjuxKwQpZJsDQiEKYIXUt6ZPIf0T04IYgSaTfAWN42DqcLN4t HbtK2GzFN5y8ISDTE8mBZqMgK52e7ag/tFcnniz6EIfk+VgUSPLWdZkZr0G25wxy+q3LsrNK4UH cxhTjBxUDpyic/Az3xDnMRfWmi0CFS6kUYbetzkosyLhZPbCE7IYI8RizNl9JWWykmIDUrdCoN6 ig= X-Received: by 2002:a05:600c:4703:b0:488:936a:6220 with SMTP id 5b1f17b1804b1-48e676ac055mr59779135e9.21.1778266025052; Fri, 08 May 2026 11:47:05 -0700 (PDT) Received: from pumpkin (82-69-66-36.dsl.in-addr.zen.co.uk. [82.69.66.36]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-48e6d895781sm14304815e9.0.2026.05.08.11.47.04 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 08 May 2026 11:47:04 -0700 (PDT) Date: Fri, 8 May 2026 19:47:03 +0100 From: David Laight To: Arnd Bergmann Cc: Bin Du , Nirujogi Pratap , Mauro Carvalho Chehab , Sultan Alsawaf , Svetoslav Stoilov , Sakari Ailus , Arnd Bergmann , linux-media@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] [v2] media: platform: amd: avoid -Wformat-security warning Message-ID: <20260508194703.058c59e6@pumpkin> In-Reply-To: <20260508124402.3082615-1-arnd@kernel.org> References: <20260508124402.3082615-1-arnd@kernel.org> X-Mailer: Claws Mail 4.1.1 (GTK 3.24.38; arm-unknown-linux-gnueabihf) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Fri, 8 May 2026 14:43:49 +0200 Arnd Bergmann wrote: > From: Arnd Bergmann > > The newly added kthread_run() call passes a string variable as an sprintf() > style format, which gcc warns about when -Wformat-security is enabled: > > drivers/media/platform/amd/isp4/isp4_subdev.c: In function 'isp4sd_start_resp_proc_threads': > include/linux/kthread.h:71:16: error: format not a string literal and no format arguments [-Werror=format-security] > 71 | struct task_struct *__k \ > | ^~~~~~~~~~~ > drivers/media/platform/amd/isp4/isp4_subdev.c:596:38: note: in expansion of macro 'kthread_run' > 596 | thread_ctx->thread = kthread_run(isp4sd_fw_resp_thread, > | ^~~~~~~~~~~ > > Use an indirect "%s" format to do this safely, avoiding the warning. > > Fixes: 4e5e7a7ddb4a ("media: platform: amd: isp4 subdev and firmware loading handling added") > Tested-by: Bin Du > Signed-off-by: Arnd Bergmann > --- > v2: avoid overly long line > --- > drivers/media/platform/amd/isp4/isp4_subdev.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/media/platform/amd/isp4/isp4_subdev.c b/drivers/media/platform/amd/isp4/isp4_subdev.c > index 48deea79ce6c..427e9ba910bb 100644 > --- a/drivers/media/platform/amd/isp4/isp4_subdev.c > +++ b/drivers/media/platform/amd/isp4/isp4_subdev.c > @@ -595,6 +595,7 @@ static int isp4sd_start_resp_proc_threads(struct isp4_subdev *isp_subdev) > > thread_ctx->thread = kthread_run(isp4sd_fw_resp_thread, > &isp_subdev->isp_resp_para[i], > + "%s", > isp4sd_thread_name[i]); I don't think the line break is needed. I always prefer a line break between the format and arguments when the whole lot doesn't fit on one line; but here it fits. -- David > if (IS_ERR(thread_ctx->thread)) { > dev_err(dev, "create thread [%d] fail\n", i);