Chinaunix首页 | 论坛 | 博客

arm&linux

首页　| 　博文目录　| 　关于我

lijun_hangzhou

  • 博客访问： 104707
  • 博文数量： 76
  • 博客积分： 50
  • 博客等级： 民兵
  • 技术积分： 400
  • 用 户 组： 普通用户
  • 注册时间： 2007-07-18 21:41
文章分类

全部博文（76）

文章存档

2011年（76）

我的朋友
最近访客
推荐博文
gdb多核调试问题

分类：

2011-08-24 17:23:42

原文地址：gdb多核调试问题 作者：liujunwei1234

1 问题发现
在用gdb调试linux内核的网卡驱动时，在多次next之后，会出现下面结果：
执行某条语句后，gdb提示
[New Thread 4]
[New Thread 3]
contiuing ... ...
导致调试无法继续。
2 问题分析
首先想到的是产生了多线程的切换，因为调试环境是在多核机器上进行的，所以猜测其原因可能是当前核上的处于running的调试线程由于某种原因挂起了，因此调试过程切换到了其他核上，但是由于当前核上的调试资源没有被释放，因此，其他某个处于running状态的核得不到调试资源，使整个gdb挂起。今天有个师弟给我发了下面这篇文章，基本证实了这个想法。
3 kgdb多核patch
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

  1. Signed-off-by: Sonic Zhang <sonic.adi@gmail.com>
  2. ---
  3.  include/linux/kgdb.h
  4.  kernel/kgdb.c
  5. Andrew Morton - Feb. 20, 2009, 12:55 a.m.
  6. On Wed, 18 Feb 2009 17:37:54 +0800
  7. sonic zhang <sonic.adi@gmail.com> wrote:

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

  25. x86_64 allmodconfig:

  27. kernel/kgdb.c: In function 'kgdb_wait':
  28. kernel/kgdb.c:589: error: 'KGDB_THR_PROC_SWAP' undeclared (first use in this function)
  29. kernel/kgdb.c:589: error: (Each undeclared identifier is reported only once
  30. kernel/kgdb.c:589: error: for each function it appears in.)
  31. kernel/kgdb.c: In function 'gdb_cmd_query':
  32. kernel/kgdb.c:1087: error: 'KGDB_THR_PROC_SWAP' undeclared (first use in this function)
  33. kernel/kgdb.c: In function 'gdb_cmd_task':
  34. kernel/kgdb.c:1113: error: 'KGDB_THR_PROC_SWAP' undeclared (first use in this function)
  35. kernel/kgdb.c: In function 'gdb_cmd_thread':
  36. kernel/kgdb.c:1163: error: 'KGDB_THR_PROC_SWAP' undeclared (first use in this function)
  37. kernel/kgdb.c: In function 'gdb_serial_stub':
  38. kernel/kgdb.c:1275: error: 'KGDB_THR_PROC_SWAP' undeclared (first use in this function)
  39. kernel/kgdb.c: In function 'kgdb_handle_exception':
  40. kernel/kgdb.c:1575: error: 'KGDB_THR_PROC_SWAP' undeclared (first use in this function)
  41. kernel/kgdb.c: In function 'kgdb_nmicallback':
  42. kernel/kgdb.c:1606: warning: suggest parentheses around && within ||
  43. kernel/kgdb.c:1606: warning: suggest parentheses around && within ||
  44. kernel/kgdb.c:1606: warning: suggest parentheses around && within ||
  45. --
  46. To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
  47. the body of a message to majordomo@vger.kernel.org
  48. More majordomo info at
  49. Please read the FAQ at
  50. Patch
  51. diff --git a/include/linux/kgdb.h b/include/linux/kgdb.h
  52. index 6adcc29..664e396 100644
  53. --- a/include/linux/kgdb.h
  54. +++ b/include/linux/kgdb.h
  55. @@ -203,6 +204,19 @@ kgdb_arch_handle_exception(int vector, int signo, int err_code,
  56.   */
  57.  extern void kgdb_roundup_cpus(unsigned long flags);

  59. +/**
  60. + *    kgdb_roundup_cpu - Get spcific CPU into a holding pattern
  61. + *    @cpu: Specific cpu id
  62. + *    @flags: Current IRQ state
  63. + *
  64. + *    On SMP systems, we need to switch cpu from current active one to
  65. + *    the other passive one. This get current active CPU into a known state
  66. + *    in kgdb_wait().
  67. + *
  68. + *    On non-SMP systems, this is not called.
  69. + */
  70. +extern void kgdb_roundup_cpu(int cpu, unsigned long flags);
  71. +
  72.  /* Optional functions. */
  73.  extern int kgdb_validate_break_address(unsigned long addr);
  74.  extern int kgdb_arch_set_breakpoint(unsigned long addr, char *saved_instr);
  75. diff --git a/kernel/kgdb.c b/kernel/kgdb.c
  76. index e4dcfb2..277986d 100644
  77. --- a/kernel/kgdb.c
  78. +++ b/kernel/kgdb.c
  79. @@ -565,6 +565,7 @@
  80.  {
  81.      unsigned long flags;
  82.      int cpu;
  83. +    struct task_struct *thread;

  85.      local_irq_save(flags);
  86.      cpu = raw_smp_processor_id();
  87. @@ -577,10 +578,26 @@
  88.      smp_wmb();
  89.      atomic_set(&cpu_in_kgdb[cpu], 1);

  91. +    kgdb_disable_hw_debug(regs);
  92. +
  93.      /* Wait till primary CPU is done with debugging */
  94.      while (atomic_read(&passive_cpu_wait[cpu]))
  95.          cpu_relax();

  97. +    /* Trap into kgdb as the active CPU if gdb asks to switch. */
  98. +    thread = getthread(regs, -raw_smp_processor_id() - 2);
  99. +    if ((arch_kgdb_ops.flags & KGDB_THR_PROC_SWAP) &&
  100. +        kgdb_contthread && kgdb_contthread == current) {
  101. +        kgdb_breakpoint();
  102. +
  103. +        kgdb_info[cpu].debuggerinfo = NULL;
  104. +        kgdb_info[cpu].task = NULL;
  105. +
  106. +        clocksource_touch_watchdog();
  107. +        local_irq_restore(flags);
  108. +        return;
  109. +    }
  110. +
  111.      kgdb_info[cpu].debuggerinfo = NULL;
  112.      kgdb_info[cpu].task = NULL;

  114. @@ -1066,13 +1083,16 @@
  115.              sprintf(tmpstr, "shadowCPU%d",
  116.                      (int)(-ks->threadid - 2));
  117.              kgdb_mem2hex(tmpstr, remcom_out_buffer, strlen(tmpstr));
  118. +
  119. +            if (arch_kgdb_ops.flags & KGDB_THR_PROC_SWAP)
  120. +                ks->thr_query = 1;
  121.          }
  122.          break;
  123.      }
  124.  }

  126.  /* Handle the 'H' task query packets */
  127. -static void gdb_cmd_task(struct kgdb_state *ks)
  128. +static int gdb_cmd_task(struct kgdb_state *ks)
  129.  {
  130.      struct task_struct *thread;
  131.      char *ptr;
  132. @@ -1089,6 +1109,15 @@
  133.          kgdb_usethread = thread;
  134.          ks->kgdb_usethreadid = ks->threadid;
  135.          strcpy(remcom_out_buffer, "OK");
  136. +#ifdef CONFIG_SMP
  137. +        if ((arch_kgdb_ops.flags & KGDB_THR_PROC_SWAP) &&
  138. +            !ks->thr_query && ks->kgdb_usethreadid < -1 &&
  139. +            - ks->kgdb_usethreadid - 2 != raw_smp_processor_id()) {
  140. +            kgdb_roundup_cpu(raw_smp_processor_id(), 0);
  141. +            kgdb_contthread = kgdb_usethread;
  142. +            return 1;
  143. +        }
  144. +#endif
  145.          break;
  146.      case 'c':
  147.          ptr = &remcom_in_buffer[2];
  148. @@ -1102,10 +1131,27 @@
  149.                  break;
  150.              }
  151.              kgdb_contthread = thread;
  152. +#ifdef CONFIG_SMP
  153. +            if ((arch_kgdb_ops.flags & KGDB_THR_PROC_SWAP) &&
  154. +                thread != current) {
  155. +                int cpu;
  156. +                for_each_online_cpu(cpu) {
  157. +                    if (thread == kgdb_info[cpu].task) {
  158. +                        kgdb_roundup_cpu(
  159. +                            raw_smp_processor_id(),
  160. +                            0);
  161. +                        ks->kgdb_usethreadid = -cpu-2;
  162. +                        return 1;
  163. +                    }
  164. +                }
  165. +            }
  166. +#endif
  167.          }
  168.          strcpy(remcom_out_buffer, "OK");
  169.          break;
  170.      }
  171. +
  172. +    return 0;
  173.  }

  175.  /* Handle the 'T' thread query packets */
  176. @@ -1114,6 +1160,9 @@
  177.      char *ptr = &remcom_in_buffer[1];
  178.      struct task_struct *thread;

  180. +    if (arch_kgdb_ops.flags & KGDB_THR_PROC_SWAP)
  181. +        ks->thr_query = 0;
  182. +
  183.      kgdb_hex2long(&ptr, &ks->threadid);
  184.      thread = getthread(ks->linux_regs, ks->threadid);
  185.      if (thread)
  186. @@ -1223,7 +1272,12 @@
  187.      /* Clear the out buffer. */
  188.      memset(remcom_out_buffer, 0, sizeof(remcom_out_buffer));

  190. -    if (kgdb_connected) {
  191. +    if ((arch_kgdb_ops.flags & KGDB_THR_PROC_SWAP) && kgdb_contthread) {
  192. +        remcom_out_buffer[0] = 'O';
  193. +        remcom_out_buffer[1] = 'K';
  194. +        remcom_out_buffer[2] = 0;
  195. +        put_packet(remcom_out_buffer);
  196. +    } else if (kgdb_connected) {
  197.          unsigned char thref[8];
  198.          char *ptr;

  200. @@ -1238,6 +1292,7 @@
  201.          put_packet(remcom_out_buffer);
  202.      }

  204. +    kgdb_contthread = current;
  205.      kgdb_usethread = kgdb_info[ks->cpu].task;
  206.      ks->kgdb_usethreadid = shadow_pid(kgdb_info[ks->cpu].task->pid);
  207.      ks->pass_exception = 0;
  208. @@ -1284,7 +1339,8 @@
  209.              gdb_cmd_query(ks);
  210.              break;
  211.          case 'H': /* task related */
  212. -            gdb_cmd_task(ks);
  213. +            if (gdb_cmd_task(ks))
  214. +                goto kgdb_exit;
  215.              break;
  216.          case 'T': /* Query thread status */
  217.              gdb_cmd_thread(ks);
  218. @@ -1324,6 +1380,7 @@
  219.              if (error >= 0 || remcom_in_buffer[0] == 'D' ||
  220.               remcom_in_buffer[0] == 'k') {
  221.                  error = 0;
  222. +                kgdb_contthread = NULL;
  223.                  goto kgdb_exit;
  224.              }

  226. @@ -1464,11 +1521,15 @@
  227.       * Get the passive CPU lock which will hold all the non-primary
  228.       * CPU in a spin state while the debugger is active
  229.       */
  230. -    if (!kgdb_single_step) {
  231. +    if (!kgdb_single_step && !kgdb_contthread) {
  232.          for (i = 0; i < NR_CPUS; i++)
  233.              atomic_set(&passive_cpu_wait[i], 1);
  234.      }

  236. +    if (kgdb_contthread)
  237. +        atomic_set(&passive_cpu_wait[raw_smp_processor_id()], 1);
  238. +
  239. +
  240.      /*
  241.       * spin_lock code is good enough as a barrier so we don't
  242.       * need one here:
  243. @@ -1477,7 +1538,7 @@

  245.  #ifdef CONFIG_SMP
  246.      /* Signal the other CPUs to enter kgdb_wait() */
  247. -    if ((!kgdb_single_step) && kgdb_do_roundup)
  248. +    if ((!kgdb_single_step && !kgdb_contthread) && kgdb_do_roundup)
  249.          kgdb_roundup_cpus(flags);
  250.  #endif

  252. @@ -1496,7 +1557,6 @@
  253.      kgdb_post_primary_code(ks->linux_regs, ks->ex_vector, ks->err_code);
  254.      kgdb_deactivate_sw_breakpoints();
  255.      kgdb_single_step = 0;
  256. -    kgdb_contthread = current;
  257.      exception_level = 0;

  259.      /* Talk to debugger with gdbserial protocol */
  260. @@ -1510,7 +1570,14 @@
  261.      kgdb_info[ks->cpu].task = NULL;
  262.      atomic_set(&cpu_in_kgdb[ks->cpu], 0);

  264. -    if (!kgdb_single_step) {
  265. +#ifdef CONFIG_SMP
  266. +    i = -(ks->kgdb_usethreadid + 2);
  267. +    if ((arch_kgdb_ops.flags & KGDB_THR_PROC_SWAP) &&
  268. +        kgdb_contthread && i != cpu) {
  269. +        atomic_set(&passive_cpu_wait[i], 0);
  270. +    }
  271. +#endif
  272. +    if (!kgdb_single_step && !kgdb_contthread) {
  273.          for (i = NR_CPUS-1; i >= 0; i--)
  274.              atomic_set(&passive_cpu_wait[i], 0);
  275.          
阅读(1140) | 评论(0) | 转发(0) |
0

上一篇：completion使用方法

下一篇：get_user_pages注意问题

给主人留下些什么吧！~~
关于我们 | 关于IT168 | 联系方式 | 广告合作 | 法律声明 | 免费注册

Copyright 2001-2010 ChinaUnix.net All Rights Reserved 北京皓辰网域网络信息技术有限公司. 版权所有

感谢所有关心和支持过ChinaUnix的朋友们

16024965号-6