linux-kernel - Re: [PATCH 109/148] include/asm-x86/serial.h: checkpatch 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: Windows password security audit tool. GUI, reports in PDF.

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

Date:	Wed, 26 Mar 2008 11:25:14 +0100
From:	Ingo Molnar <mingo@...e.hu>
To:	David Miller <davem@...emloft.net>
Cc:	jirislaby@...il.com, viro@...IV.linux.org.uk, joe@...ches.com,
	tglx@...utronix.de, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 109/148] include/asm-x86/serial.h: checkpatch cleanups
	- formatting only


* David Miller <davem@...emloft.net> wrote:

> From: Ingo Molnar <mingo@...e.hu>
> Date: Tue, 25 Mar 2008 14:17:08 +0100
> 
> > There can be fluctuations and artifacts, and obviously this is just 
> > another (arbitrary) static metric that has no forced relationship with 
> > real code quality - but in my experience it's surprisingly close to 
> > reality - closer than any other code metric i've seen.
> 
> And yet you used it to claim that the sparc64 port is an 
> unmaintainable pile of poo.

i did not claim that at all, the checkpatch average of sparc64 is pretty 
good:

                           errors   lines of code   errors/KLOC
   arch/sparc64/             1457           49785          29.2

Sparc64 is one of the cleanest architectures, in terms of average 
checkpatch.pl code-quality.

the list i generated is the '20 worst files' (out of 90 sparc64 files) 
those are the files in need of cleanups (according to that metric) - and 
a quick manual glance confirmed that impression.

Or is it your position that every single file in sparc64 is squeaky 
clean and that each of the 1457 cleanliness problems that checkpatch.pl 
reported there is bogus?

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