* [Qemu-devel] [ARM] Contributing tests for Neon
@ 2011-01-21 10:07 Christophe Lyon
2011-01-21 23:25 ` Peter Maydell
0 siblings, 1 reply; 5+ messages in thread
From: Christophe Lyon @ 2011-01-21 10:07 UTC (permalink / raw)
To: qemu-devel@nongnu.org
Hi all,
I have developed some tests for ARM-Neon in the form of C sources files calling ARM Neon intrinsics, and comparing the results of the resulting program with a known reference (eg execution on actual CPU) shows if the execution engine is follows the spec.
These tests currently represent 750KB of sources (150 files, 12000 lines), and I would like to contribute them to the qemu testing infrastructure.
I have a few questions on how to proceed:
- we wish to release the files under the MIT license, is it OK to deliver them as-is in the 'tests' qemu subdir?
- given the size of the whole stuff, I don't think it's suitable that I send it on the list for review, what should I do?
- at the time of writing, GCC/ARM fails to compile those tests correctly, and I rely on ARM's compiler to generate my reference executable and output. Is it acceptable that the Makefile references armcc?
Thanks,
Christophe.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Qemu-devel] [ARM] Contributing tests for Neon
2011-01-21 10:07 [Qemu-devel] [ARM] Contributing tests for Neon Christophe Lyon
@ 2011-01-21 23:25 ` Peter Maydell
2011-01-22 14:24 ` Edgar E. Iglesias
0 siblings, 1 reply; 5+ messages in thread
From: Peter Maydell @ 2011-01-21 23:25 UTC (permalink / raw)
To: Christophe Lyon; +Cc: qemu-devel@nongnu.org
On 21 January 2011 10:07, Christophe Lyon <christophe.lyon@st.com> wrote:
> I have developed some tests for ARM-Neon in the form of C sources files
> calling ARM Neon intrinsics, and comparing the results of the resulting
> program with a known reference (eg execution on actual CPU) shows
> if the execution engine is follows the spec.
>
> These tests currently represent 750KB of sources (150 files, 12000 lines),
That's a larger and more comprehensive test suite than I was
expecting :-) (which is fantastic)
> I have a few questions on how to proceed:
> - we wish to release the files under the MIT license, is it OK to deliver
> them as-is in the 'tests' qemu subdir?
> - given the size of the whole stuff, I don't think it's suitable that I
> send it on the list for review, what should I do?
How about you make it available somewhere (tarball via http, git
repository on gitorious, or other method of your choice) for the
moment? Then we can take a look at it and proceed from there.
> - at the time of writing, GCC/ARM fails to compile those tests correctly,
> and I rely on ARM's compiler to generate my reference executable and
> output. Is it acceptable that the Makefile references armcc?
This is gcc bugs rather than the test sources using armcc-specific
syntax, right?
-- PMM
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Qemu-devel] [ARM] Contributing tests for Neon
2011-01-21 23:25 ` Peter Maydell
@ 2011-01-22 14:24 ` Edgar E. Iglesias
2011-01-24 10:23 ` Christophe Lyon
0 siblings, 1 reply; 5+ messages in thread
From: Edgar E. Iglesias @ 2011-01-22 14:24 UTC (permalink / raw)
To: Peter Maydell; +Cc: Christophe Lyon, qemu-devel@nongnu.org
On Fri, Jan 21, 2011 at 11:25:05PM +0000, Peter Maydell wrote:
> On 21 January 2011 10:07, Christophe Lyon <christophe.lyon@st.com> wrote:
> > I have developed some tests for ARM-Neon in the form of C sources files
> > calling ARM Neon intrinsics, and comparing the results of the resulting
> > program with a known reference (eg execution on actual CPU) shows
> > if the execution engine is follows the spec.
> >
> > These tests currently represent 750KB of sources (150 files, 12000 lines),
>
> That's a larger and more comprehensive test suite than I was
> expecting :-) (which is fantastic)
>
> > I have a few questions on how to proceed:
> > - we wish to release the files under the MIT license, is it OK to deliver
> > them as-is in the 'tests' qemu subdir?
> > - given the size of the whole stuff, I don't think it's suitable that I
> > send it on the list for review, what should I do?
>
> How about you make it available somewhere (tarball via http, git
> repository on gitorious, or other method of your choice) for the
> moment? Then we can take a look at it and proceed from there.
I agree, this is a good first step. If possible, its good if you
can publish source and binaries so people with the appropriate
tools can rebuild the tests. Also, when possible, binaries with
debug info can be helpful when analyzing test failures.
Cheers
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Qemu-devel] [ARM] Contributing tests for Neon
2011-01-22 14:24 ` Edgar E. Iglesias
@ 2011-01-24 10:23 ` Christophe Lyon
2011-01-24 17:04 ` Christophe Lyon
0 siblings, 1 reply; 5+ messages in thread
From: Christophe Lyon @ 2011-01-24 10:23 UTC (permalink / raw)
To: Edgar E. Iglesias; +Cc: Peter Maydell, qemu-devel@nongnu.org
>> How about you make it available somewhere (tarball via http, git
>> repository on gitorious, or other method of your choice) for the
>> moment? Then we can take a look at it and proceed from there.
>
> I agree, this is a good first step. If possible, its good if you
> can publish source and binaries so people with the appropriate
> tools can rebuild the tests. Also, when possible, binaries with
> debug info can be helpful when analyzing test failures.
>
OK, I am going to have a look at gitorious. In the mean time, I have a bit of cleanup and minimal documentation to perform.
Christophe.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2011-01-24 17:05 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-01-21 10:07 [Qemu-devel] [ARM] Contributing tests for Neon Christophe Lyon
2011-01-21 23:25 ` Peter Maydell
2011-01-22 14:24 ` Edgar E. Iglesias
2011-01-24 10:23 ` Christophe Lyon
2011-01-24 17:04 ` Christophe Lyon
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).