Bug 5492

Summary:	pegasus: get_registers never wakes up
Product:	Drivers	Reporter:	Herbert Xu (herbert)
Component:	USB	Assignee:	Greg Kroah-Hartman (greg)
Status:	REJECTED INSUFFICIENT_DATA
Severity:	normal
Priority:	P2
Hardware:	i386
OS:	Linux
Kernel Version:	2.6.14-rc5	Subsystem:
Regression:	---	Bisected commit-id:
Bug Depends on:
Bug Blocks:	5089

Description Herbert Xu 2005-10-24 21:59:50 UTC

The pegasus get_registers function which is invoked by things such as MII
operations and EEPROM reads may lock because

1) The callback may not be invoked in case of hardware error.
2) The callback does not check the return status of update_regs_eth_async.

For 1) it should impose a timeout just like usb_control_msg.  For 2) it should
check the return status and take the appropriate action (such as waking up the
sleeping process).

Comment 1 Greg Kroah-Hartman 2005-11-14 21:24:39 UTC

Care to propose a patch for this?

Comment 2 Greg Kroah-Hartman 2006-03-06 10:28:20 UTC

No response in 4 months, closing.  If this is still a problem, please reopen
with the requested information.

Comment 3 Herbert Xu 2006-03-06 12:27:33 UTC

Hmm, just because I don't have time to write a patch it doesn't mean that this
bug doesn't exist.  Please only close this bug when it's fixed.  Thanks.

Comment 4 Greg Kroah-Hartman 2006-03-07 11:15:28 UTC

Well, as I do not have this device, nor do I know exactly what the change
should look like, and I'm neither the author or maintainer of this driver,
it's going to be a _very_ long time before I get the time to be able to
write a patch to fix this.

So I'm closing this for now, and if you want to get this fixed, I suggest that
you contact the author and maintainer of this driver.

Sorry.