On 10/18/24 12:10, Jijie Shao wrote:
From: Jian Shen <shenjian15@xxxxxxxxxx>It's unclear to me what do you mean for large traffic. Is that large
To avoid errors in pgtable prefectch, add a sync command to sync
io-pagtable.
In the case of large traffic, the TX bounce buffer may be used up.
packets instead?
Skimming over the previous patch, it looks like the for the bugger H/W
driver will use the bounce buffer for all packets with len < 64K. As
this driver does not support big tcp, such condition means all packets.
So its not clear to me the 'may' part - it looks like the critical path
will always happen on the bugged H/W
At this point, we go to mapping/unmapping on TX path again.I thought the goal of the previous patch was to avoid such sync-up.
So we added the sync command in driver to avoid hardware issue.
So I don't understand why it's there.
A more verbose explanation will help.
Signed-off-by: Jian Shen <shenjian15@xxxxxxxxxx>Also we need a fixes tag.
Signed-off-by: Peiyang Wang <wangpeiyang1@xxxxxxxxxx>
Signed-off-by: Jijie Shao <shaojijie@xxxxxxxxxx>
Thanks,
Paolo