From: Stefan Richter on
Jan Kara wrote:
> I guess your are hitting a variant of
> https://bugzilla.kernel.org/show_bug.cgi?id=15906
> Does the fix there make things better for you?

The question to me is whether or not Justin sees an (additional)
hardware/firmware bug for which we need to add a quirks list entry.
--
Stefan Richter
-=====-==-=- -=-= -==--
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: Justin P. Mattock on
well, I'm confused on this one..

If I cp -R 2.6.34(kernel) using the macbook pro
I can umount cleanly. But if I do the same for
the iMac9,1 I get this issue.

At first thought this might be an ext4 sync(and possibly)
hardware issue but if it was, then the macbook would portrait
similar issues as well(but could be wrong).

I did add(hopefully)the firelite device to the
workaround list with as many options as possible,
but had no difference(added/removed options as well)
here is what the patch looks like:


From 445d63a80e336408151f4dff7c23fb59a070d246 Mon Sep 17 00:00:00 2001
From: Justin P. Mattock <justinmattock(a)gmail.com>
Date: Sun, 16 May 2010 11:23:00 -0700
Subject: [PATCH] fix an umount bug
Signed-off-by: Justin P. Mattock <justinmattock(a)gmail.com>

---
drivers/ieee1394/sbp2.c | 11 +++++++++++
1 files changed, 11 insertions(+), 0 deletions(-)

diff --git a/drivers/ieee1394/sbp2.c b/drivers/ieee1394/sbp2.c
index 4565cb5..ce826f3 100644
--- a/drivers/ieee1394/sbp2.c
+++ b/drivers/ieee1394/sbp2.c
@@ -393,6 +393,17 @@ static const struct {
.model = SBP2_ROM_VALUE_WILDCARD,
.workarounds = SBP2_WORKAROUND_128K_MAX_TRANS,
},
+ /* SmartDsk FireLite Model #FWFL60-N */
+ {
+ .firmware_revision = 0x000470,
+ .model = 0x000000,
+ .workarounds = SBP2_WORKAROUND_128K_MAX_TRANS |
+ SBP2_WORKAROUND_INQUIRY_36 |
+ SBP2_WORKAROUND_MODE_SENSE_8 |
+ SBP2_WORKAROUND_FIX_CAPACITY |
+ SBP2_WORKAROUND_DELAY_INQUIRY |
+ SBP2_WORKAROUND_POWER_CONDITION,
+ },
/*
* iPod 2nd generation: needs 128k max transfer size workaround
* iPod 3rd generation: needs fix capacity workaround
--
1.6.5.2.180.gc5b3e


I'm might be wrong with the .firmware_revision and model
which I got echo 1 > /sys/module/sbp2/parameters/workarounds

only thing I didn't test was adding this patch, then doing
the echo command(not sure if it matters or not though).

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