netdev - RE: [PATCH net-next v3 2/5] lan743x: sync only the received area of an rx ring buffer

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <BN8PR11MB36518045F806DAAC37BBD659FA879@BN8PR11MB3651.namprd11.prod.outlook.com>
Date:   Tue, 16 Feb 2021 20:03:40 +0000
From:   <Bryan.Whitehead@...rochip.com>
To:     <thesven73@...il.com>, <UNGLinuxDriver@...rochip.com>,
        <davem@...emloft.net>, <kuba@...nel.org>
CC:     <andrew@...n.ch>, <rtgbnm@...il.com>, <sbauer@...ckbox.su>,
        <tharvey@...eworks.com>, <anders@...ningen.priv.no>,
        <hdanton@...a.com>, <hch@....de>,
        <willemdebruijn.kernel@...il.com>, <netdev@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH net-next v3 2/5] lan743x: sync only the received area of
 an rx ring buffer

> From: Sven Van Asbroeck <thesven73@...il.com>
> 
> On cpu architectures w/o dma cache snooping, dma_unmap() is a is a very
> expensive operation, because its resulting sync needs to invalidate cpu
> caches.
> 
> Increase efficiency/performance by syncing only those sections of the
> lan743x's rx ring buffers that are actually in use.
> 
> Signed-off-by: Sven Van Asbroeck <thesven73@...il.com>
> ---

Looks Good, Thanks Sven
Our testing is in progress, We will let you know our results soon.

Reviewed-by: Bryan Whitehead <Bryan.Whitehead@...rochip.com>