All of lore.kernel.org
 help / color / mirror / Atom feed
From: Murali Krishna Policharla <murali.policharla@broadcom.com>
To: Andrew Lunn <andrew@lunn.ch>
Cc: Heiner Kallweit <hkallweit1@gmail.com>,
	davem@davemloft.net, amritha.nambiar@intel.com,
	ecree@solarflare.com, ktkhai@virtuozzo.com,
	alexander.h.duyck@intel.com, netdev@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: RE: [PATCH] net: core: Fix to store new mtu setting in netdevice.
Date: Thu, 3 Jan 2019 15:24:48 +0530	[thread overview]
Message-ID: <50f99989b763da1cd630e0abf3ed940a@mail.gmail.com> (raw)
In-Reply-To: <20190102131657.GE22737@lunn.ch>

> Hi Andrew/Heiner
>
>                                        Thanks for the feedback. This
patch
> fixes a case where ndo_change_mtu function is provided but the callback
> function is not storing mtu to netdevice structure.

> Hi Murali

> At the moment, any driver which implements ndo_change_mtu MUST set
> ndev->mtu. It is a nice clean definition, easy for any driver write to
> understand.

Hi Andrew
                       Since drivers implementing ndo_change_mtu callback
function are following this approach. Will go with the existing approach
and modify the ndo_change_mtu callback function that I will be adding to
store mtu to netdevice structure.

Thanks
Murali

  reply	other threads:[~2019-01-03  9:54 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-01-01  6:42 [PATCH] net: core: Fix to store new mtu setting in netdevice Murali Krishna Policharla
2019-01-01  7:54 ` Kirill Tkhai
2019-01-01  9:40   ` Heiner Kallweit
2019-01-01 23:36     ` Andrew Lunn
2019-01-02  6:27       ` Heiner Kallweit
2019-01-01  8:34 ` Andrew Lunn
2019-01-01  9:48   ` Murali Krishna Policharla
2019-01-01 23:24     ` Andrew Lunn
2019-01-02  9:54       ` Murali Krishna Policharla
2019-01-02 13:16         ` Andrew Lunn
2019-01-03  9:54           ` Murali Krishna Policharla [this message]
2019-01-01  9:35 ` Heiner Kallweit
2019-01-01 21:44 ` David Miller

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=50f99989b763da1cd630e0abf3ed940a@mail.gmail.com \
    --to=murali.policharla@broadcom.com \
    --cc=alexander.h.duyck@intel.com \
    --cc=amritha.nambiar@intel.com \
    --cc=andrew@lunn.ch \
    --cc=davem@davemloft.net \
    --cc=ecree@solarflare.com \
    --cc=hkallweit1@gmail.com \
    --cc=ktkhai@virtuozzo.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    /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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.