harmony-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Boris Kuznetsov (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HARMONY-4039) [classlib][luni] Socket implementation is too slow
Date Wed, 13 Jun 2007 09:19:28 GMT

    [ https://issues.apache.org/jira/browse/HARMONY-4039?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12504171
] 

Boris Kuznetsov commented on HARMONY-4039:
------------------------------------------

Tim, I've just tried the patch with IBM VM. Results are similar to DRLVM and JRockit:
For the attached test: estimatedTime 15625
and after patch applying: estimatedTime 16

The idea of this path is to use zero timeout for hysock_select_read() in the availableStreamImpl()
But hysock_select_read() is called with zero timeout from pollSelectRead() also (see socket.c),
and windows implementation of hysock_select_read() resets timeout to 100 ms. I don't know
the reason of such implementation, so I used existent parameter 'accept' to distinguish calls
from availableStreamImpl() and pollSelectRead(). (Note, this parameter was not used before.)
Possibly it should be renamed.


> [classlib][luni] Socket implementation is too slow
> --------------------------------------------------
>
>                 Key: HARMONY-4039
>                 URL: https://issues.apache.org/jira/browse/HARMONY-4039
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>            Reporter: Boris Kuznetsov
>            Assignee: Tim Ellison
>         Attachments: patch.txt, test.java
>
>
> I noticed that Harmony classlib works too slowly with MySQL (both on DRLVM and Jrockit).
> The reason is the socket implementation: availableStreamImpl uses 1 ms timeout for hysock_select_read().
> It leads to 0.015 sec wait on select call for every available().
> Here are the test which demonstrate the problem and proposed patch.
> harmony-jre-r533500\bin\java test
> estimatedTime 15625
> After patch applying:
> java test
> estimatedTime 10

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message