From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joe Perches Subject: Re: [PATCH v3 net-next 1/3] ipv4: rename ip_options_echo to __ip_options_echo() Date: Thu, 25 Sep 2014 13:49:23 -0700 Message-ID: <1411678163.3460.9.camel@joe-AO725> References: <1411676092-16196-1-git-send-email-edumazet@google.com> <1411676092-16196-2-git-send-email-edumazet@google.com> <1411677008.3460.7.camel@joe-AO725> <1411677484.16953.43.camel@edumazet-glaptop2.roam.corp.google.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Cc: Eric Dumazet , "David S. Miller" , netdev@vger.kernel.org, Yuchung Cheng , Neal Cardwell , Christoph Paasch , Julia Lawall To: Eric Dumazet Return-path: Received: from smtprelay0171.hostedemail.com ([216.40.44.171]:39060 "EHLO smtprelay.hostedemail.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751645AbaIYUt0 (ORCPT ); Thu, 25 Sep 2014 16:49:26 -0400 In-Reply-To: <1411677484.16953.43.camel@edumazet-glaptop2.roam.corp.google.com> Sender: netdev-owner@vger.kernel.org List-ID: On Thu, 2014-09-25 at 13:38 -0700, Eric Dumazet wrote: > On Thu, 2014-09-25 at 13:30 -0700, Joe Perches wrote: > > > Unrelated: > > > > I wonder how much effort, if any, should be made to convert > > struct sk_buff * to const struct sk_buff * where appropriate. > > > > For instance: > > > > This __ip_options_echo could use const struct sk_buff *skb > > if fib_compute_spec_dst was changed to const struct sk_buff *skb. > > Well, this seems something certainly doable, as a follow up ;) It's doable, but it seems a non-trivial inspection task. I believe coccinelle does not have the ability to automate this.