mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "J.A. Magallon" <jamagallon@able.es>
To: Alan Stern <stern@rowland.harvard.edu>
Cc: Andrew Morton <akpm@osdl.org>, Greg KH <greg@kroah.com>,
	Kernel development list <linux-kernel@vger.kernel.org>,
	USB development list <linux-usb-devel@lists.sourceforge.net>
Subject: Re: [linux-usb-devel] Re: 2.6.15-rc5-mm1
Date: Tue, 13 Dec 2005 14:51:12 +0100	[thread overview]
Message-ID: <20051213145112.46301af0@werewolf.auna.net> (raw)
In-Reply-To: <Pine.LNX.4.44L0.0512122241270.17181-100000@netrider.rowland.org>

[-- Attachment #1: Type: text/plain, Size: 3852 bytes --]

On Mon, 12 Dec 2005 22:44:57 -0500 (EST), Alan Stern <stern@rowland.harvard.edu> wrote:

> On Mon, 12 Dec 2005, Andrew Morton wrote:
> 
> > "J.A. Magallon" <jamagallon@able.es> wrote:
> > >
> > > > Sorry for the delay. I'm just compiling all rcs from rc2 to rc5 and will
> > > > try to boot whith them.
> > > > 
> > > > For the rest of your questions:
> > > > - I have no /etc/udev/agents.d/usb/usbcore
> > > > - I have killed all the devfs compat scripts/rules (BTW, when will be finally
> > > >   erradicated from  udev ;) ?
> > > > - Distro: Mandriva Cooker, updated daily, udev-077 now (the hangs I reported
> > > >   were with 075).
> > > > 
> > > > More info soon...
> > > > 
> > > 
> > > No problems with plain rc5. It does not seem to _always_ happen on -mm1,
> > > I thing I even got a clean boot, but just one. 
> > > Detailed oops screenshot is here:
> > > 
> > > http://belly.cps.unizar.es/~magallon/oops/oops.jpg
> > > 
> > 
> > Thanks for that.
> > 
> > Let's add the usb list..
> 
> Uh-oh.  Looks like this one was my fault...  Clashing uses of a local 
> variable.  :-(
> 
> Does this patch fix it?
> 
> Alan Stern
> 
> 
> 
> Index: usb-2.6/drivers/usb/core/message.c
> ===================================================================
> --- usb-2.6.orig/drivers/usb/core/message.c
> +++ usb-2.6/drivers/usb/core/message.c
> @@ -1387,11 +1387,11 @@ free_interfaces:
>  	if (dev->state != USB_STATE_ADDRESS)
>  		usb_disable_device (dev, 1);	// Skip ep0
>  
> -	n = dev->bus_mA - cp->desc.bMaxPower * 2;
> -	if (n < 0)
> +	i = dev->bus_mA - cp->desc.bMaxPower * 2;
> +	if (i < 0)
>  		dev_warn(&dev->dev, "new config #%d exceeds power "
>  				"limit by %dmA\n",
> -				configuration, -n);
> +				configuration, -i);
>  
>  	if ((ret = usb_control_msg(dev, usb_sndctrlpipe(dev, 0),
>  			USB_REQ_SET_CONFIGURATION, 0, configuration, 0,
> 

Bingo! This corrected the problem. I applied it to rc5-mm2 and booted nicely.
One less bug.

A side question. Are this messages dangerous ?

hub 4-0:1.0: USB hub found
hub 4-0:1.0: 2 ports detected
ACPI: PCI Interrupt 0000:00:1d.7[D] -> GSI 23 (level, low) -> IRQ 19
PCI: Setting latency timer of device 0000:00:1d.7 to 64
ehci_hcd 0000:00:1d.7: EHCI Host Controller
PCI: cache line size of 128 is not supported by device 0000:00:1d.7
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
ehci_hcd 0000:00:1d.7: new USB bus registered, assigned bus number 5
ehci_hcd 0000:00:1d.7: irq 19, io mem 0xed200000
ehci_hcd 0000:00:1d.7: USB 2.0 started, EHCI 1.00, driver 10 Dec 2004
usb 1-1: unable to read config index 0 descriptor/all
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
usb 1-1: can't read configurations, error -71
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
usb usb5: configuration #1 chosen from 1 choice
hub 5-0:1.0: USB hub found
hub 5-0:1.0: 8 ports detected

lspci:
00:1d.0 USB Controller: Intel Corporation 82801EB/ER (ICH5/ICH5R) USB UHCI Controller #1 (rev 02)
00:1d.1 USB Controller: Intel Corporation 82801EB/ER (ICH5/ICH5R) USB UHCI Controller #2 (rev 02)
00:1d.2 USB Controller: Intel Corporation 82801EB/ER (ICH5/ICH5R) USB UHCI Controller #3 (rev 02)
00:1d.3 USB Controller: Intel Corporation 82801EB/ER (ICH5/ICH5R) USB UHCI Controller #4 (rev 02)
00:1d.7 USB Controller: Intel Corporation 82801EB/ER (ICH5/ICH5R) USB2 EHCI Controller (rev 02)


Thanks for all.

But don't be too happy, I have a couple things in the queue, like SMP
kernel not booting with 'nosmp' :).

--
J.A. Magallon <jamagallon()able!es>     \               Software is like sex:
werewolf!able!es                         \         It's better when it's free
Mandriva Linux release 2006.1 (Cooker) for i586
Linux 2.6.15-rc5-mm2 (gcc 4.0.2 (4.0.2-1mdk for Mandriva Linux release 2006.1))

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 189 bytes --]

  reply	other threads:[~2005-12-13 13:49 UTC|newest]

Thread overview: 51+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-12-05  7:21 2.6.15-rc5-mm1 Andrew Morton
2005-12-05  6:49 ` 2.6.15-rc5-mm1 Carlos Martín
2005-12-06  3:04   ` 2.6.15-rc5-mm1 Andrew Morton
2005-12-06  6:59     ` 2.6.15-rc5-mm1 Ingo Molnar
2005-12-05 19:06 ` 2.6.15-rc5-mm1: git-alsa-vs-git-pcmcia.patch introduces new compile errors Adrian Bunk
2005-12-05 20:05   ` Takashi Iwai
2005-12-05 21:40 ` 2.6.15-rc5-mm1: USB_IP problems Adrian Bunk
2005-12-07  0:02   ` Greg KH
2005-12-07  0:08     ` Adrian Bunk
2005-12-05 23:05 ` 2.6.15-rc5-mm1 J.A. Magallon
2005-12-05 23:06   ` 2.6.15-rc5-mm1 Randy.Dunlap
2005-12-10 23:36   ` 2.6.15-rc5-mm1 Greg KH
2005-12-10 23:46     ` 2.6.15-rc5-mm1 J.A. Magallon
2005-12-11 21:36       ` 2.6.15-rc5-mm1 J.A. Magallon
2005-12-12 22:58         ` 2.6.15-rc5-mm1 Andrew Morton
2005-12-13  3:44           ` [linux-usb-devel] 2.6.15-rc5-mm1 Alan Stern
2005-12-13 13:51             ` J.A. Magallon [this message]
2005-12-13 15:35               ` Alan Stern
2005-12-13 16:47                 ` David Brownell
2005-12-06 13:33 ` 2.6.15-rc5-mm1 KAMEZAWA Hiroyuki
2005-12-07  0:46 ` 2.6.15-rc5-mm1 (x86_64-hpet-overflow.patch breaks resume from disk) Rafael J. Wysocki
2005-12-07 23:15   ` Rafael J. Wysocki
2005-12-08  8:43     ` [discuss] " Jan Beulich
2005-12-08 10:53       ` Rafael J. Wysocki
2005-12-08 22:35         ` Rafael J. Wysocki
2005-12-09  9:15           ` Jan Beulich
2005-12-09  9:16             ` Andi Kleen
2005-12-09 11:20             ` Rafael J. Wysocki
2005-12-09 12:41               ` Jan Beulich
2005-12-09 13:10                 ` Rafael J. Wysocki
2005-12-09 17:34                 ` Rafael J. Wysocki
2005-12-12  7:58                   ` Jan Beulich
2005-12-12  8:05                     ` [discuss] " Andi Kleen
2005-12-08 22:47       ` Andi Kleen
2005-12-08 23:00         ` Rafael J. Wysocki
2005-12-09  9:08         ` Jan Beulich
2005-12-09  9:16           ` Andi Kleen
2005-12-09 10:57             ` Jan Beulich
2005-12-08 19:09 ` 2.6.15-rc5-mm1 Badari Pulavarty
2005-12-08 21:14   ` 2.6.15-rc5-mm1 James Courtier-Dutton
2005-12-08 23:02   ` 2.6.15-rc5-mm1 Adrian Bunk
2005-12-09  7:15     ` [Alsa-devel] 2.6.15-rc5-mm1 Jaroslav Kysela
2005-12-09  1:09 ` 2.6.15-rc5-mm1 Alexander E. Patrakov
2005-12-09  1:52   ` 2.6.15-rc5-mm1 Jeff Garzik
2005-12-12 16:12   ` 2.6.15-rc5-mm1 Alan Cox
2005-12-13 22:49 ` 2.6.15-rc5-mm1 J.A. Magallon
2005-12-13 23:24   ` 2.6.15-rc5-mm1 Andrew Morton
2005-12-14  0:17     ` 2.6.15-rc5-mm1 J.A. Magallon
2005-12-14  0:22       ` 2.6.15-rc5-mm1 Andrew Morton
2005-12-14  1:33         ` 2.6.15-rc5-mm1 Dmitry Torokhov
2005-12-14  0:31       ` 2.6.15-rc5-mm1 Ben Pfaff

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=20051213145112.46301af0@werewolf.auna.net \
    --to=jamagallon@able.es \
    --cc=akpm@osdl.org \
    --cc=greg@kroah.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb-devel@lists.sourceforge.net \
    --cc=stern@rowland.harvard.edu \
    /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

all inboxes | Powered by JetHome®