Re: [PATCH] sx tty_driver add .owner field remove MOD_INC_DEC_USE_COUNT

[Hanna Linder (hannal@us.ibm.com)]

Heh. I figured out why I deleted it, it was not a mistake. The whole
function is commented out! Except the func_enter and exit calls which
are printk wrappers. Besides, no one ever calls this function anyway...

So leave it out. No need to put it back in...

Hanna


--On Friday, May 30, 2003 06:05:24 PM +1000 Rusty Russell <rusty@xxxxxxxxxxx> wrote:
> In message <12430000.1054244110@w-hlinder> you write:
>> --On Thursday, May 29, 2003 12:02:50 PM -0700 Greg KH <greg@xxxxxxxxx> wrote:
>> 
>> > 
>> > Ick, this patch should be reverted, it should not be removing
>> > sx_hungup() for no reason.  I think Hanna agrees with this.
>> 
>> Yup. Sorry. Not sure what happened there. Here is the patch
>> to replace the sx_hangup function. This is based off 2.5.70-bk3
>> and I have compiled it but dont have the hardware to test it.
> 
> Yes, but I don't think you need to put back the comment about
> decrementing use counts 8)
> 
>> +/* I haven't the foggiest why the decrement use count has to happen
>> +   here. The whole linux serial drivers stuff needs to be redesigned.
>> +   My guess is that this is a hack to minimize the impact of a bug
>> +   elsewhere. Thinking about it some more. (try it sometime) Try
>> +   running minicom on a serial port that is driven by a modularized
>> +   driver. Have the modem hangup. Then remove the driver module. Then
>> +   exit minicom.  I expect an "oops".  -- REW */
> 
> Cheers,
> Rusty.
> --
>   Anyone who quotes me in their sig is an idiot. -- Rusty Russell.
> 


-
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/