From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f42.google.com (mail-wm1-f42.google.com [209.85.128.42]) (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 E466D32ED58 for ; Thu, 28 May 2026 07:26:18 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.42 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779953181; cv=none; b=jXkLsFM6OxKI2JbLFmFsb+b9cdaFSKuV3YAG1hxeHgyXTLxQguJz/4Bu338JYypesGbuNkUBxfaFjc0ZdQjotfVuz/SpQtJxcpyAvgnYKqeeD4mROKvx6uOa+IDq9kapdRpSAI79vtSpXHKFuPzTJ9zxArc18LBKO3MMoyb3hOQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779953181; c=relaxed/simple; bh=mN22yLYavxQD0qrKuY/241cKoIVJqcbyDsB4JLlHb8M=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=MlQFU8Xmevb6ki4IqvvWoxOvwfsRpcMu1X7GFBz0vq2CFRk2Ue5q3JU7dXzVjVwuIIE86CcJAVRHCFC79BpPU9SAPyJxQ9LjmvTXI3tB5wx3IIE8vsDYWRFNxaAdflGlBYfcK0XcsGE/xTrhiQVI1kn2UmuTU60sapPfi9cH7Wk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=ffwll.ch; spf=none smtp.mailfrom=ffwll.ch; dkim=pass (1024-bit key) header.d=ffwll.ch header.i=@ffwll.ch header.b=Hf+lQ9Qg; arc=none smtp.client-ip=209.85.128.42 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=ffwll.ch Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=ffwll.ch Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=ffwll.ch header.i=@ffwll.ch header.b="Hf+lQ9Qg" Received: by mail-wm1-f42.google.com with SMTP id 5b1f17b1804b1-49068493267so28832045e9.1 for ; Thu, 28 May 2026 00:26:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ffwll.ch; s=google; t=1779953177; x=1780557977; darn=vger.kernel.org; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:mail-followup-to:message-id:subject:cc:to :from:date:from:to:cc:subject:date:message-id:reply-to; bh=3Okm57YHWqP6DudQxxBoKKLC7I9Qfb5iqn9tysYDvME=; b=Hf+lQ9QgAJffwyKaEcDMtlpnRUZRqcszeDeZjaZ+s0HkYHXDvxC17xaplNmaYs1Rtz zgwbmAvzb+26qF+jju9dFJMzXewM4ssLw1ohssBFjQND1ELNfYZDecIm9tl9um1g5Y8m iJdavpmDCp0GOAZubX1JOm0SN6uFbWlFIfbBQ= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1779953177; x=1780557977; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:mail-followup-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=3Okm57YHWqP6DudQxxBoKKLC7I9Qfb5iqn9tysYDvME=; b=H9kxR9LuF8MDpGI0czCN8ICSxHKz00jSJs1BgMvXowFT4R6iDIw4nO45+RkBa2Jifp /d4aiPAP26IlNexfXNqm100qIj5dfUt1tCrh6GimCvJYTbThQSDOHmqKWUyQ9KQgHBtL /4CuMf45ce+dxjENthqBA0q88qpq17F4CUz8XRmT2LQBH0n/Qm5NvMP3Cq+jRcsg9+nv PFzaN2sySWuocXIOc09gfpUVjDP0wBIvRHrXKt/nM09o8ogbs4i+aBHY4DpefN/blrxb +LQLsNUav9Z72jEEJDyUgHjT6kCUveaFzsgVWL46S9BWWwSQb6WRe6fcuI+wpj2jNAMc ZjJw== X-Forwarded-Encrypted: i=1; AFNElJ8zwMAqGUWAzQxo5geVbMRzIoNPKD+5AIftJ5OiqVHW4FYZgVcMLbOP8XHCvjGcwltnzEo2hxq4Cn3D4IY=@vger.kernel.org X-Gm-Message-State: AOJu0Yw7e/VF79oLXSD9xB4DQSHmL7EJKdEU2NjT+NjXELftja0hXIre MnRWm1FEt5e3tYzkWyCiGCPGn8ZayPlTY5Lm0Lefl35VFrXrtvfmO/PFwofyskFxEGk= X-Gm-Gg: Acq92OEoM8egY67UxfjgE0J4XCb91UklJdcBe9eax3aP7dw0TNGQHR8Y2/6eJnOD6NF V+zykioBvUP1jDhMbgH4g9egO/uutbfl2d5Hx6NF7PyhwEyLImF7wnCBbeuX4L13ZTxJup+dy6a eoNm3SIu9iTrjnpC8tfky7O2NiICrQzj8sQn4qzTsi/gbj4HP/OKAYnaQ0C7E0jfa4QiXgfSDNF UOshatuI4thAJBmsB3URuexml35nmfz1fTSkmUSZq/9s1emu0bZ+1MZ6oMo10Uz/4nUgRxr2Wah c8KXdrsVhIlcYUy3/rTbsYe7ikNNDAyaFaBw0G13UypwiSh2PXw3MRUW7PHL8CEXG8Tt2LDTVqw 52mB1sEWyHp3aw04VmDdQgnIPEHXlloPQ+jPOrSKNsFvilnpQsJPQNMJ8ZEeXXYhOjL0ke61SQA 41IXM75J4ni0GcOplSfmxt4IP86wPOh9rcw4d2no1RdowZFlbAr/EcYpvf X-Received: by 2002:a05:600c:4510:b0:48f:e230:72fc with SMTP id 5b1f17b1804b1-490428ee268mr403369655e9.33.1779953177104; Thu, 28 May 2026 00:26:17 -0700 (PDT) Received: from phenom.ffwll.local ([2a02:168:57f4:0:5485:d4b2:c087:b497]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-4909235f98bsm30377995e9.1.2026.05.28.00.26.16 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 28 May 2026 00:26:16 -0700 (PDT) Date: Thu, 28 May 2026 09:26:14 +0200 From: Simona Vetter To: Chen-Yu Tsai Cc: Arnd Bergmann , Inki Dae , Seung-Woo Kim , Kyungmin Park , David Airlie , Simona Vetter , Krzysztof Kozlowski , Marek Szyprowski , Arnd Bergmann , Alim Akhtar , dri-devel@lists.freedesktop.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] drm/exynos: fix size_t format string Message-ID: Mail-Followup-To: Chen-Yu Tsai , Arnd Bergmann , Inki Dae , Seung-Woo Kim , Kyungmin Park , David Airlie , Simona Vetter , Krzysztof Kozlowski , Marek Szyprowski , Arnd Bergmann , Alim Akhtar , dri-devel@lists.freedesktop.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org References: <20260527194525.45762-1-arnd@kernel.org> 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=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-Operating-System: Linux phenom 6.19.10+deb14-amd64 On Wed, May 27, 2026 at 11:03:03PM +0200, Chen-Yu Tsai wrote: > On Wed, May 27, 2026 at 9:45 PM Arnd Bergmann wrote: > > > > From: Arnd Bergmann > > > > The exynos_gem->base.size argument is a size_t rather than an > > unsigned long, so adapt the printk() format string accordingly: > > > > In file included from drivers/gpu/drm/exynos/exynos_drm_gem.c:16: > > drivers/gpu/drm/exynos/exynos_drm_gem.c: In function 'exynos_drm_alloc_buf': > > drivers/gpu/drm/exynos/exynos_drm_gem.c:69:49: error: format '%lx' expects argument of type 'long unsigned int', but argument 6 has type 'size_t' {aka 'unsigned int'} [-Werror=format=] > > 69 | DRM_DEV_DEBUG_KMS(drm_dev_dma_dev(dev), "dma_addr(0x%lx), size(0x%lx)\n", > > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > > 70 | (unsigned long)exynos_gem->dma_addr, exynos_gem->base.size); > > | ~~~~~~~~~~~~~~~~~~~~~ > > | | > > | size_t {aka unsigned int} > > > > The dma_addr in the same line is already printed using a cast > > to unsigned long, so change that similarly to use the correct > > %pad format. > > > > Fixes: 11e898373fba ("drm/exynos: Drop exynos_drm_gem.size field") > > Signed-off-by: Arnd Bergmann > > I posted the same patch [1] one and a half months ago when the culprit > patch was queued up and I got a report about this from the bots. > > Inki hasn't picked up my fix yet. > > And it seems the exynos-drm-next stuff was only merged into drm-next > yesterday, now causing the warnings to pop up all over. Yeah that's some pretty bad fail for exynos and I guess also for Dave for not compile-testing on 32bit before pushing drm-next out. I've noticed it just this morning trying to do some stuff. Picked up Arnd's patch as the first one that I've found (after I've typed it myself already, but hey let's at least give some credit). Cheers, Sima > > ChenYu > > [1] https://lore.kernel.org/all/20260408064936.1342321-1-wenst@chromium.org/ > > > --- > > drivers/gpu/drm/exynos/exynos_drm_gem.c | 8 ++++---- > > 1 file changed, 4 insertions(+), 4 deletions(-) > > > > diff --git a/drivers/gpu/drm/exynos/exynos_drm_gem.c b/drivers/gpu/drm/exynos/exynos_drm_gem.c > > index 297a93b087cd..8e357f2beb9e 100644 > > --- a/drivers/gpu/drm/exynos/exynos_drm_gem.c > > +++ b/drivers/gpu/drm/exynos/exynos_drm_gem.c > > @@ -66,8 +66,8 @@ static int exynos_drm_alloc_buf(struct exynos_drm_gem *exynos_gem, bool kvmap) > > if (kvmap) > > exynos_gem->kvaddr = exynos_gem->cookie; > > > > - DRM_DEV_DEBUG_KMS(drm_dev_dma_dev(dev), "dma_addr(0x%lx), size(0x%lx)\n", > > - (unsigned long)exynos_gem->dma_addr, exynos_gem->base.size); > > + DRM_DEV_DEBUG_KMS(drm_dev_dma_dev(dev), "dma_addr(%pad), size(0x%zx)\n", > > + &exynos_gem->dma_addr, exynos_gem->base.size); > > return 0; > > } > > > > @@ -80,8 +80,8 @@ static void exynos_drm_free_buf(struct exynos_drm_gem *exynos_gem) > > return; > > } > > > > - DRM_DEV_DEBUG_KMS(dev->dev, "dma_addr(0x%lx), size(0x%lx)\n", > > - (unsigned long)exynos_gem->dma_addr, exynos_gem->base.size); > > + DRM_DEV_DEBUG_KMS(dev->dev, "dma_addr(0x%pad), size(0x%zx)\n", > > + &exynos_gem->dma_addr, exynos_gem->base.size); > > > > dma_free_attrs(drm_dev_dma_dev(dev), exynos_gem->base.size, exynos_gem->cookie, > > (dma_addr_t)exynos_gem->dma_addr, > > -- > > 2.39.5 > > -- Simona Vetter Software Engineer http://blog.ffwll.ch