From: "Philippe Mathieu-Daudé" <f4bug@amsat.org>
To: Aleksandar Rikalo <Aleksandar.Rikalo@syrmia.com>,
Aleksandar Markovic <aleksandar.markovic@rt-rk.com>,
"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
Vince Del Vecchio <Vince.DelVecchio@mediatek.com>
Cc: "Richard Henderson" <richard.henderson@linaro.org>,
"Petar Jovanovic" <petar.jovanovic@syrmia.com>,
"Aleksandar Markovic" <amarkovic@wavecomp.com>,
"Alex Bennée" <alex.bennee@linaro.org>,
"aurelien@aurel32.net" <aurelien@aurel32.net>,
"Filip Vidojevic" <Filip.Vidojevic@syrmia.com>
Subject: Re: [Qemu-devel] [PATCH v4 0/3] tests: Reorganize MIPS TCG directories and files
Date: Sat, 29 May 2021 14:41:42 +0200 [thread overview]
Message-ID: <e5ca03fc-c412-9ad1-0d45-ee36dff9e4ca@amsat.org> (raw)
In-Reply-To: <DM5PR22MB16585EFE36FC0584D37F3724D29A0@DM5PR22MB1658.namprd22.prod.outlook.com>
Hi Aleksandar Rikalo,
On 1/24/19 4:37 PM, Aleksandar Rikalo wrote:
>> From: Aleksandar Markovic <aleksandar.markovic@rt-rk.com>
>> Sent: Thursday, January 24, 2019 4:19 PM
>> To: qemu-devel@nongnu.org
>> Cc: aurelien@aurel32.net; Aleksandar Markovic; Aleksandar Rikalo
>> Subject: [PATCH v4 0/3] tests: Reorganize MIPS TCG directories and files
>>
>> From: Aleksandar Markovic <amarkovic@wavecomp.com>
>>
>> Reorganize MIPS TCG directories and files.
>>
>> Directories for DSP tests in system mode are deleted. They
>> introduce more complexity in test structure compared to the
>> value they add. Still, these tests may be moved somewhere
>> else, let's say in a new acceptance test that will boot
>> "mips64r2" virtual machine.
>>
>> All remaining test will are user mode tests, which will
>> simplify maintenance of mips tcg tests overall.
>>
>> The file movement is done using "git mv" command, so "git
>> blame" will still display the original information, regardless
>> of the new names and locations of involved files. Also, the
>> command "git config diff.renames true" was used to keep the
>> size of movement patches reasonable.
>>
>> v3->v4:
>> - DSP ASE system mode tests given up
>> - Makefiles provided for tests that remain
>>
>> v2->v3:
>> - corrected missing "v2" that caused wrong message linking
>> (the version is now "v3", of course)
>>
>> v1->v2:
>> - used "git config diff.renames true" before sending
>> - rebased to the latest code (included newly added r5900 tests)
>>
>>
>> Aleksandar Markovic (3):
>> tests: tcg: mips: Move source files to new locations
>> tests: tcg: mips: Add two new Makefiles
>> tests: tcg: mips: Remove old directories
>
> I too think that this will simplify the test dirs structure and
> maintenance. I cannot wait to see these directories populated with
> new tests for various MIPS ISAs and ASEs.
>
> For all patches in this series:
>
> Reviewed-by: Aleksandar Rikalo <arikalo@wavecomp.com>
Apparently this series was a bit too enthusiastic and removed
many tests (MIPS64 DSP r1 and r2 for example) which then got
never reintroduced. What is your plan regarding the "new
tests" you mentioned?
Thanks,
Phil.
next prev parent reply other threads:[~2021-05-29 12:43 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-01-24 15:19 [Qemu-devel] [PATCH v4 0/3] tests: Reorganize MIPS TCG directories and files Aleksandar Markovic
2019-01-24 15:19 ` [Qemu-devel] [PATCH v4 1/3] tests: tcg: mips: Move source files to new locations Aleksandar Markovic
2019-01-24 15:19 ` [Qemu-devel] [PATCH v4 2/3] tests: tcg: mips: Add two new Makefiles Aleksandar Markovic
2019-01-24 15:19 ` [Qemu-devel] [PATCH v4 3/3] tests: tcg: mips: Remove old directories Aleksandar Markovic
2019-01-24 15:37 ` [Qemu-devel] [PATCH v4 0/3] tests: Reorganize MIPS TCG directories and files Aleksandar Rikalo
2021-05-29 12:41 ` Philippe Mathieu-Daudé [this message]
2021-06-04 12:01 ` Aleksandar Rikalo
2019-01-24 17:22 ` Philippe Mathieu-Daudé
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=e5ca03fc-c412-9ad1-0d45-ee36dff9e4ca@amsat.org \
--to=f4bug@amsat.org \
--cc=Aleksandar.Rikalo@syrmia.com \
--cc=Filip.Vidojevic@syrmia.com \
--cc=Vince.DelVecchio@mediatek.com \
--cc=aleksandar.markovic@rt-rk.com \
--cc=alex.bennee@linaro.org \
--cc=amarkovic@wavecomp.com \
--cc=aurelien@aurel32.net \
--cc=petar.jovanovic@syrmia.com \
--cc=qemu-devel@nongnu.org \
--cc=richard.henderson@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).