From: Zhao Liu <zhao1.liu@intel.com>
To: Alireza Sanaee <alireza.sanaee@huawei.com>
Cc: qemu-devel@nongnu.org, qemu-arm@nongnu.org,
zhenyu.z.wang@intel.com, dapeng1.mi@linux.intel.com,
yongwei.ma@intel.com, armbru@redhat.com, farman@linux.ibm.com,
peter.maydell@linaro.org, mst@redhat.com, anisinha@redhat.com,
shannon.zhaosl@gmail.com, imammedo@redhat.com,
mtosatti@redhat.com, berrange@redhat.com,
richard.henderson@linaro.org, linuxarm@huwei.com,
shameerali.kolothum.thodi@huawei.com,
Jonathan.Cameron@huawei.com, jiangkunkun@huawei.com,
zhao1.liu@intel.com
Subject: Re: [RFC PATCH 0/2] Specifying cache topology on ARM
Date: Mon, 2 Sep 2024 20:23:43 +0800 [thread overview]
Message-ID: <ZtWuT4W5kKEtodyo@intel.com> (raw)
In-Reply-To: <20240902112519.00005b67@huawei.com>
On Mon, Sep 02, 2024 at 11:25:19AM +0100, Alireza Sanaee wrote:
>
> Hi Zhao,
>
> Yes, please keep me CCed.
>
> One thing that I noticed, sometimes, since you were going down the
> Intel path, some variables couldn't be NULL. But when I was gonna go
> down to ARM path, I faced some scenarios where I ended up with
> some uninit vars which is still OK but could have been avoided.
Ah I didn't get your point very clearly. Could you please figure out
those places on my patches? Then I can fix them in my next version. :)
Thanks,
Zhao
> Looking forward to the next revision.
>
> Alireza
next prev parent reply other threads:[~2024-09-02 12:08 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-08-23 12:54 [RFC PATCH 0/2] Specifying cache topology on ARM Alireza Sanaee via
2024-08-23 12:54 ` [PATCH 1/2] target/arm/tcg: increase cache level for cpu=max Alireza Sanaee via
2024-08-23 12:54 ` [PATCH 2/2] hw/acpi: add cache hierarchy node to pptt table Alireza Sanaee via
2024-08-31 11:47 ` Zhao Liu
2024-08-31 11:25 ` [RFC PATCH 0/2] Specifying cache topology on ARM Zhao Liu
2024-09-02 10:25 ` Alireza Sanaee via
2024-09-02 10:25 ` Alireza Sanaee via
2024-09-02 12:23 ` Zhao Liu [this message]
2024-09-02 11:49 ` Marcin Juszkiewicz
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ZtWuT4W5kKEtodyo@intel.com \
--to=zhao1.liu@intel.com \
--cc=Jonathan.Cameron@huawei.com \
--cc=alireza.sanaee@huawei.com \
--cc=anisinha@redhat.com \
--cc=armbru@redhat.com \
--cc=berrange@redhat.com \
--cc=dapeng1.mi@linux.intel.com \
--cc=farman@linux.ibm.com \
--cc=imammedo@redhat.com \
--cc=jiangkunkun@huawei.com \
--cc=linuxarm@huwei.com \
--cc=mst@redhat.com \
--cc=mtosatti@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-arm@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=richard.henderson@linaro.org \
--cc=shameerali.kolothum.thodi@huawei.com \
--cc=shannon.zhaosl@gmail.com \
--cc=yongwei.ma@intel.com \
--cc=zhenyu.z.wang@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.