Bug #1415
ext3 unmount panic
| Status: | New | Start date: | ||
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: | - | % Done: | 0% |
|
| Category: | - | |||
| Target version: | - |
Description
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.
Related todos
History
Updated by polachok almost 4 years ago
I have a possible fix here, please take a look:
http://github.com/polachok/dragonflybsd/commit/d8b7a4caaa2724148e95939207977a0730c349b2
Updated by dillon almost 4 years ago
:Alexander Polakov <polachok@gmail.com> added the comment:
:
:I have a possible fix here, please take a look:
:
:http://github.com/polachok/dragonflybsd/commit/d8b7a4caaa2724148e9593920797=
:7a0730c349b2
:
:----------
:status: unread -> chatting
That looks reasonable. Can you verify that it still works with ext2?
Then I think we can commit it.
-Matt
Matthew Dillon
<dillon@backplane.com>
Updated by polachok almost 4 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 I
just downloaded.
Updated by dillon almost 4 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=
: I
:just downloaded.
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?
-Matt
Matthew Dillon
<dillon@backplane.com>
Updated by polachok almost 4 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.