[ovs-dev] [PATCH 0/2] Implement terse dump for netdev-offload

[Simon Horman]

On Wed, Jun 03, 2020 at 09:51:10AM +0300, Roi Dayan wrote:
> 
> 
> On 2020-06-02 4:36 PM, Simon Horman wrote:
> > On Tue, Jun 02, 2020 at 01:21:16PM +0200, Simon Horman wrote:
> >> On Mon, Jun 01, 2020 at 03:54:54PM +0300, Roi Dayan wrote:

...

> > Hi,
> > 
> > unfortunately Travis-CI has flaged a number of failures.
> > I am wondering if they are explained by this one:
> > 
> > https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Ftravis-ci.org%2Fgithub%2Fhorms2%2Fovs%2Fjobs%2F693802778%23L2874&data=02%7C01%7Croid%40mellanox.com%7C02fb615747344cca672f08d806f9fcca%7Ca652971c7d2e4d9ba6a4d149256f461b%7C0%7C0%7C637267018068811121&sdata=DX5m199CmwIp5%2BlRdofkPdHRk%2B0b4LSu%2FHXMJ1KJNSQ%3D&reserved=0
> > 
> > ...
> > 
> > ofproto/ofproto-dpif-upcall.c:2585:55: error: reading variable 'stats' requires
> >       holding any mutex [-Werror,-Wthread-safety-analysis]
> >                       udpif->dpif->current_ms : ukey->stats.used;
> >                                                       ^
> > 
> > ofproto/ofproto-dpif-upcall.c:2588:18: error: reading variable 'created'
> >       requires holding any mutex [-Werror,-Wthread-safety-analysis]
> >     return ukey->created;
> >                  ^
> > 
> > ...
> > 
> 
> 
> thanks Simon. 
> we actually checked travis in earlier version of the patch and all passed
> https://travis-ci.org/github/roidayan/ovs/builds/685769626
> after some modifications we wanted we missed checking travis again.
> we'll fix it and send v2.

Thanks Roi,

much appreciated.