[GIT PULL] ieee1394 update (was 2.6.19-rc1: known regressions (v2))

[Stefan Richter]

Adrian Bunk wrote:
> This email lists some known regressions in 2.6.19-rc1 compared to 2.6.18
> that are not yet fixed Linus' tree.
[...]
> Subject    : strange ieee1394 messages
> References : http://lkml.org/lkml/2006/10/5/154
> Submitter  : "Alistair John Strachan" <s0348365@xxxxxxxxxxxx>
> Guilty     : Stefan Richter <stefanr@xxxxxxxxxxxxxxxxx>
>              commit d2f119fe319528da8c76a1107459d6f478cbf28c
> Handled-By : Stefan Richter <stefanr@xxxxxxxxxxxxxxxxx>
> Patch      : http://lkml.org/lkml/2006/10/6/235
> Status     : harmless, patch available


Linus, please pull from the upstream-linus branch at

    git://git.kernel.org/pub/scm/linux/kernel/git/ieee1394/linux1394-2.6.git upstream-linus

to receive the following patch...

Stefan Richter:
      ieee1394: nodemgr: fix startup of knodemgrd

 drivers/ieee1394/nodemgr.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

...or just apply it from this mail.


Date: Fri, 6 Oct 2006 19:49:52 +0200 (CEST)
From: Stefan Richter <stefanr@xxxxxxxxxxxxxxxxx>
Subject: ieee1394: nodemgr: fix startup of knodemgrd

Revert a thinko in commit d2f119fe319528da8c76a1107459d6f478cbf28c:
When knodemgrd starts, it needs to sleep until host->generation was
incremented above its initial value of 0.  My wrong logic caused it to
start sending requests when the bus wasn't completely ready.  Seen as
"AT dma reset ctx=0, aborting transmission" messages in 2.6.19-rc1.

Signed-off-by: Stefan Richter <stefanr@xxxxxxxxxxxxxxxxx>
---
Index: linux/drivers/ieee1394/nodemgr.c
===================================================================
--- linux.orig/drivers/ieee1394/nodemgr.c	2006-10-06 19:20:00.000000000 +0200
+++ linux/drivers/ieee1394/nodemgr.c	2006-10-06 19:26:28.000000000 +0200
@@ -1614,7 +1614,7 @@ static int nodemgr_host_thread(void *__h
 {
 	struct host_info *hi = (struct host_info *)__hi;
 	struct hpsb_host *host = hi->host;
-	unsigned int g, generation = get_hpsb_generation(host) - 1;
+	unsigned int g, generation = 0;
 	int i, reset_cycles = 0;
 
 	/* Setup our device-model entries */


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/