linux-kernel - Re: [PATCH] drm/radeon: fix signed v unsigned print formats

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CADnq5_PZxhjk8xsqzoF_nmrOyYgUjLVNMUDRUEODym7sQ7MEuQ@mail.gmail.com>
Date: Tue, 6 Jan 2026 09:52:43 -0500
From: Alex Deucher <alexdeucher@...il.com>
To: Ben Dooks <ben.dooks@...ethink.co.uk>
Cc: linux-kernel@...r.kernel.org, dri-devel@...ts.freedesktop.org, 
	amd-gfx@...ts.freedesktop.org, simona@...ll.ch, airlied@...il.com, 
	christian.koenig@....com, alexander.deucher@....com
Subject: Re: [PATCH] drm/radeon: fix signed v unsigned print formats

Applied.  thanks!

Alex

On Fri, Jan 2, 2026 at 7:36 PM Ben Dooks <ben.dooks@...ethink.co.uk> wrote:
>
> Fix several places where %ld or %d has been used in place of
> %lu or %u.
>
> Signed-off-by: Ben Dooks <ben.dooks@...ethink.co.uk>
> ---
>  drivers/gpu/drm/radeon/radeon_gem.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/radeon/radeon_gem.c b/drivers/gpu/drm/radeon/radeon_gem.c
> index f86773f3db20..891ef929cfa8 100644
> --- a/drivers/gpu/drm/radeon/radeon_gem.c
> +++ b/drivers/gpu/drm/radeon/radeon_gem.c
> @@ -110,7 +110,7 @@ int radeon_gem_object_create(struct radeon_device *rdev, unsigned long size,
>          */
>         max_size = rdev->mc.gtt_size - rdev->gart_pin_size;
>         if (size > max_size) {
> -               DRM_DEBUG("Allocation size %ldMb bigger than %ldMb limit\n",
> +               DRM_DEBUG("Allocation size %luMb bigger than %luMb limit\n",
>                           size >> 20, max_size >> 20);
>                 return -ENOMEM;
>         }
> @@ -560,7 +560,7 @@ int radeon_gem_set_tiling_ioctl(struct drm_device *dev, void *data,
>         struct radeon_bo *robj;
>         int r = 0;
>
> -       DRM_DEBUG("%d \n", args->handle);
> +       DRM_DEBUG("%u \n", args->handle);
>         gobj = drm_gem_object_lookup(filp, args->handle);
>         if (gobj == NULL)
>                 return -ENOENT;
> @@ -886,7 +886,7 @@ static int radeon_debugfs_gem_info_show(struct seq_file *m, void *unused)
>                         placement = " CPU";
>                         break;
>                 }
> -               seq_printf(m, "bo[0x%08x] %8ldkB %8ldMB %s pid %8ld\n",
> +               seq_printf(m, "bo[0x%08x] %8lukB %8luMB %s pid %8lu\n",
>                            i, radeon_bo_size(rbo) >> 10, radeon_bo_size(rbo) >> 20,
>                            placement, (unsigned long)rbo->pid);
>                 i++;
> --
> 2.37.2.352.g3c44437643
>