public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Brian J. Johnson" <brian.johnson@hpe.com>
To: <devel@edk2.groups.io>, <yoshinoyatoko@163.com>
Cc: <jonathan.cameron@huawei.com>, Laszlo Ersek <lersek@redhat.com>,
	<kraxel@redhat.com>, "Ni, Ray" <ray.ni@intel.com>,
	Sayanta Pattanayak <sayanta.pattanayak@arm.com>
Subject: Re: [edk2-devel] question about cxl device enumeration in pci bus driver
Date: Tue, 7 Nov 2023 15:54:01 -0600	[thread overview]
Message-ID: <ddd26f4b-2a3a-4415-9336-2492e1c28f61@hpe.com> (raw)
In-Reply-To: <55e93c9e.6c41.18ba45c8251.Coremail.yoshinoyatoko@163.com>

[-- Attachment #1: Type: text/plain, Size: 5786 bytes --]

I see a long discussion on lore.kernel.org:

https://lore.kernel.org/linux-cxl/BYAPR12MB3336F5ADDEDBB6245BC2B48DBD489@BYAPR12MB3336.namprd12.prod.outlook.com/T/

This document also has some info on how CXL hotplug is handled:

https://cdrdv2-public.intel.com/643805/643805_CXL%20Memory%20Device%20SW%20Guide_Rev1p0.pdf

*Brian J. Johnson
*Enterprise X86 Lab

Hewlett Packard Enterprise

------------------------------------------------------------------------
*From:* Yoshinoya [mailto:yoshinoyatoko@163.com]
*Sent:* Monday, November 6, 2023 at 5:20 AM
*To:* devel@edk2.groups.io
*Cc:* jonathan.cameron@huawei.com, Laszlo Ersek <lersek@redhat.com>, 
kraxel@redhat.com, Ni, Ray <ray.ni@intel.com>, Sayanta Pattanayak 
<sayanta.pattanayak@arm.com>
*Subject:* [edk2-devel] question about cxl device enumeration in pci bus 
driver

> Hi,
> I have a question about cxl memory device hot-plug.
>
> For example:
> 1. When cold boot, the platform has only 512GB cxl type-3 memory
> 2. During OS runtime, user hot-plug another cxl type-3  memory device 
> expanding to 1TB.
>
> So, how did OS identify another 512GB space newly added without a reboot?
>
> Could anyone help to explain the procedure draftly?
>
> Thanks
>
>
>
>
> 在 2023-10-27 09:29:31,"Yoshinoya" <yoshinoyatoko@163.com> 写道:
>
>
>         Hi,
>         Thanks for reply!
>
>         I download code from this git
>         https://github.com/SayantaP-arm/edk2-platforms/
>
>         For this ARM edk2 sample package, it provided cxldxe driver
>         which being executed after UEFI DXE PciBus enumeration finishes.
>         This ppt (https://lpc.events/event/16/contributions/1254/
>         <https://lpc.events/event/16/contributions/1254/>)
>         describes good.
>
>         Intel also provied a CXL Type 3 memory device software guide.
>         This guide also describes system firmware boot sequence and
>         uefi boot sequence.
>         But i could not match these describes with standard UEFI BIOS
>         Boot flow, such as dxe phase's standard pci enumeration driver.
>         It sees needing add some cxl discovery code into dxe pci bus
>         driver.
>
>         Thanks
>
>
>
>
>
>         At 2023-10-26 21:35:38, "Jonathan Cameron via groups.io"<jonathan.cameron=huawei.com@groups.io>  wrote:
>         >On Thu, 26 Oct 2023 11:49:28 +0200
>         >"Laszlo Ersek" <lersek@redhat.com>  wrote:
>         >
>         >> On 10/26/23 10:33, Gerd Hoffmann wrote:
>         >> > On Thu, Oct 26, 2023 at 10:36:35AM +0800, Yoshinoya wrote:
>         >> >   
>         >> >> CXL Host Bridge / Root Port / Switch / Device enumeration / HDM Config, maybe could be integrated into pci drivers stack.  
>         >> > 
>         >> > Point being?  Can or should the firmware do anything useful with
>         >> > the CXL hardware?  If so, what exactly and why?
>         >> > 
>         >> > Current state of affairs is that the PCI stack does the usual PCI
>         >> > initialization (enumerate, assign resources to PCI bars) and leaves
>         >> > everything else to the OS.  
>         >> 
>         >> (I don't know what "HDM Config" stands for.)
>         >> 
>         >> The only utility for driving CXL devices from the firmware could be, AFAICT:
>         >> 
>         >> - booting off of such a device (or at least "supporting OS boot" in some
>         >> manner)
>         >> 
>         >> - using such a device for UEFI console purposes
>         >
>         >There are different models for how to use CXL devices and what's possible depends on the
>         >version of CXL.  CXL 1.1 wasn't great for standards defined discovery, so
>         >EDK2 platform logic basically has to do everything.
>         >
>         >The one mostly expected for early CXL servers, for backwards compatibility, makes
>         >setting up the CXL memory decoders (Host managed Device Memory - HDM) in all the
>         >components in the path to memory + locking them down an EDK2 problem.  They are then
>         >presented in the memory map and in SRAT, HMAT etc the same as normal DDR memory.
>         >Idea being that an old OS will be fine with that and doesn't have to be CXL aware
>         >at all.  Note this also involves walking the CDAT tables via DOE mailboxes in PCI
>         >config space to get the magic numbers needed to compute HMAT.
>         >
>         >The other model is to do very little in EDK2 and make entirely a problem for the OS.
>         >The logic is necessary anyway if you want to support hotplug etc, so use it for the
>         >cold plug paths 2.  That's all we've currently supported on QEMU.
>         >
>         >There was a presentation at Linux Plumbers last year on some out of tree support
>         >from ARM for doing the setup on a CXL 2.0 platform (I think) in EDK2 
>         >https://lpc.events/event/16/contributions/1254/  <https://lpc.events/event/16/contributions/1254/>
>         >But I guess it never went upstream.
>         >https://github.com/SayantaP-arm/edk2-platforms/tree/cxl-type-3
>         >
>         >+CC Sayanta
>         >
>         >Jonathan
>         >> 
>         >> Laszlo
>         >> 
>         >> 
>         >> 
>         >> 
>         >> 
>         >> 
>         >
>         >
>         >
>         >
>         >
>
> 


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#110877): https://edk2.groups.io/g/devel/message/110877
Mute This Topic: https://groups.io/mt/102173204/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



[-- Attachment #2: Type: text/html, Size: 12817 bytes --]

  reply	other threads:[~2023-11-07 21:54 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-10-25  6:01 [edk2-devel] question about cxl device enumeration in pci bus driver Yoshinoya
2023-10-25 10:13 ` Ni, Ray
2023-10-26  2:36   ` Yoshinoya
2023-10-26  8:33     ` Gerd Hoffmann
2023-10-26  9:49       ` Laszlo Ersek
2023-10-26 13:35         ` Jonathan Cameron via groups.io
2023-10-27  1:26           ` Yoshinoya
     [not found]           ` <1791D2898E0C74CA.20272@groups.io>
2023-10-27  1:29             ` Yoshinoya
2023-10-27  5:49               ` Ni, Ray
2023-11-06 11:20               ` Yoshinoya
2023-11-07 21:54                 ` Brian J. Johnson [this message]
2023-11-17  1:41                   ` Yoshinoya
2023-11-07 11:31 ` [edk2-devel] Use dynamic pcd in smm mode Yoshinoya
2023-11-07 12:45   ` Laszlo Ersek
2023-11-09  6:03     ` Yoshinoya
2023-11-17  2:15   ` [edk2-devel] question about PrmPkg Yoshinoya
2023-11-17  8:42     ` Laszlo Ersek
2023-11-17 16:04       ` Michael Kubacki
2023-11-20  2:09         ` Yoshinoya
2023-11-20 13:58           ` Michael Kubacki
2024-02-27 10:06           ` [edk2-devel] Question about SMM code privilege switch Yoshinoya
2024-03-04  1:20             ` Yoshinoya
2024-05-23  2:07               ` [edk2-devel] What PTSS abbreviation means Yoshinoya
2024-02-05  9:49     ` [edk2-devel] sctpackage failed to be compile with new udk codebase Yoshinoya

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-list 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=ddd26f4b-2a3a-4415-9336-2492e1c28f61@hpe.com \
    --to=devel@edk2.groups.io \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox