From mboxrd@z Thu Jan 1 00:00:00 1970 From: Cong Wang Subject: Re: [PATCH] ipv6 addrconf: disallow rtr_solicits < -1 Date: Thu, 6 Oct 2016 17:53:19 -0700 Message-ID: References: <1475476917-17382-1-git-send-email-zenczykowski@gmail.com> <20161004.003831.2056791273400961514.davem@davemloft.net> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Cc: David Miller , Linux NetDev , Erik Kline , Lorenzo Colitti To: =?UTF-8?Q?Maciej_=C5=BBenczykowski?= Return-path: Received: from mail-it0-f52.google.com ([209.85.214.52]:36039 "EHLO mail-it0-f52.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753266AbcJGAxk (ORCPT ); Thu, 6 Oct 2016 20:53:40 -0400 Received: by mail-it0-f52.google.com with SMTP id l13so2627079itl.1 for ; Thu, 06 Oct 2016 17:53:40 -0700 (PDT) In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: On Mon, Oct 3, 2016 at 11:40 PM, Maciej =C5=BBenczykowski wrote: >> Please remove the const qualifier and the casts to be consistent >> with how we handle this elsewhere. >> >> Thanks. > > I can of course trivially make that change. > > But: > > (on net-next/master) > git grep 'extra[12].*=3D.*\(void *[*]\)' > > currently finds 45 matches, and this patch adds a 46th. Seems the sysctl layer should make them const, but I never look into it, it doesn't look like it needs to modify these min/max consts.