linux-kernel - Re: [2.6 patch] drivers/net/wireless/zd1211rw/: possible cleanups

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 for Android: free password hash cracker in your pocket

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

Date:	Mon, 17 Jul 2006 23:29:51 +0200
From:	Ulrich Kunitz <kune@...ne-taler.de>
To:	Daniel Drake <dsd@...too.org>
Cc:	Adrian Bunk <bunk@...sta.de>, Andrew Morton <akpm@...l.org>,
	linville@...driver.com, netdev@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [2.6 patch] drivers/net/wireless/zd1211rw/: possible cleanups

On 06-07-16 14:17 Daniel Drake wrote:

> Adrian Bunk wrote:
> >This patch contains the following possible cleanups:
> >- make needlessly global functions static
> >- #if 0 unused functions
> >
> >Please review which of these functions do make sense and which do 
> >conflict with pending patches.
> 
> Thanks Adrian. I have put this in my tree and made an additional change 
> along the same lines (your patched introduced an unused function warning 
> to the non-debug build). If Ulrich signifies acceptance, I will send 
> this on to John.
> 
> I have also sent in a patch to add a MAINTAINERS entry for zd1211rw, in 
> hope that this will help you send patches with myself and/or Ulrich CC'd 
> in future :)
> 
> Thanks.
> Daniel
> -
> To unsubscribe from this list: send the line "unsubscribe netdev" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

Adrian, I would like to see this patch split up into three at
least. 

Patch 1: Remove unused IO emulation functions
Patch 2: Remove other unused stuff, which could be split up
         further for each C file and header
Patch 3: Change DEBUG ifdefs to #if 0

The purpose of patch 3 is bogus, because the follow-up patch will
be called "removed useless #if 0 stuff". Keep in mind there is
some reason, why I have such code there. If they ifdefs are not
acceptable I will make this code dependent on a module parameter
and compile it into the production module. We have a lot of
different devices from different vendors out there and people
report "stuff isn't working" but almost nothing more.

The problem with patch 1 and 2 is, that almost all of the function
are completing the interface and some of them are even only static
inlines. They are there because they should be used, before
somebody reinvents the wheel or makes something completely stupid.
However if such reasoning is not acceptable I'm ready to
compromise.

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