linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Rocco Yue <rocco.yue@mediatek.com>
To: David Ahern <dsahern@gmail.com>
Cc: "David S . Miller" <davem@davemloft.net>,
	Hideaki YOSHIFUJI <yoshfuji@linux-ipv6.org>,
	David Ahern <dsahern@kernel.org>,
	Jakub Kicinski <kuba@kernel.org>,
	Matthias Brugger <matthias.bgg@gmail.com>,
	<netdev@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<linux-mediatek@lists.infradead.org>, <rocco.yue@gmail.com>,
	Rocco Yue <rocco.yue@mediatek.com>
Subject: Re: [PATCH net-next] net: ipv6: add IFLA_RA_MTU to expose mtu value in the RA message
Date: Thu, 29 Jul 2021 23:42:51 +0800	[thread overview]
Message-ID: <20210729154251.1380-1-rocco.yue@mediatek.com> (raw)
In-Reply-To: <c4099beb-1c22-ac71-ae05-e3f9a8ab69e2@gmail.com>

On Thu, 2021-07-29 at 08:41 -0600, David Ahern wrote:
> On 7/29/21 3:02 AM, Rocco Yue wrote:
>> diff --git a/include/uapi/linux/if_link.h b/include/uapi/linux/if_link.h
>> index 4882e81514b6..ea6c872c5f2c 100644
>> --- a/include/uapi/linux/if_link.h
>> +++ b/include/uapi/linux/if_link.h
>> @@ -347,7 +347,7 @@ enum {
>>  	 */
>>  	IFLA_PARENT_DEV_NAME,
>>  	IFLA_PARENT_DEV_BUS_NAME,
>> -
>> +	IFLA_RA_MTU,
>>  	__IFLA_MAX
>>  };
>>  
>> diff --git a/include/uapi/linux/ipv6.h b/include/uapi/linux/ipv6.h
>> index 70603775fe91..3dbcf212b766 100644
>> --- a/include/uapi/linux/ipv6.h
>> +++ b/include/uapi/linux/ipv6.h
>> @@ -190,6 +190,7 @@ enum {
>>  	DEVCONF_NDISC_TCLASS,
>>  	DEVCONF_RPL_SEG_ENABLED,
>>  	DEVCONF_RA_DEFRTR_METRIC,
>> +	DEVCONF_RA_MTU,
>>  	DEVCONF_MAX
>>  };
>>  
> 
> you do not need both IFLA and DEVCONF. Drop the DEVCONF completely. IFLA
> attribute can be used for both inspection and notification on change.

Hi David,

Thanks for your review.

Because the purpose of this patch is for userspace to correctly read the
ra_mtu value of different network device, if the DEVCONF is completely dropped,
does that mean I can add the "ra_mtu" member in the "struct net_device" .

> Also do not add mailing lists that cause bounces. Specifically, you tend
> to add wsd_upstream@mediatek.com as a cc and every response to you
> generates a bounce message for this address.

Thanks for pointing out the problem, I will remove it from the mailing list.

Best Regards,
Rocco

  reply	other threads:[~2021-07-29 16:01 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-07-29  9:02 Rocco Yue
2021-07-29 14:41 ` David Ahern
2021-07-29 15:42   ` Rocco Yue [this message]
2021-07-29 17:28     ` David Ahern
2021-07-30 17:39       ` Rocco Yue
2021-07-29 14:43 ` David Ahern
2021-07-29 16:45 ` kernel test robot
2021-07-29 18:32 ` kernel test robot

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210729154251.1380-1-rocco.yue@mediatek.com \
    --to=rocco.yue@mediatek.com \
    --cc=davem@davemloft.net \
    --cc=dsahern@gmail.com \
    --cc=dsahern@kernel.org \
    --cc=kuba@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=matthias.bgg@gmail.com \
    --cc=netdev@vger.kernel.org \
    --cc=rocco.yue@gmail.com \
    --cc=yoshfuji@linux-ipv6.org \
    --subject='Re: [PATCH net-next] net: ipv6: add IFLA_RA_MTU to expose mtu value in the RA message' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).