login
Header Space

 
 

Re: [PATCH 2/2] x86: modify show_shared_cpu_map in intel_cacheinfo

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Mike Travis <travis@...>
Cc: <bert.wesarg@...>, <mingo@...>, <linux-kernel@...>
Date: Monday, March 31, 2008 - 1:56 pm

>>  I did an informal survey and you are right, the majority of references do use

My recollection is that I added cpulist_scnprintf later, yes.
Looking at my email archives, I see the mask versions mentioned
starting Feb 2004, and the list versions starting Aug 2004.

My rule of thumb has been to use the mask style (00000000,0000ffff)
for lower level interfaces, and the list style (0-15) for higher level
interfaces.

For long lists, the list style is easier for humans to read, but for
one word masks, the mask style can be easier to read for -some-
purposes and are more commonly used.

If you throw enough user level software at them, the lists are no more
or less difficult to form or parse.  Hand coded C parsers are probably
easier to write for the mask style, and might be closer to what low
level (closer to the hardware) programmers expect.

Certainly, a particular interface should not change once it goes public.

Once picked for a new interface, I don't recall ever seeing any significant
controversy over which one was picked.  So another of my rules of thumb
might apply here -- coders choice.  He who writes the code gets to make
the open choices.

-- 
                  I won't rest till it's the best ...
                  Programmer, Linux Scalability
                  Paul Jackson <pj@sgi.com> 1.940.382.4214
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: [PATCH 2/2] x86: modify show_shared_cpu_map in intel_cac..., Paul Jackson, (Mon Mar 31, 1:56 pm)
speck-geostationary