Re: [PATCH v2 2/2] ACPI: HMAT: Fix initiator registration for single-initiator systems

2022-11-16 Thread Kirill A. Shutemov
On Wed, Nov 16, 2022 at 04:37:37PM -0700, Vishal Verma wrote: > In a system with a single initiator node, and one or more memory-only > 'target' nodes, the memory-only node(s) would fail to register their > initiator node correctly. i.e. in sysfs: > > # ls

[PATCH v2 2/2] ACPI: HMAT: Fix initiator registration for single-initiator systems

2022-11-16 Thread Vishal Verma
In a system with a single initiator node, and one or more memory-only 'target' nodes, the memory-only node(s) would fail to register their initiator node correctly. i.e. in sysfs: # ls /sys/devices/system/node/node0/access0/targets/ node0 Where as the correct behavior should be: # ls

[PATCH v2 1/2] ACPI: HMAT: remove unnecessary variable initialization

2022-11-16 Thread Vishal Verma
In hmat_register_target_initiators(), the variable 'best' gets initialized in the outer per-locality-type for loop. The initialization just before setting up 'Access 1' targets was unnecessary. Remove it. Cc: Rafael J. Wysocki Cc: Liu Shixin Cc: Dan Williams Acked-by: Kirill A. Shutemov

[PATCH v2 0/2] ACPI: HMAT: fix single-initiator target registrations

2022-11-16 Thread Vishal Verma
drivers/acpi/numa/hmat.c | 27 --- 1 file changed, 20 insertions(+), 7 deletions(-) --- base-commit: 9abf2313adc1ca1b6180c508c25f22f9395cc780 change-id: 20221116-acpi_hmat_fix-7acf4bca37c0 Best regards, -- Vishal Verma

Re: [PATCH 2/2] ACPI: HMAT: Fix initiator registration for single-initiator systems

2022-11-16 Thread kir...@shutemov.name
On Wed, Nov 16, 2022 at 06:02:32PM +, Verma, Vishal L wrote: > On Wed, 2022-11-16 at 15:46 +0300, Kirill A. Shutemov wrote: > > On Wed, Nov 16, 2022 at 12:57:36AM -0700, Vishal Verma wrote: > > > In a system with a single initiator node, and one or more memory-only > > > 'target' nodes, the

Re: [RFC PATCH v2 0/7] On-Demand Paging on SoftRoCE

2022-11-16 Thread Leon Romanovsky
On Fri, Nov 11, 2022 at 06:22:21PM +0900, Daisuke Matsuda wrote: > This patch series implements the On-Demand Paging feature on SoftRoCE(rxe) > driver, which has been available only in mlx5 driver[1] so far. <...> > Daisuke Matsuda (7): > IB/mlx5: Change ib_umem_odp_map_dma_single_page() to

Re: [PATCH 2/2] ACPI: HMAT: Fix initiator registration for single-initiator systems

2022-11-16 Thread Verma, Vishal L
On Wed, 2022-11-16 at 15:46 +0300, Kirill A. Shutemov wrote: > On Wed, Nov 16, 2022 at 12:57:36AM -0700, Vishal Verma wrote: > > In a system with a single initiator node, and one or more memory-only > > 'target' nodes, the memory-only node(s) would fail to register their > > initiator node

Re: [PATCH v1] virtio_pmem: populate numa information

2022-11-16 Thread Pankaj Gupta
> > > > > > > > Compute the numa information for a virtio_pmem device from the > > > > > > > > memory > > > > > > > > range of the device. Previously, the target_node was always 0 > > > > > > > > since > > > > > > > > the ndr_desc.target_node field was never explicitly set. The > > > > > > > >

Re: [PATCH 2/2] ACPI: HMAT: Fix initiator registration for single-initiator systems

2022-11-16 Thread Kirill A. Shutemov
On Wed, Nov 16, 2022 at 12:57:36AM -0700, Vishal Verma wrote: > In a system with a single initiator node, and one or more memory-only > 'target' nodes, the memory-only node(s) would fail to register their > initiator node correctly. i.e. in sysfs: > > # ls

Re: [PATCH 1/2] ACPI: HMAT: remove unnecessary variable initialization

2022-11-16 Thread Kirill A. Shutemov
On Wed, Nov 16, 2022 at 12:57:35AM -0700, Vishal Verma wrote: > In hmat_register_target_initiators(), the variable 'best' gets > initialized in the outer per-locality-type for loop. The initialization > just before setting up 'Access 1' targets was unnecessary. Remove it. > > Cc: Rafael J.

Re: [PATCH 0/2] ACPI: HMAT: fix single-initiator target registrations

2022-11-16 Thread Rafael J. Wysocki
On Wed, Nov 16, 2022 at 8:57 AM Vishal Verma wrote: > > Patch 1 is an obvious cleanup found while fixing this problem. > > Patch 2 Fixes a bug with initiator registration for single-initiator > systems. More details on this in its commit message. > > > Vishal Verma (2): > ACPI: HMAT: remove