[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 15/16] accel: Rename 'cpu_state' -> 'cpu'
- To: Philippe Mathieu-Daudé <philmd@xxxxxxxxxx>, qemu-devel@xxxxxxxxxx
- From: Richard Henderson <richard.henderson@xxxxxxxxxx>
- Date: Thu, 22 Jun 2023 19:51:09 +0200
- Cc: Reinoud Zandijk <reinoud@xxxxxxxxxx>, qemu-arm@xxxxxxxxxx, kvm@xxxxxxxxxxxxxxx, Marcel Apfelbaum <marcel.apfelbaum@xxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Anthony Perard <anthony.perard@xxxxxxxxxx>, Yanan Wang <wangyanan55@xxxxxxxxxx>, Peter Maydell <peter.maydell@xxxxxxxxxx>, Roman Bolshakov <rbolshakov@xxxxxxx>, Paolo Bonzini <pbonzini@xxxxxxxxxx>, Paul Durrant <paul@xxxxxxx>, Sunil Muthuswamy <sunilmut@xxxxxxxxxxxxx>, Alexander Graf <agraf@xxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx, Eduardo Habkost <eduardo@xxxxxxxxxxx>, Cameron Esfahani <dirty@xxxxxxxxx>
- Delivery-date: Thu, 22 Jun 2023 17:51:29 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 6/22/23 18:08, Philippe Mathieu-Daudé wrote:
Most of the codebase uses 'CPUState *cpu' or 'CPUState *cs'.
While 'cpu_state' is kind of explicit, it makes the code
harder to review. Simply rename as 'cpu' like the rest.
Signed-off-by: Philippe Mathieu-Daudé <philmd@xxxxxxxxxx>
I would have chosen 'cs', since 'cpu' is often used for ArchCPU. But ok.
Acked-by: Richard Henderson <richard.henderson@xxxxxxxxxx>
r~
|
