linux-kernel - Re: [PATCH/RFC] parport_pc: remove ancient, overeager quirk that disables EPP support on many chipsets

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: <20110915173950.GA19450@elie>
Date:	Thu, 15 Sep 2011 12:39:50 -0500
From:	Jonathan Nieder <jrnieder@...il.com>
To:	Greg KH <greg@...ah.com>
Cc:	Leopold Palomo-Avellaneda <leo@...xarxa.net>,
	linux-kernel@...r.kernel.org, Adam Baker <linux@...er-net.org.uk>,
	linux-parport@...ts.infradead.org, 630593@...s.debian.org,
	Nicos Gollan <gollan@...ormatik.uni-kl.de>,
	Alan Cox <alan@...rguk.ukuu.org.uk>,
	Alexander Gordeev <lasaine@....cs.msu.su>
Subject: Re: [PATCH/RFC] parport_pc: remove ancient, overeager quirk that
 disables EPP support on many chipsets

Hi,

Greg KH wrote:

> You can not add someone else's signed-off-by: line to a patch, please go
> re-read Documentation/SubmittingPatches as to why.
>
> And did Adam originally write this patch?  Or did you?  If Adam, please
> set the authorship information properly.

>From a quick Google search:

http://lists.infradead.org/pipermail/linux-parport/2008-March/000628.html

It looks like this one does have Adam Baker's sign-off (and it is sad
how long this patch seems to have sat without being submitted to
mainline).

I don't know who originally had the idea of removing that code.  See
[1], [2], [3], and [4] for some early discussions.

The current "intel parport bug" test this patch removes seems to have
been introduced between 2.3.10pre5 and 2.3.10 (thanks to Dave Jones
for the git tree that makes such searches easy!).  That means some
time around June or July, 1999.  At the time, the parport maintainers
according to MAINTAINERS were Phil Blundell, Tim Waugh, David
Campbell, and Andrea Arcangeli.  From the patch "[PATCH] parport is an
orphan", 2007-03-05, I infer that not all of them are still interested
in the driver and whoever _is_ interested is probably subscribed to
the (low-volume) linux-parport list.

I'd say, why not get this patch in linux-next or -mm somehow and see
if anyone screams?  It would be _very_ useful to find an actual
instance of the "intel parport bug" so we could see what that code was
supposed to do and do it better.

Thanks,
Jonathan

[1] http://thread.gmane.org/gmane.linux.parport/322/focus=326
[2] http://thread.gmane.org/gmane.linux.parport/324/focus=327
[3] http://thread.gmane.org/gmane.linux.parport/806
[4] http://thread.gmane.org/gmane.linux.parport/925/focus=929
--
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/