ChibiOS Free Embedded RTOS

Hi All,
I have been testing the speed on hal_serial_usb, testing with different size for SERIAL_USB_BUFFERS_SIZE, SERIAL_USB_BUFFERS_NUMBER and the size of buffer which goes in chnWrite(&SDU2, (const uint8_t *)&test_buff, TEST_BUFFER_SIZE).
I realized what matters most is SERIAL_USB_BUFFERS_SIZE. I was able to squeeze 19MB/s from running the driver alone on STM32F723.

But there was one more parameter which changed the speed substantially to 29MB/s. That was increasing the size check in obqWriteTimeout() [hal_buffers.c]

Do you guys think this parameter should be exposed so that it can be fine-tuned?

Thanks,
AQ

Hi,

Thanks for testing, probably yes, it should be exposed, moving this topic in "change request".

Giovanni

Hi,

A new setting has been added: BUFFERS_CHUNKS_SIZE. It applies to both obqWriteTimeout() and ibqReadTimeout(). The default is still 64.

Giovanni

Thanks Giovanni,
I do have a concern though. Should this change be pulled out to the configuration files? That way, the application would be able to fine tune this parameter.

Thanks,
AQ

Hi,

You can put that definition in halconf.h, uncommon settings are not usually present in configuration files in order to keep those not too complex. You may recommend different defaults after your testing.

Giovanni