debuggers.hg
changeset 22885:6d6465868420
xl: fix xl cpupool-list <poolid>
The help screen of xl cpupool-list promises to allow a CPU pool to
be named on the command line, which will then be listed only.
Probably caused by a "DeMorgan brain twist" this specific CPU pool
is _omitted_ instead. The patch fixes this, so single CPU pools
can be explicitly listed again.
Signed-off-by: Andre Przywara <andre.przywara@amd.com>
Acked-by: Juergen Gross <juergen.gross@ts.fujitsu.com>
Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
Committed-by: Ian Jackson <ian.jackson@eu.citrix.com>
The help screen of xl cpupool-list promises to allow a CPU pool to
be named on the command line, which will then be listed only.
Probably caused by a "DeMorgan brain twist" this specific CPU pool
is _omitted_ instead. The patch fixes this, so single CPU pools
can be explicitly listed again.
Signed-off-by: Andre Przywara <andre.przywara@amd.com>
Acked-by: Juergen Gross <juergen.gross@ts.fujitsu.com>
Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
Committed-by: Ian Jackson <ian.jackson@eu.citrix.com>
| author | Andre Przywara <andre.przywara@amd.com> |
|---|---|
| date | Fri Jan 28 17:45:24 2011 +0000 (2011-01-28) |
| parents | aab67c1c6b87 |
| children | 3e928084336f |
| files | tools/libxl/xl_cmdimpl.c |
line diff
1.1 --- a/tools/libxl/xl_cmdimpl.c Fri Jan 28 17:43:50 2011 +0000 1.2 +++ b/tools/libxl/xl_cmdimpl.c Fri Jan 28 17:45:24 2011 +0000 1.3 @@ -5446,7 +5446,7 @@ int main_cpupoollist(int argc, char **ar 1.4 printf("CPUs Sched Active Domain count\n"); 1.5 1.6 for (p = 0; p < n_pools; p++) { 1.7 - if (!ret && (!pool || (poolinfo[p].poolid != poolid))) { 1.8 + if (!ret && (!pool || (poolinfo[p].poolid == poolid))) { 1.9 name = libxl_cpupoolid_to_name(&ctx, poolinfo[p].poolid); 1.10 if (!name) { 1.11 fprintf(stderr, "error getting cpupool info\n");