On Tue, Oct 5, 2010 at 8:57 AM, Nigel Cunningham <nigel@xxxxxxxxxxxx> wrote: > Running a kernel based on the Rafael's -next tree, under VMware, I get the following oops while booting: Should already be fixed in Linus tree by, http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=f1a28ee238bddfa48c5233543926af65a4445bf6 Dave. > > Entering kdb (current=0xd73e2f70, pid 1024) on processor 0 Oops: (null) > due to oops @ 0xc108bc94 > <d>Modules linked in: ext4 jbd2 crc16 mptspi mptscsih mptbase > <c> > <d>Pid: 1024, comm: plymouthd Not tainted 2.6.36-rc4+ #60 440BX Desktop Reference Platform/VMware Virtual Platform > <d>EIP: 0060:[<c108bc94>] EFLAGS: 00010246 CPU: 0 > EIP is at kfree+0x36/0x88 > <d>EAX: c146ccbd EBX: dc46e980 ECX: 40000400 EDX: c182cd80 > <d>ESI: dfabf800 EDI: dfabf8c0 EBP: dfa7befc ESP: dfa7beec > <d> DS: 007b ES: 007b FS: 00d8 GS: 0033 SS: 0068 > <0>Process plymouthd (pid: 1024, ti=dfa7a000 task=d73e2f70 task.ti=dfa7a000) > <0>Stack: > dfabf800 dc46e980 dfabf800 dfabf8c0 dfa7bf18 c11c4ea0 c11d237c dfabf8c0 > <0> dc46e980 c11c4e13 c11d5bd9 dfa7bf28 c113d3d1 dc437468 dc46e780 dfa7bf34 > <0> c11c4d9d dc437468 dfa7bf40 c11d5f35 dfabf800 dfa7bf68 c11c1e3e dfabf800 > <0>Call Trace: > <0> [<c11c4ea0>] ? drm_master_destroy+0x8d/0xf0 > <0> [<c11d237c>] ? ttm_object_file_destroy+0x0/0xd > <0> [<c11c4e13>] ? drm_master_destroy+0x0/0xf0 > <0> [<c11d5bd9>] ? vmw_master_drop+0x0/0x76 > <0> [<c113d3d1>] ? kref_put+0x39/0x42 > <0> [<c11c4d9d>] ? drm_master_put+0x12/0x1b > [0]more> > Only 'q' or 'Q' are processed at more prompt, input ignored > <0> [<c11d5f35>] ? vmw_postclose+0x1b/0x25 > <0> [<c11c1e3e>] ? drm_release+0x459/0x4cb > <0> [<c1091274>] ? fput+0xcc/0x1b1 > <0> [<c108ec5b>] ? filp_close+0x51/0x5b > <0> [<c108ecbf>] ? sys_close+0x5a/0x88 > <0> [<c1002690>] ? sysenter_do_call+0x12/0x26 > <0>Code: 10 76 72 8d 90 00 00 00 40 c1 ea 0c c1 e2 05 03 15 00 1b 7e c1 66 83 3a 00 79 03 8b 52 0c 8b 0a 84 c9 78 14 66 f7 c1 00 c0 75 04 <0f> 0b eb fe 89 d0 e8 0a 3a fe ff eb 3d 8b 75 04 8b 5a 0c 9c 8f > Call Trace: > [<c11c4ea0>] drm_master_destroy+0x8d/0xf0 > [<c11d237c>] ? ttm_object_file_destroy+0x0/0xd > [<c11c4e13>] ? drm_master_destroy+0x0/0xf0 > [<c11d5bd9>] ? vmw_master_drop+0x0/0x76 > [<c113d3d1>] kref_put+0x39/0x42 > [<c11c4d9d>] drm_master_put+0x12/0x1b > [<c11d5f35>] vmw_postclose+0x1b/0x25 > [<c11c1e3e>] drm_release+0x459/0x4cb > [<c1091274>] fput+0xcc/0x1b1 > [<c108ec5b>] filp_close+0x51/0x5b > [<c108ecbf>] sys_close+0x5a/0x88 > [<c1002690>] sysenter_do_call+0x12/0x26 > > This oops is caused by vmwgfx setting it's dev->devicename to a static char * instead of kmallocing memory. The kfree that's done in drm_master_destroy then explodes :) > > Signed-off-by: Nigel Cunningham <nigel@xxxxxxxxxxxx> > > diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_drv.c b/drivers/gpu/drm/vmwgfx/vmwgfx_drv.c > index 72ec2e2..1ca0ebc 100644 > --- a/drivers/gpu/drm/vmwgfx/vmwgfx_drv.c > +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_drv.c > @@ -343,8 +343,16 @@ static int vmw_driver_load(struct drm_device *dev, unsigned long chipset) > > dev->dev_private = dev_priv; > > - if (!dev->devname) > - dev->devname = vmw_devname; > + if (!dev->devname) { > + dev->devname = kmalloc(strlen(vmw_devname) + 1, GFP_KERNEL); > + if (!dev->devname) { > + DRM_ERROR("Unable to allocate memory for device " > + "name.\n"); > + ret = -ENOMEM; > + goto out_err4; > + } > + strcpy(dev->devname, vmw_devname); > + } > > if (dev_priv->capabilities & SVGA_CAP_IRQMASK) { > ret = drm_irq_install(dev); > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ > _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
