From: "Yuquan Wang" <wangyuquan1236@phytium.com.cn>
To: marcin.juszkiewicz <marcin.juszkiewicz@linaro.org>,
jonathan.cameron <Jonathan.Cameron@Huawei.com>,
quic_llindhol <quic_llindhol@quicinc.com>,
peter.maydell <peter.maydell@linaro.org>
Cc: qemu-devel <qemu-devel@nongnu.org>, qemu-arm <qemu-arm@nongnu.org>
Subject: Re: [RFC PATCH 1/2] hw/arm/sbsa-ref: Enable CXL Host Bridge by pxb-cxl
Date: Thu, 24 Oct 2024 18:10:34 +0800 [thread overview]
Message-ID: <2024102418103368919519@phytium.com.cn> (raw)
In-Reply-To: e632758d-893f-4a44-b081-9fdd92c19548@linaro.org
Hi,Marcin
I am updating this patches into v2 with Separate MMIO address space for CXL,
however, I'm not confident about the addresss design on sbsa-ref. Below are some
questions about that.
1) With the pxb-cxl-host, any cxl root ports and cxl endpoint devices would occupy the
BDF number of the original pcie domain. Hence, the max available pcie devices on sbsa-ref
would decrease, which seems to bring a series of trouble. Do you have any suggestions?
2) In the situation described above, is it necessary to add a separate ecam space for cxl host?
--------------
Many thanks
Yuquan Wang
>On 30.08.2024 06:15, Yuquan Wang wrote:
>> The memory layout places 1M space for 16 host bridge register regions
>> in the sbsa-ref memmap. In addition, this creates a default pxb-cxl
>> (bus_nr=0xfe) bridge with one cxl-rp on sbsa-ref.
>
>With this patchset applied I no longer can add pcie devices to sbsa-ref.
>
>-device nvme,serial=deadbeef,bus=root_port_for_nvme1,drive=hdd
>-drive file=disks/full-debian.hddimg,format=raw,id=hdd,if=none
>
>Normally this adds NVME as pcie device but now it probably ends on
>pxb-cxl bus instead.
>
>Also please bump platform_version.minor and document adding CXL in
>docs/system/arm/sbsa.rst file.
next prev parent reply other threads:[~2024-10-24 10:11 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-08-30 4:15 [RFC PATCH 0/2] Sbsa-ref CXL Enablement Yuquan Wang
2024-08-30 4:15 ` [RFC PATCH 1/2] hw/arm/sbsa-ref: Enable CXL Host Bridge by pxb-cxl Yuquan Wang
2024-08-30 9:52 ` Jonathan Cameron
2024-08-30 9:52 ` Jonathan Cameron via
2024-08-30 9:52 ` Jonathan Cameron via
2024-09-09 10:05 ` Marcin Juszkiewicz
2024-10-24 10:10 ` Yuquan Wang [this message]
2024-08-30 4:15 ` [RFC PATCH 2/2] hw/arm/sbsa-ref: Support CXL Fixed Memory Window Yuquan Wang
2024-08-30 10:23 ` Jonathan Cameron
2024-08-30 10:23 ` Jonathan Cameron via
2024-08-30 11:18 ` [RFC PATCH 0/2] Sbsa-ref CXL Enablement Jonathan Cameron
2024-08-30 11:18 ` Jonathan Cameron via
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=2024102418103368919519@phytium.com.cn \
--to=wangyuquan1236@phytium.com.cn \
--cc=Jonathan.Cameron@Huawei.com \
--cc=marcin.juszkiewicz@linaro.org \
--cc=peter.maydell@linaro.org \
--cc=qemu-arm@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=quic_llindhol@quicinc.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.