[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20241015181657.14fe9227@kernel.org>
Date: Tue, 15 Oct 2024 18:16:57 -0700
From: Jakub Kicinski <kuba@...nel.org>
To: Jijie Shao <shaojijie@...wei.com>
Cc: <davem@...emloft.net>, <edumazet@...gle.com>, <pabeni@...hat.com>,
<shenjian15@...wei.com>, <salil.mehta@...wei.com>,
<liuyonglong@...wei.com>, <wangpeiyang1@...wei.com>, <lanhao@...wei.com>,
<chenhao418@...wei.com>, <netdev@...r.kernel.org>,
<linux-kernel@...r.kernel.org>
Subject: Re: [PATCH net 1/9] net: hns3: default enable tx bounce buffer when
smmu enabled
On Fri, 11 Oct 2024 17:45:13 +0800 Jijie Shao wrote:
> From: Peiyang Wang <wangpeiyang1@...wei.com>
>
> When TX bounce buffer is enabled, dma map is used only when the buffer
> initialized. When spending packages, the driver only do dma sync. To
packages -> packets
> avoid SMMU prefetch, default enable tx bounce buffer if smmu enabled.
you seem to force it to be enabled, rather than just changing
the default. That is strange. Why not let the user lower the value?
Also I don't see why this is a fix. Seems like a performance
improvement.
--
pw-bot: cr
Powered by blists - more mailing lists