I think he's referring to the actual commit, which I find weird too.
Regards,
Aleksey Bragin
On 05.04.2013 2:42, Kamil Hornicek wrote:
"LOL", I'm not "trying" to
"fix" "anything" - disabling the whole SSE
support is on par with disabling just the check. You fix the kernel
and you better do it soon.
Dne 4.4.2013 21:58, Timo Kreuzer napsal(a):
>
> LOL, we have a bug in the kernel and you try to "fix" this in MESA?
>
>
> Am 04.04.2013 14:33, schrieb Kamil Hornicek:
>> If I recall correctly there's a check whether the OS can handle
>> masked/unmasked sse exceptions. It causes trouble even in Windows if
>> the app using Mesa has it's own exception handlers installed IIRC. SSE
>> works just fine. Just disable the (useless) _mesa_check_os_sse_suppor
>> stuff (ReactOS supports this, no need to do the check) or find a way
>> to stop the exception from propagating.
>>
>> Regards,
>> Kamil
>>
>> Dne 4.4.2013 12:34, Jérôme Gardou napsal(a):
>>> It causes some kernel mode exception. The code deliberately throws an
>>> SSE exception to see if the OS supports them. The trap handler
>>> considers
>>> this as a k-mode exception and bug checks.
>>>
>>> See
http://jira.reactos.org/browse/CORE-6776
>>>
>>> Timo Kreuzer a écrit :
>>>>
>>>> What exactly does it cause? And shouldn't we rather fix that,
>>>> instead of
>>>> disabling optimizations? mesa is already slow enough.
>>>>
>>>> Am 03.04.2013 14:02, schrieb jgardou(a)svn.reactos.org:
>>>>> Author: jgardou
>>>>> Date: Wed Apr 3 12:02:58 2013
>>>>> New Revision: 58660
>>>>>
>>>>> URL:
http://svn.reactos.org/svn/reactos?rev=58660&view=rev
>>>>> Log:
>>>>> [MESA32]
>>>>> * Disable SSE optimizations, as they only cause mayhem.
>>>>>
>>>>> Modified:
>>>>> trunk/reactos/dll/opengl/mesa/src/mesa/CMakeLists.txt
>>>>>
>>>>> Modified: trunk/reactos/dll/opengl/mesa/src/mesa/CMakeLists.txt
>>>>> URL:
>>>>>
http://svn.reactos.org/svn/reactos/trunk/reactos/dll/opengl/mesa/src/mesa/C…
>>>>>
>>>>>
>>>>>
>>>>>
>>>>>
==============================================================================
>>>>>
>>>>>
>>>>>
>>>>>
>>>>> --- trunk/reactos/dll/opengl/mesa/src/mesa/CMakeLists.txt
>>>>> [iso-8859-1]
>>>>> (original)
>>>>> +++ trunk/reactos/dll/opengl/mesa/src/mesa/CMakeLists.txt
>>>>> [iso-8859-1]
>>>>> Wed Apr 3 12:02:58 2013
>>>>> @@ -33,17 +33,18 @@
>>>>> x86/3dnow_xform3.S
>>>>> x86/3dnow_xform4.S
>>>>> x86/3dnow_normal.S
>>>>> - x86/sse_xform1.S
>>>>> - x86/sse_xform2.S
>>>>> - x86/sse_xform3.S
>>>>> - x86/sse_xform4.S
>>>>> - x86/sse_normal.S
>>>>> + # x86/sse_xform1.S
>>>>> + # x86/sse_xform2.S
>>>>> + # x86/sse_xform3.S
>>>>> + # x86/sse_xform4.S
>>>>> + # x86/sse_normal.S
>>>>> x86/read_rgba_span_x86.S)
>>>>> add_definitions(
>>>>> -DUSE_X86_ASM
>>>>> -DUSE_MMX_ASM
>>>>> -DUSE_3DNOW_ASM
>>>>> - -DUSE_SSE_ASM)
>>>>> + # -DUSE_SSE_ASM
>>>>> + )
>>>>> endif()
>>>>> list(APPEND SOURCE
>>>>>
>>>>>
>>>>>
>>>>