[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL v2 12/27] target/mips: Convert to CPUClass::tlb_f
From: |
Alex Bennée |
Subject: |
Re: [Qemu-devel] [PULL v2 12/27] target/mips: Convert to CPUClass::tlb_fill |
Date: |
Tue, 14 May 2019 17:28:00 +0100 |
User-agent: |
mu4e 1.3.1; emacs 26.1 |
Philippe Mathieu-Daudé <address@hidden> writes:
> On 5/14/19 5:48 PM, Alex Bennée wrote:
>>
>> Aleksandar Markovic <address@hidden> writes:
>>
>>> On May 10, 2019 8:57 PM, "Richard Henderson" <address@hidden>
>>> wrote:
>>>>
>>>
>>> Please change the title to 'target/mips: Switch to using
>>> mips_cpu_tlb_fill()', or something along that line.
>>
>> It does seem a little redundant as "target/mips:" already marks it as a
>> mips specific change and viewing the log you can see a series of
>> architectures being converted to a new API.
>>
>>> Also, the reason for changing the field access_type to mips_access type
>>> should be explained in the commit message.
>>
>> ok
>>
>>> This commit message is generally poor, as it explains relatively
>>> unimportant logging issue, while not explaining the core of the
>>> change.
>>
>> Surely the core of the change is explained in the main patches that
>> introduce the new API? I think it would be redundant to repeat that for
>> every individual architecture touched. It's a shame it's hard to
>> explicitly reference a patch in the same series as the commit hashes are
>> not yet permanent. At least when we fix things referring to the short
>> hash of the original commit is fairly easy.
>
> Except in the case the maintainer is sending a pull request (like here)
> where he can manually fix the commits. Still this is a PITA...
If there was tooling that can go from a patch series to a pull request
then maybe. But generally a PR is a series of patches that have now
passed some standard and can now be sent. I'm not sure I'd want to go
over all my commits and re-edit the messages at that point.
--
Alex Bennée
- [Qemu-devel] [PULL v2 00/27] tcg: Add CPUClass::tlb_fill, Richard Henderson, 2019/05/10
- [Qemu-devel] [PULL v2 03/27] target/arm: Convert to CPUClass::tlb_fill, Richard Henderson, 2019/05/10
- [Qemu-devel] [PULL v2 06/27] target/i386: Convert to CPUClass::tlb_fill, Richard Henderson, 2019/05/10
- [Qemu-devel] [PULL v2 12/27] target/mips: Convert to CPUClass::tlb_fill, Richard Henderson, 2019/05/10
- Re: [Qemu-devel] [PULL v2 12/27] target/mips: Convert to CPUClass::tlb_fill, Alex Bennée, 2019/05/14
- Re: [Qemu-devel] [PULL v2 12/27] target/mips: Convert to CPUClass::tlb_fill, Philippe Mathieu-Daudé, 2019/05/14
- Re: [Qemu-devel] [PULL v2 12/27] target/mips: Convert to CPUClass::tlb_fill,
Alex Bennée <=
[Qemu-devel] [PULL v2 16/27] target/ppc: Convert to CPUClass::tlb_fill, Richard Henderson, 2019/05/10
[Qemu-devel] [PULL v2 17/27] target/riscv: Convert to CPUClass::tlb_fill, Richard Henderson, 2019/05/10
[Qemu-devel] [PULL v2 18/27] target/s390x: Convert to CPUClass::tlb_fill, Richard Henderson, 2019/05/10
[Qemu-devel] [PULL v2 27/27] tcg: Use tlb_fill probe from tlb_vaddr_to_host, Richard Henderson, 2019/05/10
Re: [Qemu-devel] [PULL v2 00/27] tcg: Add CPUClass::tlb_fill, Aleksandar Markovic, 2019/05/11
Re: [Qemu-devel] [PULL v2 00/27] tcg: Add CPUClass::tlb_fill, Peter Maydell, 2019/05/14