RE: [PATCH v2] daxctl: Remove unused memory_zone and mem_zone

2023-11-30 Thread Ira Weiny
Xiao Yang (Fujitsu) wrote: > Ping. ^_^ > > -Original Message- > From: Xiao Yang > Sent: 2023年8月11日 9:16 > To: vishal.l.ve...@intel.com; fan...@gmx.us; nvdimm@lists.linux.dev > Cc: linux-...@vger.kernel.org; Yang, Xiao/杨 晓 > Subject: [PATCH v2] daxctl: Remove unused memory_zone and

[PATCH ndctl RESEND 2/2] cxl/region: Fix memory device teardown in disable-region

2023-11-30 Thread Ira Weiny
When a region is requested to be disabled, memory devices are normally automatically torn down. Commit 9399aa667ab0 prevents tear down if memory is online without a force flag. However, daxctl_dev_get_memory() may return NULL if the memory device in question is not system-ram capable as is the

[PATCH ndctl RESEND 1/2] ndctl/test: Add destroy region test

2023-11-30 Thread Ira Weiny
Commit 9399aa667ab0 ("cxl/region: Add -f option for disable-region") introduced a regression when destroying a region. Add a tests for destroying a region. Cc: Dave Jiang Signed-off-by: Ira Weiny --- test/cxl-destroy-region.sh | 76 ++

[PATCH ndctl RESEND 0/2] ndctl: Fix ups for region destroy

2023-11-30 Thread Ira Weiny
/meson.build | 2 ++ 6 files changed, 89 insertions(+), 2 deletions(-) --- base-commit: cbf049039482a56c2b66ede3e10d5e9c652890b7 change-id: 20231130-fix-region-destroy-a85ad1fde87b Best regards, -- Ira Weiny

[NDCTL PATCH v3 2/2] cxl: Add check for regions before disabling memdev

2023-11-30 Thread Dave Jiang
Add a check for memdev disable to see if there are active regions present before disabling the device. This is necessary now regions are present to fulfill the TODO that was left there. The best way to determine if a region is active is to see if there are decoders enabled for the mem device. This

Re: [NDCTL PATCH v2 2/2] cxl: Add check for regions before disabling memdev

2023-11-30 Thread Dave Jiang
On 11/30/23 01:29, Cao, Quanquan/曹 全全 wrote: > >>   static int action_disable(struct cxl_memdev *memdev, struct action_context >> *actx) >>   { >> +    struct cxl_endpoint *ep; >> +    struct cxl_port *port; >> + >>   if (!cxl_memdev_is_enabled(memdev)) >>   return 0; >>   -    if