Message ID | 20230901012914.226527-1-gregory.price@memverge.com |
---|---|
Headers | show |
Series | CXL: SK hynix Niagara MHSLD Device | expand |
On Tue, Sep 05, 2023 at 11:04:56AM +0200, Philippe Mathieu-Daudé wrote: > On 1/9/23 03:29, Gregory Price wrote: > > v2: > > - 5 patch series, first 4 are pull-aheads that can be merged separately > > > > This patch set includes an emulation of the SK hynix Niagara MHSLD > > platform with custom CCI commands that allow for isolation of memory > > blocks between attached hosts. > > > > Being at commit 17780edd81 I can't apply this series: > > Applying: cxl/mailbox: move mailbox effect definitions to a header > error: patch failed: hw/cxl/cxl-mailbox-utils.c:12 > error: hw/cxl/cxl-mailbox-utils.c: patch does not apply > Patch failed at 0001 cxl/mailbox: move mailbox effect definitions to a > header > > On what is it based? This is based on jonathan's branch here: https://gitlab.com/jic23/qemu/-/tree/cxl-2023-08-30 I try to keep things on top of his working branch to avoid any rebasing issues. I should have included that with the cover letter, my bad. ~Gregory
On 1/9/23 03:29, Gregory Price wrote: > v2: > - 5 patch series, first 4 are pull-aheads that can be merged separately > - cci: rebased on 8-30 branch from jic23, dropped cci patches > - mailbox: dropped MHD commands, integrated into niagara (for now) > - mailbox: refactor CCI defines to avoid redefinition in niagara > - type3: cleanup duplicate typecasting > - type3: expose ct3 functions so inheriting devices may access them > - type3: add optional mhd validation function for memory access > - niagara: refactor to make niagara inherit type3 and override behavior > - niagara: refactor command definitions and types into header to make > understanding the device a bit easier for users > - style and formatting > > This patch set includes an emulation of the SK hynix Niagara MHSLD > platform with custom CCI commands that allow for isolation of memory > blocks between attached hosts. > > This device allows hosts to request memory blocks directly from the device, > rather than requiring full the DCD command set. As a matter of simplicity, > this is beneficial to for testing and applications of dynamic memory > pooling on top of the 1.1 and 2.0 specification. > > Note that these CCI commands are not servicable without a proper driver or > the kernel allowing raw CXL commands to be passed through the mailbox > driver, so users should enable `CONFIG_CXL_MEM_RAW_COMMANDS=y` on the > kernel of their QEMU instance if they wish to test it > > Signed-off-by: Gregory Price <gregory.price@memverge.com> > > Gregory Price (5): > cxl/mailbox: move mailbox effect definitions to a header > cxl/type3: Cleanup multiple CXL_TYPE3() calls in read/write functions > cxl/type3: Expose ct3 functions so that inheriters can call them > cxl/type3: add an optional mhd validation function for memory accesses > cxl/vendor: SK hynix Niagara Multi-Headed SLD Device Being at commit 17780edd81 I can't apply this series: Applying: cxl/mailbox: move mailbox effect definitions to a header error: patch failed: hw/cxl/cxl-mailbox-utils.c:12 error: hw/cxl/cxl-mailbox-utils.c: patch does not apply Patch failed at 0001 cxl/mailbox: move mailbox effect definitions to a header On what is it based? Thanks, Phil.
v2: - 5 patch series, first 4 are pull-aheads that can be merged separately - cci: rebased on 8-30 branch from jic23, dropped cci patches - mailbox: dropped MHD commands, integrated into niagara (for now) - mailbox: refactor CCI defines to avoid redefinition in niagara - type3: cleanup duplicate typecasting - type3: expose ct3 functions so inheriting devices may access them - type3: add optional mhd validation function for memory access - niagara: refactor to make niagara inherit type3 and override behavior - niagara: refactor command definitions and types into header to make understanding the device a bit easier for users - style and formatting This patch set includes an emulation of the SK hynix Niagara MHSLD platform with custom CCI commands that allow for isolation of memory blocks between attached hosts. This device allows hosts to request memory blocks directly from the device, rather than requiring full the DCD command set. As a matter of simplicity, this is beneficial to for testing and applications of dynamic memory pooling on top of the 1.1 and 2.0 specification. Note that these CCI commands are not servicable without a proper driver or the kernel allowing raw CXL commands to be passed through the mailbox driver, so users should enable `CONFIG_CXL_MEM_RAW_COMMANDS=y` on the kernel of their QEMU instance if they wish to test it Signed-off-by: Gregory Price <gregory.price@memverge.com> Gregory Price (5): cxl/mailbox: move mailbox effect definitions to a header cxl/type3: Cleanup multiple CXL_TYPE3() calls in read/write functions cxl/type3: Expose ct3 functions so that inheriters can call them cxl/type3: add an optional mhd validation function for memory accesses cxl/vendor: SK hynix Niagara Multi-Headed SLD Device hw/cxl/Kconfig | 4 + hw/cxl/cxl-mailbox-utils.c | 35 +- hw/cxl/meson.build | 2 + hw/cxl/vendor/meson.build | 1 + hw/cxl/vendor/skhynix/.gitignore | 1 + hw/cxl/vendor/skhynix/init_niagara.c | 99 +++++ hw/cxl/vendor/skhynix/meson.build | 1 + hw/cxl/vendor/skhynix/skhynix_niagara.c | 516 ++++++++++++++++++++++++ hw/cxl/vendor/skhynix/skhynix_niagara.h | 169 ++++++++ hw/mem/cxl_type3.c | 33 +- include/hw/cxl/cxl_device.h | 8 + include/hw/cxl/cxl_mailbox.h | 18 + 12 files changed, 863 insertions(+), 24 deletions(-) create mode 100644 hw/cxl/vendor/meson.build create mode 100644 hw/cxl/vendor/skhynix/.gitignore create mode 100644 hw/cxl/vendor/skhynix/init_niagara.c create mode 100644 hw/cxl/vendor/skhynix/meson.build create mode 100644 hw/cxl/vendor/skhynix/skhynix_niagara.c create mode 100644 hw/cxl/vendor/skhynix/skhynix_niagara.h create mode 100644 include/hw/cxl/cxl_mailbox.h