Hello.

Bob Picco wrote:

> Some suggested kernel coding style cleanups.

> Signed-off-by: Bob Picco <[EMAIL PROTECTED]>

>  kernel/kgdb.c |   57 
> ++++++++++++++++++++++++++++++---------------------------
>  1 file changed, 30 insertions(+), 27 deletions(-)

> Index: git.jwessel/kernel/kgdb.c
> ===================================================================
> --- git.jwessel.orig/kernel/kgdb.c    2007-05-18 14:53:58.000000000 -0400
> +++ git.jwessel/kernel/kgdb.c 2007-05-21 11:50:58.000000000 -0400
[...]
> @@ -1417,7 +1418,8 @@ static struct notifier_block kgdb_module
>  void kgdb_nmihook(int cpu, void *regs)
>  {
>  #ifdef CONFIG_SMP
> -     if (!atomic_read(&procindebug[cpu]) && atomic_read(&debugger_active) != 
> (cpu + 1))
> +     if (!atomic_read(&procindebug[cpu]) &&
> +             atomic_read(&debugger_active) != (cpu + 1))

    Usually, condition carried from the previous line starts on the net column 
after the paren in that line.

>               kgdb_wait((struct pt_regs *)regs);
>  #endif
>  }
> @@ -1761,7 +1763,8 @@ static int kgdb_notify_reboot(struct not
>        * and print. */
>       if (!kgdb_connected || atomic_read(&debugger_active) != 0)
>               return 0;
> -     if ((code == SYS_RESTART) || (code == SYS_HALT) || (code == 
> SYS_POWER_OFF)){
> +     if ((code == SYS_RESTART) || (code == SYS_HALT) ||
> +             (code == SYS_POWER_OFF)){

    Same here.

WBR, Sergei

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Kgdb-bugreport mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/kgdb-bugreport

Reply via email to