On Mon, Jun 18, 2018 at 12:14:09PM +0800, Waiman Long wrote: > +#ifdef CONFIG_DEBUG_KERNEL > +static inline void debug_print_domains(cpumask_var_t *doms, int ndoms) > +{ > + int i; > + char buf[200]; > + char *ptr, *end = buf + sizeof(buf) - 1; > + > + for (i = 0, ptr = buf, *end = '\0'; i < ndoms; i++) > + ptr += snprintf(ptr, end - ptr, "dom%d=%*pbl ", i, > + cpumask_pr_args(doms[i])); > + > + pr_debug("Generated %d domains: %s\n", ndoms, buf); > +} Why not use pr_cont() and do away with that static buffer? -- To unsubscribe from this list: send the line "unsubscribe cgroups" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html