linux-hotplug.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Oliver Neukum <Oliver.Neukum@lrz.uni-muenchen.de>
To: linux-hotplug@vger.kernel.org
Subject: Re: [linux-usb-devel] Re: hotplugging to deal with firmware download
Date: Wed, 05 Jun 2002 14:32:51 +0000	[thread overview]
Message-ID: <marc-linux-hotplug-102328772611916@msgid-missing> (raw)
In-Reply-To: <marc-linux-hotplug-102297360623547@msgid-missing>

Am Mittwoch, 5. Juni 2002 16:19 schrieb David Brownell:
> Oliver Neukum wrote:
> >>>I agree. But usbcore should help as far as possible.
> >>
> >>What were you thinking would transform the "device model" level
> >>suspend/resume calls into USB level ones?  :)
> >
> > OK, but where do we handle the case where resumption is impossible
> > because the device has been unplugged ?
>
> If that's not already handled, it'd be a bug in the hub driver.

Well, how will we handle it with respect to the resume function ?
Do we report failure if there's no longer a device at the physical
location ? IMHO we do.
Do we have checks in the core for identity, or do we leave that
to the driver ? I am not sure.

> >>>>For example, suspending a bus-powered hub would need to morph into
> >>>>disconnecting the devices it could no longer power ... and in your
> >>>>case, suspending a network device that discards its firmware would
> >>>>also need to morph into a disconnect.
> >>>
> >>>That's exactly what you must _not_ do, you need to retain the
> >>> information which devices was on which port to resume correctly.
> >>
> >>Why would that be?  In those cases, the device MUST re-enumerate from
> >>scratch, there will be no USB state to resume.  In those cases the
> >>devices can't suspend; they can only disconnect/reconnect.
> >>
> >>And it'd be pointless, since if any device driver wants to save
> >>information about devices across reconnects -- like usb-storage does
> >>today -- it has all the tools it needs to do that already.  There's
> >>no need to bloat the core with such stuff.
> >
> > That is not the entire truth.
> > The hub will be told to resume first. If we than scan the physical bus
> > and initiate initialisation of the devices found, there'll be chaos
> > once the "driverfs" layer tells devices to resume.
>
> You said "initiate initialisation"; that's normally called "enumeration".
> Which by definition is NOT done for a USB device being resumed!!!!

Now I am puzzeled. There must be some misunderstanding.
Could you please outline, how we deal with a resumption
if the devices lost power under the new scheme ?

> > The point of suspend/resume cycles is that the system is restored
> > to the state it had. IP, mac, filtering etc... must be restored, not only
> > names.
>
> Then there's always reality to deal with.  Like when a network link gets
> removed during suspend.

Yes, that should concern the network layer, not a device driver.
But the device driver must reset MAC and IP at least.

> > In fact, if we have persistent names we don't need to care about names.
>
> Erm, that doesn't make sense.  Having them persistent means someone is
> caring a heck of a lot about names and assignment policies.

We meaning device driver maintainers ;-)

	Regards
		Oliver

_______________________________________________________________

Don't miss the 2002 Sprint PCS Application Developer's Conference
August 25-28 in Las Vegas -- http://devcon.sprintpcs.com/adp/index.cfm

_______________________________________________
Linux-hotplug-devel mailing list  http://linux-hotplug.sourceforge.net
Linux-hotplug-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel

  parent reply	other threads:[~2002-06-05 14:32 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-06-01 23:16 [linux-usb-devel] Re: hotplugging to deal with firmware download Brad Hards
2002-06-02  5:50 ` Oliver Neukum
2002-06-02  8:19 ` Oliver Neukum
2002-06-02 11:11 ` Brad Hards
2002-06-02 17:16 ` David Brownell
2002-06-02 21:59 ` Oliver Neukum
2002-06-02 22:21 ` Brad Hards
2002-06-03  4:18 ` Oliver Neukum
2002-06-03 17:52 ` Greg KH
2002-06-03 21:00 ` Oliver Neukum
2002-06-03 21:55 ` Greg KH
2002-06-03 22:02 ` David Brownell
2002-06-03 22:26 ` Oliver Neukum
2002-06-03 22:35 ` Greg KH
2002-06-03 22:37 ` Greg KH
2002-06-03 22:48 ` Oliver Neukum
2002-06-03 22:58 ` Oliver Neukum
2002-06-03 23:05 ` Greg KH
2002-06-03 23:30 ` Oliver Neukum
2002-06-03 23:40 ` Greg KH
2002-06-04  8:06 ` Oliver Neukum
2002-06-04 19:32 ` David Brownell
2002-06-04 19:44 ` David Brownell
2002-06-05 11:45 ` Oliver Neukum
2002-06-05 14:19 ` David Brownell
2002-06-05 14:32 ` Oliver Neukum [this message]
2002-06-05 14:54 ` David Brownell
2002-06-05 21:48 ` Oliver Neukum
2002-06-06  0:25 ` David Brownell
2002-06-06  9:04 ` Andries.Brouwer
2002-06-06 12:54 ` Oliver Neukum
2002-06-06 14:55 ` Oliver Neukum
2002-06-06 17:16 ` David Brownell
2002-06-06 19:19 ` Andries.Brouwer

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=marc-linux-hotplug-102328772611916@msgid-missing \
    --to=oliver.neukum@lrz.uni-muenchen.de \
    --cc=linux-hotplug@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 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).