Commit 14f16d47 authored by Vishal Verma's avatar Vishal Verma Committed by Dan Williams
Browse files

ACPI: HMAT: remove unnecessary variable initialization 



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 <rafael@kernel.org>
Cc: Liu Shixin <liushixin2@huawei.com>
Cc: Dan Williams <dan.j.williams@intel.com>
Acked-by: default avatarKirill A. Shutemov <kirill.shutemov@linux.intel.com>
Acked-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
Signed-off-by: default avatarVishal Verma <vishal.l.verma@intel.com>
Link: https://lore.kernel.org/r/20221116-acpi_hmat_fix-v2-1-3712569be691@intel.com


Signed-off-by: default avatarDan Williams <dan.j.williams@intel.com>
parent 094226ad

drivers/acpi/numa/hmat.c

+0 −1
Original line number Original line Diff line number Diff line
@@ -644,7 +644,6 @@ static void hmat_register_target_initiators(struct memory_target *target) 
	/* Access 1 ignores Generic Initiators */

	/* Access 1 ignores Generic Initiators */

	bitmap_zero(p_nodes, MAX_NUMNODES);

	bitmap_zero(p_nodes, MAX_NUMNODES);

	list_sort(p_nodes, &initiators, initiator_cmp);

	list_sort(p_nodes, &initiators, initiator_cmp);

	best = 0;

	for (i = WRITE_LATENCY; i <= READ_BANDWIDTH; i++) {

	for (i = WRITE_LATENCY; i <= READ_BANDWIDTH; i++) {

		loc = localities_types[i];

		loc = localities_types[i];

		if (!loc)

		if (!loc)