From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alexei Starovoitov Subject: Re: [net PATCH 3/3] bpf: devmap, use cond_resched instead of cpu_relax Date: Fri, 8 Sep 2017 15:40:47 -0700 Message-ID: <893ece6d-ac19-fd3e-aee2-f5032d2d13d8@fb.com> References: <150490397545.11590.1409723973253492363.stgit@john-XPS-13-9360> <150490447017.11590.17727368565733920857.stgit@john-XPS-13-9360> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit Cc: , To: John Fastabend , Return-path: Received: from mx0a-00082601.pphosted.com ([67.231.145.42]:43136 "EHLO mx0a-00082601.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752952AbdIHWlO (ORCPT ); Fri, 8 Sep 2017 18:41:14 -0400 In-Reply-To: <150490447017.11590.17727368565733920857.stgit@john-XPS-13-9360> Sender: netdev-owner@vger.kernel.org List-ID: On 9/8/17 2:01 PM, John Fastabend wrote: > Be a bit more friendly about waiting for flush bits to complete. > Replace the cpu_relax() with a cond_resched(). > > Suggested-by: Daniel Borkmann > Acked-by: Daniel Borkmann > Signed-off-by: John Fastabend unlike patch 1 and 2, this one could have waited till net-next opens, but I don't mind now. lgtm Acked-by: Alexei Starovoitov