search

Seeed-Studio / Seeed_Arduino_rpcUnified

RTL8720DN wireless abstraction layer using the eRPC protocol.
MIT License
19 stars 10 forks source link

Too slow get_ssl_receive() #7

Closed matsujirushi closed 3 years ago

matsujirushi commented 3 years ago

get_ssl_receive() takes 50milliseconds.

image image

This issue to lower application performance because this function is called from WiFiClientSecure::available() repeatly. Could you be faster?

matsujirushi commented 3 years ago

I changed this library and RTL firmware. Please check and merge.

Timing: image

Modified code: https://github.com/matsujirushi/Seeed_Arduino_rpcUnified/tree/matsujirushi-patch2 https://github.com/matsujirushi/seeed-ambd-firmware/tree/matsujirushi-patch2

Pillar1989 commented 3 years ago

@matsujirushi Thank you so much.

Pillar1989 commented 3 years ago

@matsujirushi Do you think it is necessary to include RTL FW version checking at the beginning of the code?

matsujirushi commented 3 years ago

Hi @Pillar1989 , Yes. Developer friendly :-)

matsujirushi commented 3 years ago

FYI Next idea for performance up. https://twitter.com/matsujirushi12/status/1341359957619314689

matsujirushi commented 3 years ago

More updated.

Timing: image

Modified code: https://github.com/matsujirushi/Seeed_Arduino_rpcUnified/tree/matsujirushi-patch3 https://github.com/matsujirushi/seeed-ambd-firmware/tree/matsujirushi-patch3

Pillar1989 commented 3 years ago

@matsujirushi Can this issue be closed?

matsujirushi commented 3 years ago

Hi @Pillar1989 , Yes, if you are merged the patch3.