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

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

[Cliff Wickman]

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 <mort@sgi.com>
---
 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)

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

[Jason Wessel]

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 <mort@sgi.com>
> ---
>  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)
>