[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: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PULL v2 12/27] target/mips: Convert to CPUClass::tlb_fill |
Date: |
Tue, 14 May 2019 16:25:53 +0100 |
On Sat, 11 May 2019 at 14:43, Aleksandar Markovic
<address@hidden> wrote:
> This commit message is generally poor, as it explains relatively unimportant
> logging issue, while not explaining the core of the change.
I think the assumption with this sort of "refactor to change
all instances of an API" change is that you will go and
look at the commit message (or cover letter for the patch
series) that introduces the new API; there's no great need
to repeat the same justification for every commit that
applies the refactoring to each of our dozen or more
guest front-ends.
I'm not sure that wordsmithing a commit message really
justifies rerolling this pull request at this point.
thanks
-- PMM
- [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, 2019/05/14
[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