Re: [Qemu-devel] [RFC PATCH 04/13] tests/tcg/mips: enable mips32-dsp/mips32-dspr2/mipsr5900 linux-user (WIP)

[Alex Bennée]

Aleksandar Markovic <address@hidden> writes:

> HI, Alex, just a heads up that I plan to submit directory and file
> reorganization of tests/tcg/mips mini patch series today or tomorrow.
> Aleksandar

My current tree state is:

  https://github.com/stsquad/qemu/tree/testing/enable-system-tcg-tests-v2

But I'm still having trouble generating nicely linked no-pic code. But
I'll happily re-base after you've re-organised the tree. Will you be
testing the build as is with the current Makefiles?

>
> On Wednesday, December 19, 2018, Alex Bennée <address@hidden> wrote:
>
>>
>> Aleksandar Markovic <address@hidden> writes:
>>
>> > On Dec 10, 2018 4:29 PM, "Alex Bennée" <address@hidden> wrote:
>> >>
>> >> Convert the existing tests to use our common cross build
>> >> infrastructure.
>> >>
>> >> [WIP: mips32r2 disabled to avoid name clash]
>> >> [WIP: mipsr5900 disabled due to clashing build flags]
>> >>
>> >> Signed-off-by: Alex Bennée <address@hidden>
>> >> ---
>> >>  tests/tcg/mips/Makefile.target       |  15 ++-
>> >>  tests/tcg/mips/mips32-dsp/Makefile   | 166 +++++----------------------
>> >>  tests/tcg/mips/mips32-dspr2/Makefile |  83 +++-----------
>> >>  tests/tcg/mips/mipsr5900/Makefile    |  40 +++----
>> >>  4 files changed, 75 insertions(+), 229 deletions(-)
>> >>
>> >
>> > Alex,
>> >
>> > How about reorganizing directories in tests/tcg/mips altogether, and make
>> > it less confusing, and easier for future developers to approach an work
>> on?
>> >
>> > Let's say like this:
>> >
>> > tests/tcg/mips
>> >     user
>> >         isa
>> >             r5900
>> >         ase
>> >             dsp
>> >             dsp_r2
>> >     system
>> >         isa   《for now empty》
>> >         ase
>> >             dsp
>> >             dsp_r2
>>
>> Yeah sounds like a plan. I'm going to do the same with cris (bare and
>> libc).
>>
>> >
>> > Thanks,
>> > Aleksandar
>>
>>
>> --
>> Alex Bennée
>>


--
Alex Bennée