linux-kernel - Re: [PATCH] kdb: use macro instead of const number

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

Date:	Wed, 10 Nov 2010 07:21:32 -0600
From:	Jason Wessel <jason.wessel@...driver.com>
To:	jovi zhang <bookjovi@...il.com>
CC:	Martin Hicks <mort@....com>, Dmitry Torokhov <dtor@...l.ru>,
	Rusty Russell <rusty@...tcorp.com.au>,
	kgdb-bugreport@...ts.sourceforge.net, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] kdb: use macro instead of const number

The 3 patches you have sent me so far all have white space that is
completely mangled.  In the future, please use a mail client that does
not mangle white space, or use "git send-email".

I did fix up your patches so as to add them to the merge queue.  The
code cleanup (this patch) will be deferred to the next merge window.

Please see my other comment about signed off by tags.


On 11/10/2010 04:51 AM, jovi zhang wrote:
> It's better to use macro KDB_BASE_CMD_MAX instead of 50
>
> Signed-off-by: jovi zhang <bookjovi@...il.com>
>   

For a signed off tag, you want to make sure these are consistent for all
the patches you send to any kernel maintainer or kernel mailing list. 
You already have one patch in the kernel
(85e00b55517a5696282b80836655e6b6f5702df0), and it does not match your
signed-off-by line.

If you are using git to make your patches, please edit the ~/.gitconfig
and use something like:
[user]
        name = Jovi Zhang
        email = bookjovi@...il.com

Thanks,
Jason.

> kernel/debug/kdb/kdb_main.c |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/kernel/debug/kdb/kdb_main.c b/kernel/debug/kdb/kdb_main.c
> index 37755d6..3579105 100644
> --- a/kernel/debug/kdb/kdb_main.c
> +++ b/kernel/debug/kdb/kdb_main.c
> @@ -78,7 +78,7 @@ static unsigned int kdb_continue_catastrophic;
> static kdbtab_t *kdb_commands;
> #define KDB_BASE_CMD_MAX 50
> static int kdb_max_commands = KDB_BASE_CMD_MAX;
> -static kdbtab_t kdb_base_commands[50];
> +static kdbtab_t kdb_base_commands[KDB_BASE_CMD_MAX];
> #define for_each_kdbcmd(cmd, num)                                      \
>        for ((cmd) = kdb_base_commands, (num) = 0;                      \
>             num < kdb_max_commands;                                    \
>   

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