Created attachment 38212 [details]
kernel oops message
Bug detailed description:
When operation system begin to run, the i915 module has a dependence with not sure process(if someone make sure it, please tell me the methods).So, it was unable to rmmod.To think that process may be vtconsole, I kill it by echo "0".Then, there is nothing dependence with i915.Unfortunately, kernel will finally be panic when rmmod i915.
2.(make sure i915 module has just one dependence) lsmod | grep i915
3.echo 0 >/sys/class/vtconsole/vtcon1/bind
Created attachment 38213 [details]
kernel oops dmesg
More than likely coupled to the memory leak, though Daniel Vetter's unload patches are obviously required here as well.
Yeah this looks like an issue that was fixed in Daniel's patchset. I think I have those patches included in my edp-testing branch at git://git.kernel.org/pub/scm/linux/kernel/git/jbarnes/drm-intel.git, can you give it a try?
OK.I will try it
From: email@example.com [mailto:firstname.lastname@example.org]
Sent: Thursday, September 02, 2010 12:49 AM
To: Wang, Jinjin
Subject: [Bug 29828] Kernel OOPs when rmmod i915
--- Comment #3 from Jesse Barnes <email@example.com> 2010-09-01 09:49:20 PDT ---
Yeah this looks like an issue that was fixed in Daniel's patchset. I think I
have those patches included in my edp-testing branch at
git://git.kernel.org/pub/scm/linux/kernel/git/jbarnes/drm-intel.git, can you
give it a try?
Created attachment 38461 [details] [review]
I builded the kernel source which from git://git.kernel.org/pub/scm/linux/kernel/git/jbarnes/drm-intel.git and tested it. But the Oop still has as before.
Jinjin can you verify the bug against
that has both the module unload fixes from Daniel Vetter and the handle leak fix, which I think is the likely suspect here.
I tested with (drm-intel-staging)86fd065f75d5ed9e404d4fd124c5e67f9ac4b9fd and found that it was not OOPS after rmmod i915.
Thanks! Just to spoil the fun, we won't been pushing Daniel's module unload fixes upstream just yet.
The current proposed set of patches for upstream are in drm-intel-fixes and include the fix for the handle leak (which I think is the real culprit here). It will be vital to repeat the test once we have negotiated which patches Linus will pull to see if we have successfully fixed this issue in the upstream kernel.
Ok, drm-intel-fixes now represents what is upstream. Please can you verify that we have successfully fixed the leak on that branch.
I tested with Kernel: (drm-intel-fixes)205c6c88f8a8d274cacfaf84f46f5bbb06f4ac66 and
found that it was not OOPS after rmmod i915.
dmesg:Console: switching to colour VGA+ 80x25
drm: unregistered panic notifier
[drm] Module unloaded
That is a relief, thanks.
Thanks, Chris.Now. I do verify it!!
Closing old verified.