linux-kernel - [PATCH][0/5][resend] floppy.c: first round of cleanup patches

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>] [day] [month] [year] [list]

Date:	Mon, 19 Mar 2007 16:09:45 +0100
From:	Jesper Juhl <jesper.juhl@...il.com>
To:	Andrew Morton <akpm@...ux-foundation.org>
Cc:	LKML <linux-kernel@...r.kernel.org>, Andi Kleen <ak@...e.de>,
	Trent Waddington <trent.waddington@...il.com>,
	Bartlomiej Zolnierkiewicz <bzolnier@...il.com>,
	Alan Cox <alan@...rguk.ukuu.org.uk>,
	Jesper Juhl <jesper.juhl@...il.com>
Subject: [PATCH][0/5][resend] floppy.c: first round of cleanup patches - the simple stuff

Greetings people,

A while back I promised to do some floppy.c cleanups and created an initial 
set of patches. Those patches got some good feedback but never got merged.
I want to continue with more substantial cleanups, but it would be nice to get 
the initial (rather trivial) bits in first so I can submit further patches on 
top.

I've been using these patches myself without any problems and they have also 
been Ack'ed by a few people;
	Alan Cox said:
		"Acked-by: Alan Cox <alan@...hat.com>
		 for the set of 5 patches. Its a beginning and removing the dead wood so
		 you can see what is actually there is a good a beginning as any other."
	Bartlomiej Zolnierkiewicz said:
		"Seconded, looks like a good beginning.
		 You may also add my ACK for the whole set."

Andi Kleen commented that the real problem with floppy.c is the basic code 
logic and that most of the code I'm fixing up here probably has to be 
replaced later anyway - there's no doubt that's correct, but I need to start 
somewhere and starting with these small patches makes the file a lot easier 
for me to work with and I think they are a nice little initial cleanup until 
I get a chance to do something more substantial (which probably won't be for 
another month and a half since I'm currently without internet connectivity at 
home).

There are 5 patches in this series. They are based on current Linus -git and 
are incremental.

These are the patches (they'll be posted to the same recipients as this 
mail) :

       [PATCH][1/5][resend] floppy.c: Initial (partial) CodingStyle cleanup
       [PATCH][2/5][resend] floppy.c: Remove pointless register keywords
       [PATCH][3/5][resend] floppy.c: Remove dead/commented out code
       [PATCH][4/5][resend] floppy.c: Add explicit/better printk() levels
       [PATCH][5/5][resend] floppy.c: Fix device_create_file() warning

The Cc list is composed of people who expressed interrest in this cleanup +
people who look like original authors of the code (for whom I could locate
email addresses) etc.

Andrew: Could you please merge these into -mm? And if you could push them to 
Linus after a bit of time in -mm that would be greatly appreciated as well.

Kind regards,

 Jesper Juhl <jesper.juhl@...il.com>

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