[patch] update the jamvm patch for at least 2.3.10

[Andrew Hughes]

----- Original Message -----
> 2013-07-08 10:21, Andrew Hughes skrev:
> > ----- Original Message -----
> >> In the 2.3.10 release, the jamvm patch needs an update. Didn't check the
> >> other
> >> releases.  Updated patch attached.
> >>
> >>   Matthias
> >>
> >>
> > 
> > Any idea why these changes are needed?
> > 
> 
> The remove-sun.misc.Perf-debug-code.patch was needed for use of OpenJDK 7 & 8
> in combination with JamVM prior to 2012-12-28.
> The more recent JamVM versions have implemented a workaround:
> Robert's git commit explain:
> 
> http://git.berlios.de/cgi-bin/cgit.cgi/jamvm/commit/?id=e8dbdac844c3837f3f697e34dcf7b7c6506158ce
> "The OpenJDK 7 and OpenJDK 8 sources contain various calls to sun.misc.Perf
> for performance profiling of class-loading.
> Whether this is left-over debug code or is intentional is not clear.
> 
> However, this causes JamVM to fail as it doesn't implement the native methods
> for sun.misc.Perf.
> IcedTea 7 contains a patch to remove these calls to enable JamVM to run.
> But this doesn't help when running with "vanilla" OpenJDK 8, which I'm using
> for the JSR 292 work (no IcedTea version for OpenJDK 8 exists, that I'm
> aware off).
> 
> Hacking the sources to remove these calls everytime I get a new version is a
> huge pain-in-the-ass.
> So this is a dummy implementation, sufficient to get the sources to run
> without hacking them.
> Signed-off-by: Robert Lougher <rob at jamvm.org.uk>"
> 
> 
> 2013-07-01 15:22, Xerxes Rånby skrev:> 2013-07-01 14:57, Matthias Klose
> skrev:
> >> In the 2.3.10 release, the jamvm patch needs an update. Didn't check the
> >> other releases.  Updated patch attached.
> >>
> >> Matthias
> >>
> >
> > Looks OK,
> >
> > We have one option and that is to update JamVM to match the version used in
> > 2.4.*,
> > http://icedtea.classpath.org/hg/release/icedtea7-2.4/rev/933d082ec889
> > if we update JamVM then we may drop this
> > remove-sun.misc.Perf-debug-code.patch for the 2.3.10 branch as well.
> >
> 
> Cheers
> Xerxes
> 

But we're talking about JamVM as it already stood on the 2.3.x branch. It hasn't
changed and we don't change it between release branches.

So there must be something in the security patches that caused this.
-- 
Andrew :)

Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)

PGP Key: 248BDC07 (https://keys.indymedia.org/)
Fingerprint = EC5A 1F5E C0AD 1D15 8F1F  8F91 3B96 A578 248B DC07