ext3 unmount panic
I have a linux ext3 partition here. When trying to mount it, it doesn't work
(Bad file descriptor). When trying to unmount, panic appears. I don't have
firewire or serial here, so got a screenshot.
#1 Updated by polachok over 10 years ago
I have a possible fix here, please take a look:
#2 Updated by dillon over 10 years ago
:Alexander Polakov <firstname.lastname@example.org> added the comment:
:I have a possible fix here, please take a look:
:status: unread -> chatting
That looks reasonable. Can you verify that it still works with ext2?
Then I think we can commit it.
#4 Updated by dillon over 10 years ago
:>That looks reasonable. Can you verify that it still works with ext2?
:>Then I think we can commit it.
:Still works with a 9 years old ext2 from a random linux floppy distribution=
Excellent. Ok, I cherry-picked it from your github, with one
whitespace complaint fix by git.
GIT is so nice. Just being able to pull directly from other repos as a
matter of expectation makes it worth its weight in gold.
What about ce2c38dd9eaa1cb098507b936f39fca5476b0435 in your repo?
Would you like that integrated into the main tree too?
#5 Updated by polachok over 10 years ago
>What about ce2c38dd9eaa1cb098507b936f39fca5476b0435 in your repo?
>Would you like that integrated into the main tree too?
Not really yet. It needs strcasecmp() and strsep() which should probably belong
to libkern (like it's done in FreeBSD). I put them into compat.c, but it doesn't
look like a good solution for me.
And we should probably wait for 8.0 to port shining new features described in
http://people.freebsd.org/~ariff/SOUND_4.TXT.html. It fixes some suspend/resume
problems on my eee, but ACPI in the main tree is not ready for suspend anyway.
#6 Updated by tuxillo over 10 years ago
- Description updated (diff)
- Category set to VFS subsystem
- Status changed from New to Closed
- Assignee deleted (
- Target version set to 3.8.0
In our tree:
Author: Matthew Dillon <email@example.com>
Date: Tue Jul 21 12:39:03 2009 -0700
EXT2FS: support variable inode size.
Submitted-by: Alexander Polakov
The github repo is no longer available and the other commit matt referred seemed to not be ready so I'm closing this.