Optimization improvements

[ioef]

read the object reference before the loop and not every-time inside it.

---

This change is

[tomato42]

patch's ok, thanks.

could you mention, here in comment, what gains it brings?

[ioef]

Hi Hubert.

Well in my local file i have applied this approach in more than one place
in the code and i have achieved significant improvements in the througput
speed of the AES128GCM. I am currently attaching the file that you may
have a look at and maybe also apply in your main track.

Please have a look at the below results

Test 1: aes128gcm python: 1MB exchanged at rate of 201525 bytes/sec
Used Ciphersuite: TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256

Test 2: aes128 python: 1MB exchanged at rate of 312258 bytes/sec
Used Ciphersuite: TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256

Test 3: aes256 python: 1MB exchanged at rate of 238730 bytes/sec
Used Ciphersuite: TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA

Test 4: rc4 python: 1MB exchanged at rate of 2010652 bytes/sec
Used Ciphersuite: TLS_ECDHE_RSA_WITH_RC4_128_SHA

Test 5: chacha20-poly1305 python: 1MB exchanged at rate of 283674 bytes/sec
Used Ciphersuite: TLS_ECDHE_RSA_WITH_CHACHA20_POLY1305

Test 6: speck128 python: 1MB exchanged at rate of 318835 bytes/sec
Used Ciphersuite: TLS_ECDHE_RSA_WITH_SPECK_128_CBC_SHA256

Test 7: speck128gcm python: 1MB exchanged at rate of 202298 bytes/sec
Used Ciphersuite: TLS_ECDHE_RSA_WITH_SPECK_128_GCM_SHA256

Test 8: speck192gcm python: 1MB exchanged at rate of 199748 bytes/sec
Used Ciphersuite: TLS_ECDHE_RSA_WITH_SPECK_192_GCM_SHA256

Test succeeded, 8 good

2016-03-20 11:11 GMT+02:00 Hubert Kario notifications@github.com:

patch's ok, thanks.

could you mention, here in comment, what gains it brings?

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#98 (comment)

[tomato42]

I meant for this patch only, baseline and with it...

[ioef]

Would you like me to apply the updates in the aesgcm file according to the
file i have provided?

Thank you.

2016-03-20 13:33 GMT+02:00 Hubert Kario notifications@github.com:

I meant for this patch only, baseline and with it...

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#98 (comment)

[tomato42]

github does not attach any patches, so I have not seen that file.

I'm still asking for a comparison between current master branch and this patch only

[ioef]

I have just provided you the file updates through your email. Thank you.