[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RISU PATCH 00/10] Initial support for SVE
From: |
Alex Bennée |
Subject: |
Re: [Qemu-devel] [RISU PATCH 00/10] Initial support for SVE |
Date: |
Wed, 08 Nov 2017 11:02:15 +0000 |
User-agent: |
mu4e 1.0-alpha0; emacs 26.0.90 |
Dave Martin <address@hidden> writes:
> On Tue, Nov 07, 2017 at 03:05:48PM +0000, Alex Bennée wrote:
>> Hi,
>>
>> These patches apply on-top of the last clean-up series:
>>
>> Subject: [RISU PATCH 0/7] Add @Group support and some aarch64.risu cleanups
>> Date: Tue, 31 Oct 2017 14:54:37 +0000
>> Message-Id: <address@hidden>
>>
>> This series adds support for SVE to RISU. Most of the initial patches
>> are plumbing changes to better support arch specific option flags
>> (cleaning up a TODO in the process). I also needed to ensure configure
>> actually honoured CPPFLAGS so it could be passed yet to be released
>> headers.
>
> Should there be a getauxval(AT_HWCAP) & HWCAP_SVE check in this series
> somewhere?
>
> I don't know enough about how RISU is structured to know whether/where
> this is needed.
That would be a saner runtime check to do but it's a balance as RISU is
a fairly specialist tool which kind of assumes people know what they are
doing.
The current check is on SVE_MAGIC in the header files which does mean a
binary compiled on an SVE headered system is now carrying about a much
larger register dump even when run without the --test-sve flag.
Whether it makes sense to be more flexible is a call I'll leave up to
Peter.
>
> [...]
>
> Cheers
> ---Dave
--
Alex Bennée
- Re: [Qemu-devel] [RISU PATCH 10/10] risu_reginfo_aarch64: add SVE support to reginfo_dump_mismatch, (continued)
- [Qemu-devel] [RISU PATCH 03/10] risu.c: add missing --trace longopt, Alex Bennée, 2017/11/07
- [Qemu-devel] [RISU PATCH 07/10] risugen: add --sve support, Alex Bennée, 2017/11/07
- [Qemu-devel] [RISU PATCH 08/10] aarch64.risu: initial SVE instruction, Alex Bennée, 2017/11/07
- [Qemu-devel] [RISU PATCH 09/10] risu_reginfo_aarch64: add reginfo_copy_sve, Alex Bennée, 2017/11/07
- Re: [Qemu-devel] [RISU PATCH 00/10] Initial support for SVE, Dave Martin, 2017/11/08
- Re: [Qemu-devel] [RISU PATCH 00/10] Initial support for SVE,
Alex Bennée <=
- Re: [Qemu-devel] [RISU PATCH 00/10] Initial support for SVE, Peter Maydell, 2017/11/21