From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Borkmann Subject: Re: [PATCH bpf-next v5 0/3] bpf: btf: print bpftool map data with btf Date: Sat, 14 Jul 2018 00:35:03 +0200 Message-ID: <1e80f262-9b3f-e4ff-0e89-7a07098d1b50@iogearbox.net> References: <20180712030803.875913594@fb.com> <20180711203010.51bab0cf@cakuba.lan> <20180713213550.GA5667@w1t1fb> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: Jakub Kicinski , Martin KaFai Lau , Alexei Starovoitov , Yonghong Song , Quentin Monnet , "David S. Miller" , netdev@vger.kernel.org, kernel-team@fb.com, linux-kernel@vger.kernel.org To: Okash Khawaja Return-path: Received: from www62.your-server.de ([213.133.104.62]:39498 "EHLO www62.your-server.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727241AbeGMWvw (ORCPT ); Fri, 13 Jul 2018 18:51:52 -0400 In-Reply-To: <20180713213550.GA5667@w1t1fb> Content-Language: en-US Sender: netdev-owner@vger.kernel.org List-ID: On 07/13/2018 11:35 PM, Okash Khawaja wrote: > On Fri, Jul 13, 2018 at 10:49:01PM +0200, Daniel Borkmann wrote: >> On 07/12/2018 05:30 AM, Jakub Kicinski wrote: >>> On Wed, 11 Jul 2018 20:08:03 -0700, Okash Khawaja wrote: >>>> Hi, >>>> >>>> Here are the changes from v4: >>>> >>>> patch 2: >>>> >>>> - sort headers in btf_dumper.c >>>> - remove extra parentheses >>>> - include asm/byteorder.h >>>> - compile error when big and small endian bitfields macro undefined >>> >>> Reviewed-by: Jakub Kicinski >> >> Hmm, strange, by accident I just noticed that only your bpf fix ever made >> it to patchwork, Okash. >> >> https://urldefense.proofpoint.com/v2/url?u=https-3A__patchwork.ozlabs.org_project_netdev_list_-3Fsubmitter-3D74458-26state-3D-2A&d=DwICaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=4wHrS7MHHFLZe_WCJwRVhA&m=wkiBQFYWPyiN9WONHLY0WiZxcOwNRhXaMMLIE551mCA&s=RggQzClRdkwawboGLPgPXHOdUtYffxeOwlcBlFru-P4&e= >> >> Potentially because you've sent with attachments which got dropped on >> the list? > interesting because i send all patches using quilt mail, the same way i > sent bpf fix. i can try git-send-email. > > also i dropped Acked-by as i changed patch versions. is it common thing > to do? or should i keep the Acked-by? Depends on whether the pieces that have been ACKed changed in the meantime or not.