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