From mboxrd@z Thu Jan 1 00:00:00 1970 From: John Fastabend Subject: Re: [bpf-next PATCH v2 1/7] bpf: refactor sockmap sample program update for arg parsing Date: Thu, 11 Jan 2018 20:58:36 -0800 Message-ID: <83d8af14-5cbe-e8a9-8f09-15da6bbd015c@gmail.com> References: <20180110183600.5930.68261.stgit@john-Precision-Tower-5810> <20180110183904.5930.42185.stgit@john-Precision-Tower-5810> <86c79df5-5b5e-a996-41d1-48b7aa960c02@iogearbox.net> <456f5c18-b1c6-77b3-58bd-6f954a83756c@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: netdev@vger.kernel.org To: Daniel Borkmann , borkmann@iogearbox.net, ast@kernel.org Return-path: Received: from mail-pg0-f66.google.com ([74.125.83.66]:40666 "EHLO mail-pg0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932734AbeALE6t (ORCPT ); Thu, 11 Jan 2018 23:58:49 -0500 Received: by mail-pg0-f66.google.com with SMTP id q12so3863092pgt.7 for ; Thu, 11 Jan 2018 20:58:49 -0800 (PST) In-Reply-To: <456f5c18-b1c6-77b3-58bd-6f954a83756c@gmail.com> Content-Language: en-US Sender: netdev-owner@vger.kernel.org List-ID: On 01/11/2018 08:31 PM, John Fastabend wrote: > On 01/10/2018 05:25 PM, Daniel Borkmann wrote: >> On 01/10/2018 07:39 PM, John Fastabend wrote: >>> sockmap sample program takes arguments from cmd line but it reads them >>> in using offsets into the array. Because we want to add more arguments >>> in the future lets do proper argument handling. >>> >>> Also refactor code to pull apart sock init and ping/pong test. This >>> allows us to add new tests in the future. >>> >>> Signed-off-by: John Fastabend >>> --- >>> samples/sockmap/sockmap_user.c | 142 +++++++++++++++++++++++++++++----------- >>> 1 file changed, 103 insertions(+), 39 deletions(-) >> [...] >>> >>> /* Accept Connecrtions */ >>> @@ -149,23 +177,32 @@ static int sockmap_test_sockets(int rate, int dot) >>> goto out; >>> } >>> >>> - max_fd = p2; >>> - timeout.tv_sec = 10; >>> - timeout.tv_usec = 0; >>> - >>> printf("connected sockets: c1 <-> p1, c2 <-> p2\n"); >>> printf("cgroups binding: c1(%i) <-> s1(%i) - - - c2(%i) <-> s2(%i)\n", >>> c1, s1, c2, s2); >>> +out: >>> + return err; >> >> Maybe rather than setting err and goto out where we now just return >> err anyway, return from those places directly. >> > > Perhaps but how about doing this in another patch. This > patch is not changing the goto err pattern. I can send > a follow up. OK I take it back. I went ahead an removed the goto as you suggested. As Martin noticed the accept err was missing and also most of those should have been errno instead of err. v3 in-flight. .John