Hi, Craig.

2010/7/14 Craig Small <csm...@debian.org>:
> tags 588986 pending
> thankyou
> On Wed, Jul 14, 2010 at 12:36:36PM +0900, Nobuhiro Iwamatsu wrote:
>> psmisc FTBFS on SH4.
>> When peekfd is checked, configure checks structure (struct pt_regs) of 
>> asm/ptrace.h.
>> However, because sh4 has variable (regs) of the same name as mips, it is 
>> treated as mips.
> It's even worse than that, the code only works on little endian mipsel
> not mips, so not only do I need to add your check, I need to make sure
> it is a mipsel architecture.

Yes, you are right.

>
>> I made the patch that added unique variable cp0_status of mips to the check.
>> When this patch is applied, this problem can be avoided.
>> Could you apply this patch?
> I've got your patch and the additional architecture check in the
> upstream git repository:
> http://psmisc.git.sourceforge.net/git/gitweb.cgi?p=psmisc/psmisc;a=commitdiff;h=a5e6a1b66d1a4c809cd5bf932184755f6172c01e
>

Your patch can not revise problem of mips.
Because when WANT_PEEKFD_MIPS is checked, $build_cpu  is empty.
When the check code is moved below, it becomes possible to get value
of $build_cpu.
I create new patch and attached.

Best regards,
  Nobuhiro
-- 
Nobuhiro Iwamatsu
   iwamatsu at {nigauri.org / debian.org}
   GPG ID: 40AD1FA6

Attachment: 0001-Move-struct-of-ptrace.h-stuff.patch
Description: application/octetstream

Reply via email to