linux-kernel - Re: [PATCH 07/37] kgdb,sparc: Add in kgdb_arch_set

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <4B32A1EB.2020805@windriver.com>
Date:	Wed, 23 Dec 2009 17:04:11 -0600
From:	Jason Wessel <jason.wessel@...driver.com>
To:	David Miller <davem@...emloft.net>
CC:	linux-kernel@...r.kernel.org, kgdb-bugreport@...ts.sourceforge.net,
	kdb@....sgi.com, mingo@...e.hu
Subject: Re: [PATCH 07/37] kgdb,sparc: Add in kgdb_arch_set_pc for sparc


David Miller wrote:
> From: Jason Wessel <jason.wessel@...driver.com>
> Date: Wed, 23 Dec 2009 15:19:20 -0600
>
>   
>> +void kgdb_arch_set_pc(struct pt_regs *regs, unsigned long ip)
>> +{
>> +       regs->pc = regs->npc;
>> +       regs->npc = ip;
>> +}
>> +
>>     
>
> Is this right?
>
> It should probably be something like:
>
>   
Just because it didn't crash doesn't mean it is right :-)

I'll go with your implementation which according to the instruction
manual is the right way.  This particular function doesn't really get
exercised with out the arch specific hardware or software single
stepping code, which is not a part of this series at the moment.

This fix and the other fix you asked about are both in kgdb-next which
feeds into linux-next, if you are ultimately ok with the changes, I'll
add an ack from you as well.

Hopefully there is enough time to get the kdb shape for the next kernel
merge window.

http://git.kernel.org/?p=linux/kernel/git/jwessel/linux-2.6-kgdb.git;a=commitdiff;h=ed088ec6e6e7e245c060d068d333648eff06dd4b
and
http://git.kernel.org/?p=linux/kernel/git/jwessel/linux-2.6-kgdb.git;a=commitdiff;h=c5dc61eb8d1c347ff3eadc1bccae07437d85779a

Thanks,
Jason.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/