netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: David Miller <davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org>
To: dh.herrmann-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org
Cc: netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	teg-B22kvLQNl6c@public.gmane.org,
	johannes-cdvu00un1VgdHxzADdlk8Q@public.gmane.org,
	linux-wireless-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	kay.sievers-tD+1rO4QERM@public.gmane.org
Subject: Re: [PATCH v6 0/4] Provide netdev naming-policy via sysfs
Date: Thu, 03 Apr 2014 14:03:24 -0400 (EDT)	[thread overview]
Message-ID: <20140403.140324.1744832114380351572.davem@davemloft.net> (raw)
In-Reply-To: <1396432706-14470-1-git-send-email-dh.herrmann-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>

From: David Herrmann <dh.herrmann-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Date: Wed,  2 Apr 2014 11:58:22 +0200

> This is v6 of the name_assign_type attribute for netdevs. Changes since v5 [1]:
>  - Acked-by: Kay Sievers
>  - rebased on net-next of today
> 
> Aimed for 3.15, but feel free to delay for 3.16 if you want it in -next for some
> more time.

I'm fine with the basic idea, but the implementation needs work and
this will be deferred to 3.16 in any event.

We really need to specify the name allocation type at alloc_netdev()
time, I know that's more work but that's the only way you can be sure
you caught all such cases.

I know you missed the ISDN layer, where the name can come from the
user via an ioctl().  I'm sure there were several more.

You can even do something like make alloc_netdev_enumerated()
be a shorthand for alloc_netdev(size, name, NET_NAME_ENUM, ...)

Thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

      parent reply	other threads:[~2014-04-03 18:03 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-02  9:58 [PATCH v6 0/4] Provide netdev naming-policy via sysfs David Herrmann
2014-04-02  9:58 ` [PATCH v6 1/4] net: add name_assign_type netdev attribute David Herrmann
2014-04-02  9:58 ` [PATCH v6 2/4] mac80211: set NET_NAME_USER for user-space created ifs David Herrmann
2014-04-02  9:58 ` [PATCH v6 3/4] ath6kl: set NET_NAME_USER for P2P ifs David Herrmann
     [not found] ` <1396432706-14470-1-git-send-email-dh.herrmann-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2014-04-02  9:58   ` [PATCH v6 4/4] brcmfmac: " David Herrmann
2014-04-03 18:03   ` David Miller [this message]

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=20140403.140324.1744832114380351572.davem@davemloft.net \
    --to=davem-ft/pcqaiutieiz0/mpfg9q@public.gmane.org \
    --cc=dh.herrmann-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=johannes-cdvu00un1VgdHxzADdlk8Q@public.gmane.org \
    --cc=kay.sievers-tD+1rO4QERM@public.gmane.org \
    --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-wireless-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=teg-B22kvLQNl6c@public.gmane.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 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).