linux-kernel - Re: [v4l-dvb-maintainer] -mm: cx88-blackbird.c: unused code re-added

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: <455DBED5.5030204@linuxtv.org>
Date:	Fri, 17 Nov 2006 08:53:25 -0500
From:	Michael Krufky <mkrufky@...uxtv.org>
To:	Adrian Bunk <bunk@...sta.de>
CC:	Andrew Morton <akpm@...l.org>, v4l-dvb-maintainer@...uxtv.org,
	linux-kernel@...r.kernel.org,
	Mauro Carvalho Chehab <mchehab@...urbo.com.br>
Subject: Re: [v4l-dvb-maintainer] -mm: cx88-blackbird.c: unused code re-added

Adrian Bunk wrote:

>On Tue, Nov 14, 2006 at 01:41:25AM -0800, Andrew Morton wrote:
>  
>
>>...
>>Changes since 2.6.19-rc5-mm2:
>>...
>> git-dvb.patch
>>...
>> git trees
>>...
>>    
>>
>
>Why does this patch re-add the still unused cx88_ioctl_hook and 
>cx88_ioctl_translator in drivers/media/video/cx88/cx88-blackbird.c 
>I removed a few months ago?
>

Adrian,

I have added those hooks back into the driver, because they are needed 
for the cx88-ivtv ioctl emulation layer.  However, I have instructed 
Mauro not to merge that patch upstream -- it is meant to stay at our 
mercurial repository only, since the vanilla kernel does not need to use 
the cx88-ivtv ioctl emulation layer.

It looks like somehow these got into -mm ... I think that is because 
Andrew's scripts pull from Mauro's devel branch in his git tree.

The problem commit is:  de513acdbafc48c52017255fee02af727dcfcc32

It appears that Mauro has folded my patch into Steve's patch, which he 
should not have done.  :-(  What a mess!

I will send a new patch to Mauro later on today to be applied to his git 
devel branch, which will remove those hooks again.

As soon as ivtv gets merged into our tree, that's when we will be able 
to remove cx88-ivtv alltogether, as there will no longer be any need to 
support the ivtv API.

regards,

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