* [dpdk-dev] [PATCH] net/ena/base: fix doorbell evaluation for the LLQ case
@ 2020-08-12 16:37 Artur Rojek
2020-08-13 18:07 ` Marcin Wojtas
0 siblings, 1 reply; 3+ messages in thread
From: Artur Rojek @ 2020-08-12 16:37 UTC (permalink / raw)
To: Marcin Wojtas, Michal Krawczyk, Guy Tzalik, Igor Chauskin
Cc: dev, stable, Artur Rojek
From: Michal Krawczyk <mk@semihalf.com>
This patch adds a missing LLQ-related check in the
ena_com_is_doorbell_needed() routine, which is relevant for the feature
supported by the next generation HW of the ENA.
Fixes: b2b02edeb0d6 ("net/ena/base: upgrade HAL for new HW features")
CC: stable@dpdk.org
Signed-off-by: Michal Krawczyk <mk@semihalf.com>
[Extracted from a bigger patch]
Signed-off-by: Artur Rojek <ar@semihalf.com>
---
drivers/net/ena/base/ena_eth_com.h | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/net/ena/base/ena_eth_com.h b/drivers/net/ena/base/ena_eth_com.h
index e37b642d4..3d66237b8 100644
--- a/drivers/net/ena/base/ena_eth_com.h
+++ b/drivers/net/ena/base/ena_eth_com.h
@@ -133,7 +133,8 @@ static inline bool ena_com_is_doorbell_needed(struct ena_com_io_sq *io_sq,
llq_info = &io_sq->llq_info;
num_descs = ena_tx_ctx->num_bufs;
- if (unlikely(ena_com_meta_desc_changed(io_sq, ena_tx_ctx)))
+ if (llq_info->disable_meta_caching ||
+ unlikely(ena_com_meta_desc_changed(io_sq, ena_tx_ctx)))
++num_descs;
if (num_descs > llq_info->descs_num_before_header) {
--
2.28.0
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [dpdk-dev] [PATCH] net/ena/base: fix doorbell evaluation for the LLQ case
2020-08-12 16:37 [dpdk-dev] [PATCH] net/ena/base: fix doorbell evaluation for the LLQ case Artur Rojek
@ 2020-08-13 18:07 ` Marcin Wojtas
2020-08-18 15:26 ` [dpdk-dev] [dpdk-stable] " Ferruh Yigit
0 siblings, 1 reply; 3+ messages in thread
From: Marcin Wojtas @ 2020-08-13 18:07 UTC (permalink / raw)
To: Artur Rojek; +Cc: Michal Krawczyk, Guy Tzalik, Igor Chauskin, dev, stable
Hi,
śr., 12 sie 2020 o 18:37 Artur Rojek <ar@semihalf.com> napisał(a):
>
> From: Michal Krawczyk <mk@semihalf.com>
>
> This patch adds a missing LLQ-related check in the
> ena_com_is_doorbell_needed() routine, which is relevant for the feature
> supported by the next generation HW of the ENA.
>
> Fixes: b2b02edeb0d6 ("net/ena/base: upgrade HAL for new HW features")
> CC: stable@dpdk.org
>
> Signed-off-by: Michal Krawczyk <mk@semihalf.com>
> [Extracted from a bigger patch]
> Signed-off-by: Artur Rojek <ar@semihalf.com>
Reviewed-by: Marcin Wojtas <mw@semihalf.com>
> ---
> drivers/net/ena/base/ena_eth_com.h | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/ena/base/ena_eth_com.h b/drivers/net/ena/base/ena_eth_com.h
> index e37b642d4..3d66237b8 100644
> --- a/drivers/net/ena/base/ena_eth_com.h
> +++ b/drivers/net/ena/base/ena_eth_com.h
> @@ -133,7 +133,8 @@ static inline bool ena_com_is_doorbell_needed(struct ena_com_io_sq *io_sq,
> llq_info = &io_sq->llq_info;
> num_descs = ena_tx_ctx->num_bufs;
>
> - if (unlikely(ena_com_meta_desc_changed(io_sq, ena_tx_ctx)))
> + if (llq_info->disable_meta_caching ||
> + unlikely(ena_com_meta_desc_changed(io_sq, ena_tx_ctx)))
> ++num_descs;
>
> if (num_descs > llq_info->descs_num_before_header) {
> --
> 2.28.0
>
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [dpdk-dev] [dpdk-stable] [PATCH] net/ena/base: fix doorbell evaluation for the LLQ case
2020-08-13 18:07 ` Marcin Wojtas
@ 2020-08-18 15:26 ` Ferruh Yigit
0 siblings, 0 replies; 3+ messages in thread
From: Ferruh Yigit @ 2020-08-18 15:26 UTC (permalink / raw)
To: Marcin Wojtas, Artur Rojek
Cc: Michal Krawczyk, Guy Tzalik, Igor Chauskin, dev, stable
On 8/13/2020 7:07 PM, Marcin Wojtas wrote:
> Hi,
>
> śr., 12 sie 2020 o 18:37 Artur Rojek <ar@semihalf.com> napisał(a):
>>
>> From: Michal Krawczyk <mk@semihalf.com>
>>
>> This patch adds a missing LLQ-related check in the
>> ena_com_is_doorbell_needed() routine, which is relevant for the feature
>> supported by the next generation HW of the ENA.
>>
>> Fixes: b2b02edeb0d6 ("net/ena/base: upgrade HAL for new HW features")
>> CC: stable@dpdk.org
>>
>> Signed-off-by: Michal Krawczyk <mk@semihalf.com>
>> [Extracted from a bigger patch]
>> Signed-off-by: Artur Rojek <ar@semihalf.com>
>
> Reviewed-by: Marcin Wojtas <mw@semihalf.com>
>
Applied to dpdk-next-net/main, thanks.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-08-18 15:26 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-12 16:37 [dpdk-dev] [PATCH] net/ena/base: fix doorbell evaluation for the LLQ case Artur Rojek
2020-08-13 18:07 ` Marcin Wojtas
2020-08-18 15:26 ` [dpdk-dev] [dpdk-stable] " Ferruh Yigit
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).