From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Brivio Subject: Re: [Patch net-next] net: move __skb_checksum_complete*() to skbuff.c Date: Thu, 8 Nov 2018 21:14:35 +0100 Message-ID: <20181108211435.1f8d953f@redhat.com> References: <20181108194949.11866-1-xiyou.wangcong@gmail.com> <20181108205413.2008a004@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: Linux Kernel Network Developers To: Cong Wang Return-path: Received: from mx1.redhat.com ([209.132.183.28]:38316 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725884AbeKIFvo (ORCPT ); Fri, 9 Nov 2018 00:51:44 -0500 In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: On Thu, 8 Nov 2018 12:02:00 -0800 Cong Wang wrote: > On Thu, Nov 8, 2018 at 11:54 AM Stefano Brivio wrote: > > > > Hi, > > > > On Thu, 8 Nov 2018 11:49:49 -0800 > > Cong Wang wrote: > > > > > +EXPORT_SYMBOL(__skb_checksum_complete); > > > + > > > /* Both of above in one bottle. */ > > > > Maybe you should also update/drop this comment now? > > I have no idea what that comment means. Do you? I think it refers to the fact that skb_copy_and_csum_bits() implements both skb_checksum() and skb_copy_bits(), that were just above it at 1da177e4c3f4. Then more stuff was moved in between, and the comment was never updated or dropped. -- Stefano