netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Stephen Hemminger <shemminger@linux-foundation.org>
To: Brandon Philips <brandon@ifup.org>
Cc: netdev@vger.kernel.org, teheo@suse.de
Subject: Re: [patch 0/5][RFC] Update network drivers to use devres
Date: Fri, 3 Aug 2007 09:58:57 +0100	[thread overview]
Message-ID: <20070803095857.5fb3f368@oldman.hamilton.local> (raw)
In-Reply-To: <20070802224206.GB5181@ifup.org>

On Thu, 2 Aug 2007 15:42:06 -0700
Brandon Philips <brandon@ifup.org> wrote:

> This patch set adds support for devres in the net core and converts the
> e100 and e1000 drivers to devres.  Devres is a simple resource manager
> for device drivers, see Documentation/driver-model/devres.txt for more
> information.
> 
> The use of devres will remain optional for drivers with this patch set.
> Drivers can be converted when it makes sense.

Just because devres exists is not sufficient motivation to change.

It seems that devres was a band-aid rather than fixing storage drivers
to have proper DMA lifetimes. 
Network devices seem to work fine thanks, and the resource requirements
are different. If ain't broke, don't fix it.

  reply	other threads:[~2007-08-03  8:58 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-08-02 22:42 [patch 0/5][RFC] Update network drivers to use devres Brandon Philips
2007-08-03  8:58 ` Stephen Hemminger [this message]
2007-08-03 10:26   ` Tejun Heo
2007-08-03 11:07     ` Stephen Hemminger
2007-08-03 11:33       ` Tejun Heo
2007-08-03 13:44         ` Stephen Hemminger
2007-08-03 18:23           ` Brandon Philips

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=20070803095857.5fb3f368@oldman.hamilton.local \
    --to=shemminger@linux-foundation.org \
    --cc=brandon@ifup.org \
    --cc=netdev@vger.kernel.org \
    --cc=teheo@suse.de \
    /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).