linux-kernel - Re: [linux-dvb] DST/BT878 module customization (.. was: Critical points about ...)

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <463F9B4F.7020708@linuxtv.org>
Date:	Mon, 07 May 2007 17:34:07 -0400
From:	Michael Krufky <mkrufky@...uxtv.org>
To:	Manu Abraham <abraham.manu@...il.com>
CC:	Mauro Carvalho Chehab <mchehab@...radead.org>,
	linux-dvb@...uxtv.org, linux-kernel@...r.kernel.org
Subject: Re: [linux-dvb] DST/BT878 module customization (.. was: Critical
 points about ...)

Manu Abraham wrote:
> Mauro Carvalho Chehab wrote:
>> Hi Manu,
>>
>>> From my side, quite some time has been put forward to write that mail.
>>> Inspite of that if you feel that you do have to go your own way, then
>>> it is completely upto you. I would say: do as you feel in such a case.
>> The point is that those issues are pending for a long time, and they
>> should be solved, especially the module removal issue (*)
>>
>> If you are so afraid about applying those changes, maybe the better is
>> to apply those patches at v4l-dvb tree and at -mm, asking people to
>> test.
>>
>> We may hold their commit on kernel mainstream until the next kernel
>> release, if nobody complains about, or otherwise revert the changes, if
>> they proofed to cause troubles at dst and/or dvb-bt8xx.
>>
>> Also, you (or others) may write another approach keeping the fixes with
>> an strategy more adequate for dst.
>>
>> Do you agree with this way?
> 
> NACK
> 
>> -- 
>> Cheers,
>> Mauro
>>
>> (*) Currently, dst can be removed only with "rmmod -f", due to a wrong
>> usage count. With Trent's patches, this were fixed.
> 
> http://article.gmane.org/gmane.linux.drivers.dvb/27792/match=re+linux+dvb+unable+rmmod+dst+bt8xx

Just to make things clear, Manu, Are you telling us that the patch in the above
link addresses the use count bug?

If that is the case, are you suggesting that that patch be applied to the
repository instead of Trent's changesets?

Moreso, if that is the case, then the patch in the above link lacks a
sign-off...  We need to apply SOMETHING to fix this problem, and you know the
rules...

What should be done to fix the use count bug?

Regards,

Michael Krufky

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/