[Mesa-dev] [PATCH mesa 1/2] Add processor topology calculation implementation for Darwin/OSX targets.

Jeremy Sequoia jeremyhu at apple.com
Thu Mar 15 00:07:55 UTC 2018



Sent from my iPhone...

> On Mar 14, 2018, at 16:41, Matt Turner <mattst88 at gmail.com> wrote:
> 
> Subject should have a swr prefix or similar.
> 
> On Wed, Mar 14, 2018 at 4:19 PM, Jeremy Huddleston Sequoia
> <jeremyhu at apple.com> wrote:
>> From: Apple SWE <jeremyhu at apple.com>
> 
> Explain?

I didn’t author it.  Author is an Apple Employee in SWE who does not want attribution.

> 
>> 
>> The implementation for bootstrapping SWR on Darwin targets is based on the Linux version.
>> Instead of reading the output of /proc/cpuinfo, sysctlbyname is used to determine the
>> physical identifiers, processor identifiers, core counts and thread-processor affinities.
>> 
>> With this patch, it is possible to use SWR as an alternate renderer on OSX to softpipe and
>> llvmpipe.
> 
> All of these look like they're too long to fix in 80 columns in git show.
> 
>> Reviewed-by: Jeremy Huddleston Sequoia <jeremyhu at apple.com>
> 
> I'm guessing you're just pushing code someone else wrote...
> 
>> Signed-off-by: Jeremy Huddleston Sequoia <jeremyhu at apple.com>
> 
> We don't  have a DCO, so Signed-off-by has no meaning. Worse, in the
> case we added a DCO in the future, all of the stupid Signed-off-by's
> people have been cargo culting for years would confuse everything.
> Please stop doing it.

Ok.  I’ll remove it.

> 
> Please don't push code to maintained drivers without going through the
> mailing list. I feel like I shouldn't have to say that.

In the past there hasn’t been much care about code in __APPLE__, so I figured it want that big of a deal, but if you are interested in reviewing these changes, that’s great to hear.


More information about the mesa-dev mailing list