On Tue, Jun 03, 2014 at 01:51:40PM -0700, gregkh@xxxxxxxxxxxxxxxxxxx wrote:
>
> The patch below does not apply to the 3.14-stable tree.
> If someone wants it applied there, or to any other stable or longterm
> tree, then please email the backport, including the original git commit
> id to <stable@xxxxxxxxxxxxxxx>.
>
> thanks,
>
> greg k-h
Nevermind, I got this to apply, some other patches had to be added
first, sorry for the noise.
greg k-h
>
> ------------------ original commit in Linus's tree ------------------
>
> >From 8c79bae6a30f606b7a4e17c994bc5f72f8fdaf11 Mon Sep 17 00:00:00 2001
> From: Alex Deucher <alexdeucher@xxxxxxxxx>
> Date: Wed, 16 Apr 2014 09:42:22 -0400
> Subject: [PATCH] drm/radeon/si: make sure mc ucode is loaded before checking
> the size
>
> Avoid a possible segfault.
>
> Noticed-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
> Signed-off-by: Alex Deucher <alexander.deucher@xxxxxxx>
> Cc: stable@xxxxxxxxxxxxxxx
>
> diff --git a/drivers/gpu/drm/radeon/si.c b/drivers/gpu/drm/radeon/si.c
> index 86f8c9c87f5c..ac708e006180 100644
> --- a/drivers/gpu/drm/radeon/si.c
> +++ b/drivers/gpu/drm/radeon/si.c
> @@ -1472,11 +1472,13 @@ int si_mc_load_microcode(struct radeon_device *rdev)
> const __be32 *fw_data;
> u32 running, blackout = 0;
> u32 *io_mc_regs;
> - int i, regs_size, ucode_size = rdev->mc_fw->size / 4;
> + int i, regs_size, ucode_size;
>
> if (!rdev->mc_fw)
> return -EINVAL;
>
> + ucode_size = rdev->mc_fw->size / 4;
> +
> switch (rdev->family) {
> case CHIP_TAHITI:
> io_mc_regs = (u32 *)&tahiti_io_mc_regs;
>
> --
> To unsubscribe from this list: send the line "unsubscribe stable" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html