RFR[8237480]: Add micros for DatagramSocket send/receive

[Patrick Concannon]

Hi Claes,


Thanks for the feedback.

I've reduced the parameters to 5 i.e. "128", "512", "2048", "8192", "32768"

I've also removed the main method to conform with the layout of the other benchmark tests.

You can find the changes in the updated webrev below:
http://cr.openjdk.java.net/~pconcannon/8237480/webrevs/webrev.02/

Kind regards,

Patrick


On 12/02/2020 15:22, Claes Redestad wrote:
> Hi,
>
> looks reasonable to me.
>
> To keep default execution times at a manageable level, I'd recommend
> excluding some of the 9 values for the size parameter (3-5 values seem
> reasonable).
>
> /Claes
>
> On 2020-02-12 16:04, Patrick Concannon wrote:
>> Hi,
>>
>>
>> Could someone please review my webrev for JDK-8237480 'Add micros for 
>> DatagramSocket send/receive' ?
>>
>> This change adds some benchmarks for the DatagramSocket::send and 
>> DatagramSocket::receive methods.
>>
>>
>> bug: https://bugs.openjdk.java.net/browse/JDK-8237480
>>
>> webrev: 
>> http://cr.openjdk.java.net/~pconcannon/8237480/webrevs/webrev.01/
>>
>>
>> Kind regards,
>>
>> Patrick
>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/net-dev/attachments/20200213/a49ade83/attachment.htm>