RFR 8179602: Fix for JDK-8165437 is broken on 32-bit Linux
    Langer, Christoph 
    christoph.langer at sap.com
       
    Thu May  4 08:57:15 UTC 2017
    
    
  
Hi Vyom,
the fix looks good and seems straightforward to resolve the reported issue.
Small item: I spotted an extra space in net_util_md.h, line 75, between "jlong" and "nanoTimeStamp" which could be removed.
Reviewed.
Best regards
Christoph
> -----Original Message-----
> From: net-dev [mailto:net-dev-bounces at openjdk.java.net] On Behalf Of
> Vyom Tewari
> Sent: Donnerstag, 4. Mai 2017 10:50
> To: net-dev <net-dev at openjdk.java.net>
> Subject: RFR 8179602: Fix for JDK-8165437 is broken on 32-bit Linux
> 
> Hi All,
> 
> Please  review the below change.
> 
> Webrev:
> http://cr.openjdk.java.net/~vtewari/8179602/webrev0.0/index.html
> 
> Bugid: https://bugs.openjdk.java.net/browse/JDK-8179602
> 
> This issue is because of side effect of "JDK-8165437" where we are using
> "JVM_NanoTime" which returns a 64 bit jlong and return value was getting
> assigned to long type. On 32 bit OS long is 4 byte, which leads to
> integer overflow.
> 
> Our internal test JPRT is  still running.
> 
> Thanks,
> 
> Vyom
    
    
More information about the net-dev
mailing list