linux-kernel - Re: [RESEND][PATCH] early platform drivers V2

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, 14 Jan 2009 14:47:05 -0800
From:	Andrew Morton <akpm@...ux-foundation.org>
To:	Magnus Damm <magnus.damm@...il.com>
Cc:	linux-kernel@...r.kernel.org, lethal@...ux-sh.org,
	johnstul@...ibm.com, gregkh@...e.de, mingo@...hat.com,
	magnus.damm@...il.com, tglx@...utronix.de
Subject: Re: [RESEND][PATCH] early platform drivers V2

On Wed, 14 Jan 2009 19:54:36 +0900
Magnus Damm <magnus.damm@...il.com> wrote:

> This patch is V2 of the early platform driver implementation.

Is this checkpatch whine:

WARNING: consider using strict_strtoul in preference to simple_strtoul
#112: FILE: drivers/base/platform.c:1017:
+                       id = simple_strtoul(buf +

total: 0 errors, 1 warnings, 201 lines checked

appropriate?

If the user provided "43foo", do we want to treat that as 42, or as
error?

If the user provided just "foo" then afacit we'll currently treat that
as zero.


<this is all mad guesswork, as I don't have any idea what the user
interfacce to this feature looks like>

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