Re: [PATCH 0/4] x86: add cpus_scnprintf function v2

From: Paul Jackson
Date: Thu Apr 10 2008 - 11:30:25 EST


Ingo, responding to Paul, three days ago:
> i dont mind the old patch either (which did an ugly temporary
> allocation), if it keeps the ABI.

But this v2 and earlier patch versions broke the kernel-user interface,
incompatibly, for about a dozen different /sys and /proc files, and
without even stating so very clearly in the patch commentary. We'd
have to be rather desperate before I'd agree to that.

Mike's latest v3 version of this cpus_scnprintf patchset has resolved
these incompatibilities, by adding new *list files in /sys and /proc,
and new *list line items in the /proc/<pid>/status files, rather than
breaking existing files and lines.

That is much better. Better to add files and lines, than to change the
format of existing ones.

--
I won't rest till it's the best ...
Programmer, Linux Scalability
Paul Jackson <pj@xxxxxxx> 1.940.382.4214
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/