Messages in this thread | | | Date | Fri, 28 Feb 2020 10:40:34 +0000 | From | Sudeep Holla <> | Subject | Re: [PATCH] cpu-topology: Fix the potential data corruption |
| |
On Fri, Feb 28, 2020 at 04:35:45PM +0800, Zeng Tao wrote: > Currently there are only 10 bytes to store the cpu-topology info. > That is: > snprintf(buffer, 10, "cluster%d",i); > snprintf(buffer, 10, "thread%d",i); > snprintf(buffer, 10, "core%d",i); > > In the boundary test, if the cluster number exceeds 100, there will be a
I don't understand you mention of 100 in particular above. I can see issue if there are cluster with more than 2-digit id. Though highly unlikely for now, but I don't have objection to the patch.
-- Regards, Sudeep
| |