[quake3-bugzilla] [Bug 4977] x86 qvm improvements

bugzilla-daemon at icculus.org bugzilla-daemon at icculus.org
Wed May 11 11:49:44 EDT 2011


https://bugzilla.icculus.org/show_bug.cgi?id=4977

Thilo Schulz <arny at ats.s.bawue.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #2703|0                           |1
        is obsolete|                            |

--- Comment #14 from Thilo Schulz <arny at ats.s.bawue.de> 2011-05-11 11:49:41 EDT ---
Created attachment 2704
  --> https://bugzilla.icculus.org/attachment.cgi?id=2704
x86 qvm improvements with jump checks

(In reply to comment #13)
> I'm not sure about adding/changing opstack/edi stuff because of
> EmitCommand/etc. which may require more than just range checks

We do need this, however. If you won't do this I can take a look at this in the
next couple of days. Just tell me whether you're going to do it or not because
I don't want to invent the wheel twice.

> For MAGIC_V2 it fixes all unsafe places and utilizes all optimizations, but not
> for VQ3 so I'm disabled const-optimizations for that case, old stable mods will
> work fine

Right. I've tested your changes and they appear to be sound. I still had to
make a little modification to the ErrJump routine, though, or it will still
crash, because it resumes execution in the VM after Com_Error and Sys_Error
were recursively entered.

-- 
Configure bugmail: https://bugzilla.icculus.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.


More information about the quake3-bugzilla mailing list