<div dir="ltr">So, I finally figured this out, and for the benefit of web posterity and the search engines, I'm gonna lay it out:<div>I was using *both* the register_chrdev() and the cdev_*() functions on the same device. </div>
<div><br></div><div>I guess you can't do that. I can't, anyway. Rubini's scull code uses both, and it looks like he uses both on the same device, although I bet he doesn't (haven't looked closely at the code). </div>
<div><br></div><div>I switched to using register_chrdev()/unregister_chrdev() and all is happy now. </div><div><br></div><div>Thanks to all who helped. </div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Mon, Dec 30, 2013 at 10:57 AM, Eric Fowler <span dir="ltr"><<a href="mailto:eric.fowler@gmail.com" target="_blank">eric.fowler@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">Actually, I was working with a homegrown makefile. I made the change you recommended and am still having the problem. <div><br></div><div>I am thinking now about something else as being the problem. <br><br>
</div><div>Is this code legitimate?</div><div><br></div><div>//// file scope </div><div><br></div><div>struct cdev my_cdev; </div><div><br></div><div>///inside init function</div><div>cdev_init(&my_cdev, ....); </div>
<div>cdev_add(&my_cdev, ....); </div><div><br></div><div><br></div><div>///exit fxn</div><div>cdev_del(&my_cdev); </div><div><br></div><div>In other words, the memory for the cdev struct comes from static memory for the driver, not a call to cdev_alloc() or kmalloc(). </div>
<div><br></div><div><br></div><div><br></div><div><br></div></div><div class="gmail_extra"><div><div class="h5"><br><br><div class="gmail_quote">On Mon, Dec 30, 2013 at 10:18 AM, Rajat Sharma <span dir="ltr"><<a href="mailto:fs.rajat@gmail.com" target="_blank">fs.rajat@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>Hi Eric,<br><br></div>I have seen some errors with module reference counting with a nicely written code, but culprit for my case was a missing compilation flag -DMODULE which gives definition of THIS_MODULE, otherwise it is null e.g. for modules which are compiled in kernel, so they are never unloaded. Unless you have some customization done to Makefiles, this definition should be included, but its anyways good to double check and rule out this possibility.<br>
<br></div>-Rajat<br></div><div class="gmail_extra"><br><br><div class="gmail_quote"><div><div>On Mon, Dec 30, 2013 at 9:48 AM, Eric Fowler <span dir="ltr"><<a href="mailto:eric.fowler@gmail.com" target="_blank">eric.fowler@gmail.com</a>></span> wrote:<br>
</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><div dir="ltr">Still working on this. Here is some dmesg spew:<div><br></div><div><div>
[ 514.245846] foobar: module verification failed: signature and/or required key missing - tainting kernel</div>
<div>[ 514.245937] kobject: 'foobar' (f7f060c8): kobject_add_internal: parent: 'module', set: 'module'</div>
<div>[ 514.245951] kobject: 'holders' (f5ff3d40): kobject_add_internal: parent: 'foobar', set: '<NULL>'</div><div>[ 514.245981] kobject: 'notes' (f2d25f80): kobject_add_internal: parent: 'foobar', set: '<NULL>'</div>
<div>[ 514.245987] kobject: 'foobar' (f7f060c8): kobject_uevent_env</div><div>[ 514.245998] kobject: 'foobar' (f7f060c8): fill_kobj_path: path = '/module/foobar'</div></div><div><br></div><div>So it looks like kernel validation is failing. I have printk's in my init fxn that are never turning up in /var/log/messages, until, weirdly, AFTER I remove the device:</div>
<div><br></div><div><insmod device></div><div><div>Dec 30 09:43:03 localhost kernel: [ 514.245846] foobar: module verification failed: signature and/or required key missing - tainting kernel</div><div>Dec 30 09:43:16 localhost fprintd[1085]: ** Message: No devices in use, exit</div>
</div><div><br></div><div><rmmod device></div><div><div>Dec 30 09:45:53 localhost kernel: [ 514.249323] foobar: got device number 248, minor is 0 <<<<----THIS IS IN init() fxn</div><div>Dec 30 09:45:53 localhost kernel: [ 684.102912] unregister_chrdev(248) called for foobar<7>[ 684.102927] kobject: '(null)' (f7f06220): kobject_cleanup, parent (null)</div>
</div><div><br></div><div><insmod></div><div>insmod: ERROR: could not insert module ./foobar.ko: Device or resource busy<br></div><div><br></div><div><br></div><div><br></div><div><br></div><div><br></div><div><br>
</div>
</div><div class="gmail_extra"><div><div><br><br><div class="gmail_quote">On Fri, Dec 27, 2013 at 9:13 PM, <span dir="ltr"><<a href="mailto:Valdis.Kletnieks@vt.edu" target="_blank">Valdis.Kletnieks@vt.edu</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>On Fri, 27 Dec 2013 19:33:50 -0800, Eric Fowler said:<br>
<br>
> I suspect I am doing something wrong in the code with<br>
> register/unregister_chrdev(), but I have been over that code a million<br>
> times. It looks fine.<br>
><br>
> Now:<br>
> insmod the device, OK<br>
> rmmod the device, OK<br>
> Check /proc/devices , device # is present<br>
> insmod the device again, fails with ERROR: could not insert module<br>
> ./foobar.ko: Device or resource busy<br>
<br>
</div>It does smell like an unregister issue. You may want to try adding<br>
printk() calls to print out the return code from register and unregister.<br>
I'm willing to bet that (a) the unegister is failing because somebody<br>
still has a reference on the device, and (b) the second register call fails<br>
because the device already exists, causing your module_init() to bail out.<br>
<br>
The fun is that you may not have taken a reference on the device directly<br>
yourself - you may have called some other get_foo() that ends up taking an<br>
implicit reference under the covers, causing issues when you fail to call<br>
put_foo() at the right place...<br>
</blockquote></div><br><br clear="all"><div><br></div></div></div><span><font color="#888888">-- <br><div dir="ltr">cc:NSA<br></div>
</font></span></div>
<br></div></div>_______________________________________________<br>
Kernelnewbies mailing list<br>
<a href="mailto:Kernelnewbies@kernelnewbies.org" target="_blank">Kernelnewbies@kernelnewbies.org</a><br>
<a href="http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies" target="_blank">http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies</a><br>
<br></blockquote></div><br></div>
</blockquote></div><br><br clear="all"><div><br></div></div></div><span class="HOEnZb"><font color="#888888">-- <br><div dir="ltr">cc:NSA<br></div>
</font></span></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">cc:NSA<br></div>
</div>