Bug #1415

ext3 unmount panic

Added by polachok almost 4 years ago. Updated almost 4 years ago.

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.

P7030299.JPG (79.8 kB) polachok, 07/03/2009 08:31 pm


Related todos

History

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.

Also available in: Atom PDF