[PATCH] arm: add RESET_PID in the clone impl

[Carmelo Amoroso]

Il 04 settembre 2014 22:32:01 Bernhard Reutner-Fischer 
<rep.dot.nop at gmail.com> ha scritto:

> >On 4 September 2014 19:32, Carmelo Amoroso <carmelo73 at gmail.com> wrote:
> >
> >
> > Il 04 settembre 2014 00:01:38 Waldemar Brodkorb <wbx at openadk.org> ha
> > scritto:
> >
> >> Hi Carmelo,
> >
> >
> > Hello,
> >
> >
> >> Carmelo Amoroso wrote,
> >>
> >> > Hi,
> >> > I don't think clone on arm is missing RESET_PID if you use the
> >> > correct implementation from
> >> > libpthread/nptl/sysdeps/unix/sysv/linux/arm/clone.S.
> >>
> >> Can you explain this a little bit more?
> >>
> >
> > What I mean to say is that if the root cause is the lack of RESET_PIN
> > definition it's not the case.
>
> No, it's not the define of RESET_PID but nothing in the arm clone.S actually
> does anything with RESET_PID, i.e. the actual implementation is missing,
> AFAICS.

Indeed by looking at the patch better I've noticed the actual fix.

> >
> > Then, I did not enter into the details of the patch, but at first glance I
> > could see several other unrelated changes that could eventually have been
> > split in more patches, but I could be wrong.
> > On my platform armv7 I don't remember to have such failures.
>
> Curious how you get a correct getpid() on arm then?
> This whole caching stuff is a pain and should be removed instead.

I will test and check it again as soon as back from my bz trip... I could 
not remember correctly. For sure I did not debug it.

Carmelo

Inviato con AquaMail per Android
http://www.aqua-mail.com