All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andrew Morton <akpm@linux-foundation.org>
To: sonic zhang <sonic.adi@gmail.com>
Cc: jason.wessel@windriver.com, linux-kernel@vger.kernel.org,
	kgdb-bugreport@lists.sourceforge.net
Subject: Re: [PATCH] [kgdb] Switch master cpu after gdb thread command for SMP (v4)
Date: Thu, 19 Feb 2009 16:55:27 -0800	[thread overview]
Message-ID: <20090219165527.2ad1a889.akpm@linux-foundation.org> (raw)
In-Reply-To: <1234949874.17467.6.camel@eight.analog.com>

On Wed, 18 Feb 2009 17:37:54 +0800
sonic zhang <sonic.adi@gmail.com> wrote:

> This is the updated kgdb SMP patch, which is tested on blackfin bf561 dual core
> system. A new bug is fixed, which causes debugging halt in SMP kernel if a
> breakpoint is hit and cpu is switched in gdb before continue running kernel.
> 
>    In blackfin SMP architecture, different core has its own L1 SRAM and MMR
>    memory, which code running on the other core can't access. In current kgdb
>    impelemntation, cpus are represented by thread with minus prefix.
> 
> If user run thread command in gdb to switch to the thread of the other cpu,
> kgdb should:
> 1. send IPI signal to master cpu
> 2. release the specific passive cpu waiting in IPI handler
> 3. exit kgdb exception loop on master cpu and trap into kgdb wait in IPI handler
> 4. trap the released passive cpu into kgdb exception in IPI handler
> 

x86_64 allmodconfig:

kernel/kgdb.c: In function 'kgdb_wait':
kernel/kgdb.c:589: error: 'KGDB_THR_PROC_SWAP' undeclared (first use in this function)
kernel/kgdb.c:589: error: (Each undeclared identifier is reported only once
kernel/kgdb.c:589: error: for each function it appears in.)
kernel/kgdb.c: In function 'gdb_cmd_query':
kernel/kgdb.c:1087: error: 'KGDB_THR_PROC_SWAP' undeclared (first use in this function)
kernel/kgdb.c: In function 'gdb_cmd_task':
kernel/kgdb.c:1113: error: 'KGDB_THR_PROC_SWAP' undeclared (first use in this function)
kernel/kgdb.c: In function 'gdb_cmd_thread':
kernel/kgdb.c:1163: error: 'KGDB_THR_PROC_SWAP' undeclared (first use in this function)
kernel/kgdb.c: In function 'gdb_serial_stub':
kernel/kgdb.c:1275: error: 'KGDB_THR_PROC_SWAP' undeclared (first use in this function)
kernel/kgdb.c: In function 'kgdb_handle_exception':
kernel/kgdb.c:1575: error: 'KGDB_THR_PROC_SWAP' undeclared (first use in this function)
kernel/kgdb.c: In function 'kgdb_nmicallback':
kernel/kgdb.c:1606: warning: suggest parentheses around && within ||
kernel/kgdb.c:1606: warning: suggest parentheses around && within ||
kernel/kgdb.c:1606: warning: suggest parentheses around && within ||

  reply	other threads:[~2009-02-20  0:56 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-02-18  9:37 [PATCH] [kgdb] Switch master cpu after gdb thread command for SMP (v4) sonic zhang
2009-02-20  0:55 ` Andrew Morton [this message]
2009-03-04  9:17 ` sonic zhang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20090219165527.2ad1a889.akpm@linux-foundation.org \
    --to=akpm@linux-foundation.org \
    --cc=jason.wessel@windriver.com \
    --cc=kgdb-bugreport@lists.sourceforge.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=sonic.adi@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.