[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 23/43] target/riscv: Remove misa_mxl validation
From: |
Alex Bennée |
Subject: |
Re: [PATCH v2 23/43] target/riscv: Remove misa_mxl validation |
Date: |
Mon, 15 Jan 2024 11:51:27 +0000 |
User-agent: |
mu4e 1.11.27; emacs 29.1 |
Alex Bennée <alex.bennee@linaro.org> writes:
> Alistair Francis <alistair23@gmail.com> writes:
>
>> On Tue, Jan 9, 2024 at 1:43 AM Alex Bennée <alex.bennee@linaro.org> wrote:
>>>
>>> Alex Bennée <alex.bennee@linaro.org> writes:
>>>
>>> > From: Akihiko Odaki <akihiko.odaki@daynix.com>
>>> >
>>> > It is initialized with a simple assignment and there is little room for
>>> > error. In fact, the validation is even more complex.
>>> >
>>> > Signed-off-by: Akihiko Odaki <akihiko.odaki@daynix.com>
>>> > Acked-by: LIU Zhiwei <zhiwei_liu@linux.alibaba.com>
>>> > Reviewed-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
>>> > Acked-by: Alistair Francis <alistair.francis@wdc.com>
>>> > Message-Id: <20231213-riscv-v7-2-a760156a337f@daynix.com>
>>> > Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
>>>
>>> ping: along with this are the RiscV maintainers happy for me to take:
>>>
>>> [PATCH v2 23/43] target/riscv: Remove misa_mxl validation
>>> [PATCH v2 24/43] target/riscv: Move misa_mxl_max to class
>>> [PATCH v2 25/43] target/riscv: Validate misa_mxl_max only once
>>>
>>> through my next pull request?
>>
>> Yep! Go for it
>
> Sadly some of the recent changes that went through have created a merge
> conflict.
>
> Akihiko,
>
> Any chance you could re-spin just the MISA_MXL patches?
Never mind, I think I got it figured out.
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
- [PATCH v2 16/43] tests/qtest: Bump the device-introspect-test timeout to 12 minutes, (continued)
- [PATCH v2 16/43] tests/qtest: Bump the device-introspect-test timeout to 12 minutes, Alex Bennée, 2024/01/03
- [PATCH v2 15/43] qtest: bump bios-table-test timeout to 9 minutes, Alex Bennée, 2024/01/03
- [PATCH v2 27/43] target/ppc: Use GDBFeature for dynamic XML, Alex Bennée, 2024/01/03
- [PATCH v2 13/43] qtest: bump qos-test timeout to 2 minutes, Alex Bennée, 2024/01/03
- [PATCH v2 32/43] gdbstub: Simplify XML lookup, Alex Bennée, 2024/01/03
- [PATCH v2 31/43] gdbstub: Change gdb_get_reg_cb and gdb_set_reg_cb, Alex Bennée, 2024/01/03
- [PATCH v2 23/43] target/riscv: Remove misa_mxl validation, Alex Bennée, 2024/01/03
[PATCH v2 28/43] target/riscv: Use GDBFeature for dynamic XML, Alex Bennée, 2024/01/03
[PATCH v2 36/43] plugins: Use different helpers when reading registers, Alex Bennée, 2024/01/03
[PATCH v2 22/43] hw/riscv: Use misa_mxl instead of misa_mxl_max, Alex Bennée, 2024/01/03
[PATCH v2 42/43] docs/devel: lift example and plugin API sections up, Alex Bennée, 2024/01/03
[PATCH v2 20/43] mtest2make: stop disabling meson test timeouts, Alex Bennée, 2024/01/03
[PATCH v2 19/43] tests/fp: Bump fp-test-mulAdd test timeout to 3 minutes, Alex Bennée, 2024/01/03
[PATCH v2 39/43] contrib/plugins: fix imatch, Alex Bennée, 2024/01/03