Commit f0e0b4d1 authored by Srinivas Pandruvada's avatar Srinivas Pandruvada Committed by Andy Shevchenko
Browse files

tools/power/x86/intel-speed-select: Warn for invalid package id



When CPU is offline, we can't get package id. So print error for this
and don't use output.

Signed-off-by: default avatarSrinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Signed-off-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
parent ced2f530
Loading
Loading
Loading
Loading
+8 −0
Original line number Diff line number Diff line
@@ -261,6 +261,10 @@ static void for_each_online_package_in_set(void (*callback)(int, void *, void *,
		if (die_id < 0)
			die_id = 0;
		pkg_id = get_physical_package_id(i);
		if (pkg_id < 0) {
			fprintf(stderr, "Failed to get package id, CPU %d may be offline\n", i);
			continue;
		}
		/* Create an unique id for package, die combination to store */
		pkg_id = (MAX_PACKAGE_COUNT * pkg_id + die_id);

@@ -362,6 +366,10 @@ static void set_cpu_present_cpu_mask(void)
				die_id = 0;

			pkg_id = get_physical_package_id(i);
			if (pkg_id < 0) {
				fprintf(stderr, "Failed to get package id, CPU %d may be offline\n", i);
				continue;
			}
			if (pkg_id < MAX_PACKAGE_COUNT &&
			    die_id < MAX_DIE_PER_PACKAGE) {
				int core_id = get_physical_core_id(i);