mhi: pci_generic: Set irq moderation value to 1ms for hw channels
authorLoic Poulain <loic.poulain@linaro.org>
Mon, 4 Jan 2021 16:14:59 +0000 (17:14 +0100)
committerManivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Mon, 4 Jan 2021 16:20:25 +0000 (21:50 +0530)
MHI hardware channels are usually the hardware accelerated data path
e.g. IP packets path for modems. This path needs to be optimized for
low latency and high throughput. After several tests on FN980m SDX55
based modem, it seems 1ms is a good default irq_moderation value:

- It allows to reach the maximum download throughput
- It introduces limited latency (5ms is too high)
- It prevents interrupt flooding

Signed-off-by: Loic Poulain <loic.poulain@linaro.org>
Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
drivers/bus/mhi/pci_generic.c

index 5104084d9c082e4ec580d79b95b2f274d152968a..c13de0f2053efd55baf138e037fe4e7659396a3c 100644 (file)
@@ -130,7 +130,7 @@ struct mhi_pci_dev_info {
 #define MHI_EVENT_CONFIG_HW_DATA(ev_ring, ch_num) \
        {                                       \
                .num_elements = 256,            \
-               .irq_moderation_ms = 5,         \
+               .irq_moderation_ms = 1,         \
                .irq = (ev_ring) + 1,           \
                .priority = 1,                  \
                .mode = MHI_DB_BRST_DISABLE,    \