[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/4] accel/tcg: Add cluster number to TCG TB has
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH 3/4] accel/tcg: Add cluster number to TCG TB hash |
Date: |
Mon, 14 Jan 2019 11:53:06 +0000 |
On Mon, 14 Jan 2019 at 01:08, Aleksandar Markovic
<address@hidden> wrote:
> I do understand the definition of cluster_index in the sense
> of this series. However, it looks to me that the term
> "cluster" is generally overused in areas where we work.
> This may lead to some confusion for future developers, and
> let me suggest some other name, like "tcg_cluster_index" or
> "tcg_group_id", or "translation_group_id".
Note also that the cluster index is not purely a TCG
concept -- it also (in master at the moment) affects
the gdbstub interface. Different clusters appear as
separate processes in gdb, whereas different CPUs in
the same cluster are different threads in the same CPU.
(And by default only the first cluster's CPUs will
appear, unless you explicitly attach to the second cluster:
this is a limitation of how gdb's UI handles multiple
processes.)
thanks
-- PMM
[Qemu-devel] [PATCH 4/4] gdbstub: Simplify gdb_get_cpu_pid() to use cpu->cluster_index, Peter Maydell, 2019/01/08
Re: [Qemu-devel] [PATCH 0/4] tcg: support heterogenous CPU clusters, Richard Henderson, 2019/01/09