From mboxrd@z Thu Jan 1 00:00:00 1970 From: James Simmons Date: Mon, 16 Apr 2018 00:10:04 -0400 Subject: [lustre-devel] [PATCH 22/25] staging: lustre: libcfs: update debug messages in CPT code In-Reply-To: <1523851807-16573-1-git-send-email-jsimmons@infradead.org> References: <1523851807-16573-1-git-send-email-jsimmons@infradead.org> Message-ID: <1523851807-16573-23-git-send-email-jsimmons@infradead.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Greg Kroah-Hartman , devel@driverdev.osuosl.org, Andreas Dilger , Oleg Drokin , NeilBrown Cc: Dmitry Eremin , Linux Kernel Mailing List , Lustre Development List From: Dmitry Eremin Update the debug messages for the CPT table creation code. Place the passed in string in quotes to make it clear what it is. Captialize cpu in the debug strings. Signed-off-by: Dmitry Eremin Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-8703 Reviewed-on: https://review.whamcloud.com/23306 Reviewed-by: James Simmons Reviewed-by: Andreas Dilger Reviewed-by: Patrick Farrell Reviewed-by: Olaf Weber Reviewed-by: Oleg Drokin Signed-off-by: James Simmons --- drivers/staging/lustre/lnet/libcfs/linux/linux-cpu.c | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/staging/lustre/lnet/libcfs/linux/linux-cpu.c b/drivers/staging/lustre/lnet/libcfs/linux/linux-cpu.c index 28b2acb..a08816a 100644 --- a/drivers/staging/lustre/lnet/libcfs/linux/linux-cpu.c +++ b/drivers/staging/lustre/lnet/libcfs/linux/linux-cpu.c @@ -466,7 +466,7 @@ void cfs_cpt_unset_cpu(struct cfs_cpt_table *cptab, int cpt, int cpu) } else if (cpt != cptab->ctb_cpu2cpt[cpu]) { CDEBUG(D_INFO, - "CPU %d is not in cpu-partition %d\n", cpu, cpt); + "CPU %d is not in CPU partition %d\n", cpu, cpt); return; } @@ -910,14 +910,14 @@ static struct cfs_cpt_table *cfs_cpt_table_create_pattern(char *pattern) if (!ncpt || (node && ncpt > num_online_nodes()) || (!node && ncpt > num_online_cpus())) { - CERROR("Invalid pattern %s, or too many partitions %d\n", + CERROR("Invalid pattern '%s', or too many partitions %d\n", pattern, ncpt); return NULL; } cptab = cfs_cpt_table_alloc(ncpt); if (!cptab) { - CERROR("Failed to allocate cpu partition table\n"); + CERROR("Failed to allocate CPU partition table\n"); return NULL; } @@ -948,11 +948,11 @@ static struct cfs_cpt_table *cfs_cpt_table_create_pattern(char *pattern) if (!bracket) { if (*str) { - CERROR("Invalid pattern %s\n", str); + CERROR("Invalid pattern '%s'\n", str); goto failed; } if (c != ncpt) { - CERROR("expect %d partitions but found %d\n", + CERROR("Expect %d partitions but found %d\n", ncpt, c); goto failed; } @@ -960,7 +960,7 @@ static struct cfs_cpt_table *cfs_cpt_table_create_pattern(char *pattern) } if (sscanf(str, "%d%n", &cpt, &n) < 1) { - CERROR("Invalid cpu pattern %s\n", str); + CERROR("Invalid CPU pattern '%s'\n", str); goto failed; } @@ -977,20 +977,20 @@ static struct cfs_cpt_table *cfs_cpt_table_create_pattern(char *pattern) str = strim(str + n); if (str != bracket) { - CERROR("Invalid pattern %s\n", str); + CERROR("Invalid pattern '%s'\n", str); goto failed; } bracket = strchr(str, ']'); if (!bracket) { - CERROR("Missing right bracket for partition %d, %s\n", + CERROR("Missing right bracket for partition %d in '%s'\n", cpt, str); goto failed; } if (cfs_expr_list_parse(str, (bracket - str) + 1, 0, high, &el)) { - CERROR("Can't parse number range: %s\n", str); + CERROR("Can't parse number range in '%s'\n", str); goto failed; } -- 1.8.3.1