2010-07-13 18:51:43

by Cliff Wickman

[permalink] [raw]
Subject: Re: [PATCH] kdb: ll command won't terminate




Hi Jason,

A question, please, about a patch similar to the below that Martin Hicks
sent, around 5/26/2010.

Martin is on vacation, and suse is wondering whether the below change
is queued for 2.6.35. I can't find anything about it. Could you tell me
its status?

Thanks.

-Cliff


Signed-off-by: Martin Hicks <[email protected]>
---
kernel/debug/kdb/kdb_main.c | 3 +++
1 file changed, 3 insertions(+)

Index: linux-2.6/kernel/debug/kdb/kdb_main.c
===================================================================
--- linux-2.6.orig/kernel/debug/kdb/kdb_main.c
+++ linux-2.6/kernel/debug/kdb/kdb_main.c
@@ -2291,6 +2291,9 @@ static int kdb_ll(int argc, const char *
while (va) {
char buf[80];

+ if (KDB_FLAG(CMD_INTERRUPT))
+ return 0;
+
sprintf(buf, "%s " kdb_machreg_fmt "\n", command, va);
diag = kdb_parse(buf);
if (diag)


2010-07-13 19:04:55

by Jason Wessel

[permalink] [raw]
Subject: Re: [PATCH] kdb: ll command won't terminate


This particular patch is already in the queue and will be in a pull
request very soon.

There are several 1-3 line regression fixes that have been found since
the kdb merge in 2.6.35.

Jason.

On 07/13/2010 01:52 PM, Cliff Wickman wrote:
>
> Hi Jason,
>
> A question, please, about a patch similar to the below that Martin Hicks
> sent, around 5/26/2010.
>
> Martin is on vacation, and suse is wondering whether the below change
> is queued for 2.6.35. I can't find anything about it. Could you tell me
> its status?
>
> Thanks.
>
> -Cliff
>
>
> Signed-off-by: Martin Hicks <[email protected]>
> ---
> kernel/debug/kdb/kdb_main.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> Index: linux-2.6/kernel/debug/kdb/kdb_main.c
> ===================================================================
> --- linux-2.6.orig/kernel/debug/kdb/kdb_main.c
> +++ linux-2.6/kernel/debug/kdb/kdb_main.c
> @@ -2291,6 +2291,9 @@ static int kdb_ll(int argc, const char *
> while (va) {
> char buf[80];
>
> + if (KDB_FLAG(CMD_INTERRUPT))
> + return 0;
> +
> sprintf(buf, "%s " kdb_machreg_fmt "\n", command, va);
> diag = kdb_parse(buf);
> if (diag)
>