On 24/02/15 22:48, Jonathan Gray wrote:
> On Tue, Feb 24, 2015 at 04:53:03PM +0000, Emil Velikov wrote:
>> On 22 February 2015 at 08:19, Jonathan Gray <j...@jsg.id.au> wrote:
>>> The length argument passed to sysctl was the size of the pointer
>>> not the type. The result of this is sysctl calls would fail on
>>> 32 bit BSD/Mac OS X.
>>>
>>> Additionally the wrong pointer was passed as an argument to store
>>> the result of the sysctl call.
>>>
>>> Cc: "10.4, 10.5" <mesa-sta...@lists.freedesktop.org>
>>> Signed-off-by: Jonathan Gray <j...@jsg.id.au>
>> Seems like my attempt was enough but not quite there yet.
>> Thanks for fixing my goof-up.
>>
>> Reviewed-by: Emil Velikov <emil.l.veli...@gmail.com>
>>
>> I'll push this in a couple of days unless there are any objections.
>>
>> Cheers
>> Emil
>
> It should be possible to use the sysconf path in more places
> as well. Classic swrast for example doesn't use sysctl at all.
>
> OpenBSD/FreeBSD have _SC_PHYS_PAGES/_SC_PAGE_SIZE
> NetBSD/Mac OS X don't document _SC_PAGE_SIZE but do document
> _SC_PAGESIZE, though I wouldn't be surprised if they
> had _SC_PAGE_SIZE in their headers.
>
If you feel like unifying some code that would be appreciated.
Although, I have a greater request from you - can you look at removing
the symlinks in src/mesa/drivers/dri/r{adeon,200} ? Pretty please :-)
Restructuring the common bits into one place (radeon_common perhaps ?)
will allow one to drop the links, and this will also cut down a fair
hunk of the classic dri module binary.
Btw did you have the chance to try 10.5 with a simple wrapper, similar
to xorg-server ? I believe that most concerns should be sorted out now -
no python, lex, etc... dependency.
Cheers,
Emil
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
