"Internal resource leak" error with ARM NEON vmull.s32 insn

[Wolfgang Schildbach]

This bug occurs in qemu, commit 78a59470e6bbc6e16dc4033767492649c1ae4cfd (most recent as of 01/14/2011).

Compile, assemble, and link the code below, with the ARM tools. (I use ARM C/C++ Compiler, 4.1 [Build 462]).

armasm --cpu Cortex-A8 --licensing=flex foo.s
armcc --cpu Cortex-A8 --licensing=flex -o main -L--sysv main.c foo.o

Execute on qemu-arm and observe an "Internal resource leak" message.

> qemu-arm main
Internal resource leak before 0000818c

- Wolfgang

main.c:
int main(void)
{
  void foofunc(void);
  foofunc();

  return 0 ;
}

foo.s:
    ARM
    REQUIRE8
    PRESERVE8
    AREA code, CODE, READONLY, ALIGN=2

foofunc PROC
    VMULL.S32 q1, d2, d4
    MOV pc, lr

    ENDP

    EXPORT foofunc [CODE]
    END

[Wolfgang Schildbach (wschi)]

This bug is fixed on HEAD in the qemu-meego tree (commit 8493a687d54e542ac4eec8b2f8326415edf37ec4
A)
- Wolfgang

[Peter Maydell (pmaydell)]

On 19 January 2011 12:42, Wolfgang Schildbach <email address hidden> wrote:
> This bug is fixed on HEAD in the qemu-meego tree (commit 8493a687d54e542ac4eec8b2f8326415edf37ec4
> A)

Note that the qemu-meego tree disables these warnings using
an #ifdef, so even if the message is not printed we might still
not be handling the temporaries correctly (although there are a
lot of fixes for mishandled temps in that tree).

-- PMM

[Wolfgang Schildbach (wschi)]

> From: <email address hidden>
> [mailto:<email address hidden>] On
> Behalf Of Peter Maydell
> Sent: Wednesday, January 19, 2011 5:16 AM
> To: Bug 702885
> Cc: <email address hidden>
> Subject: Re: [Qemu-devel] [Bug 702885] Re: "Internal resource
> leak" error withARM NEON vmull.s32 insn
>
> On 19 January 2011 12:42, Wolfgang Schildbach
> <email address hidden> wrote:
> > This bug is fixed on HEAD in the qemu-meego tree (commit
> > 8493a687d54e542ac4eec8b2f8326415edf37ec4
> > A)
>
> Note that the qemu-meego tree disables these warnings using
> an #ifdef, so even if the message is not printed we might
> still not be handling the temporaries correctly (although
> there are a lot of fixes for mishandled temps in that tree).
>
> -- PMM

Just when I was getting my hopes up :-)

Indeed, I just checked with the meego tree that qemu still segfaults one
of my testcases. I'll try to isolate the case such that I can document
it on launchpad.

- Wolfgang

[Peter Maydell (pmaydell)]

That binary executes OK for me with no resource leak messages with:
qemu master as of commit b646968336
http://patchwork.ozlabs.org/patch/79728/
http://patchwork.ozlabs.org/patch/79581/

(i386 host.)

[Wolfgang Schildbach (wschi)]

Duh. I had missed the greater part of Christophe's patch (I am still
having trouble with my mail client; applying patches off the list is
manual for me).

With both patches applied, indeed the bug filed on launchpad seems
fixed. On my second test case, behaviour is much improved. Thanks much!

- Wolfgang

> -----Original Message-----
> From: <email address hidden> [mailto:<email address hidden>] On
> Behalf Of Peter Maydell
> Sent: Friday, January 21, 2011 8:01 AM
> To: Schildbach, Wolfgang
> Subject: [Bug 702885] Re: "Internal resource leak" error with
> ARM NEONvmull.s32 insn
>
> That binary executes OK for me with no resource leak messages with:
> qemu master as of commit b646968336
> http://patchwork.ozlabs.org/patch/79728/
> http://patchwork.ozlabs.org/patch/79581/
>
> (i386 host.)
>
> --
> You received this bug notification because you are a direct
> subscriber of the bug.
> https://bugs.launchpad.net/bugs/702885
>
> Title:
> "Internal resource leak" error with ARM NEON vmull.s32 insn
>
> Status in QEMU:
> New
>
> Bug description:
> This bug occurs in qemu, commit
> 78a59470e6bbc6e16dc4033767492649c1ae4cfd (most recent as of
> 01/14/2011).
>
> Compile, assemble, and link the code below, with the ARM
> tools. (I use
> ARM C/C++ Compiler, 4.1 [Build 462]).
>
> armasm --cpu Cortex-A8 --licensing=flex foo.s
> armcc --cpu Cortex-A8 --licensing=flex -o main -L--sysv main.c foo.o
>
> Execute on qemu-arm and observe an "Internal resource leak" message.
>
> > qemu-arm main
> Internal resource leak before 0000818c
>
> - Wolfgang
>
> main.c:
> int main(void)
> {
> void foofunc(void);
> foofunc();
>
> return 0 ;
> }
>
>
> foo.s:
> ARM
> REQUIRE8
> PRESERVE8
> AREA code, CODE, READONLY, ALIGN=2
>
> foofunc PROC
> VMULL.S32 q1, d2, d4
> MOV pc, lr
>
> ENDP
>
> EXPORT foofunc [CODE]
> END
>
> To unsubscribe from this bug, go to:
> https://bugs.launchpad.net/qemu/+bug/702885/+subscribe
>

[Wolfgang Schildbach (wschi)]

I can confirm that this bug is gone with QEMU commit commit 0fad6efce5d3f18278b7239dece3c251b3e7c04d.