Messages in this thread | | | Date | Tue, 05 Oct 2004 10:40:58 -0700 | From | Hans Reiser <> | Subject | Re: [PATCH 0/4] I/O Error Handling for ReiserFS v3 |
| |
Alex Zarochentsev wrote:
>On Tue, Oct 05, 2004 at 08:44:22AM -0700, Hans Reiser wrote: > > >>These have received design approval from zam (and thus me), but zam, did >>they receive stress testing by Elena under your guidance? >> >> > >No. We have a long queue of test tasks. There are fsck.reiser4 testing, >reiser4/dmapper crashes and the benchmarks in the queue. > > Well, we cannot let our process be a barrier to good patches getting in, so let me ask, Jeff, did you test each of these conditions you improved? How? Did anyone else test them?
> > >>Hans >> >>Jeffrey Mahoney wrote: >> >> >> >>>Hey all - >>> >>>One of the most common complaints I've heard about ReiserFS is how graceless >>>it is in handling critical I/O errors. >>> >>>ext[23] can handle I/O errors anywhere, with the results being up to the >>>system admin to determine: continue, go read only, or panic. >>> >>>ReiserFS doesn't offer the admin any such choice, instead panicking on any >>>I/O error in the journal. >>> >>>The available options are read only or panic, since ReiserFS does not >>>currently support operations without the journal. >>> >>>In the four messages that follow, you'll find: * >>>reiserfs-cleanup-buffer-heads.diff - Cleans up handling of buffer head >>>bitfields - uses the kernel supplied FNS_BUFFER macros instead. * >>>reiserfs-cleanup-sb-journal.diff - Cleans up accessing of the journal >>>structure, prefering to create a temporary variable in functions that access >>>the journal structure non-trivially. Should make 0 difference at compile >>>time. * reiserfs-io-error-handling.diff - Allows ReiserFS to gracefully >>>handle I/O errors in critical code paths. The admin has the option to go >>>read-only or panic. Since ReiserFS has no option to ignore the use of the >>>journal, the "continue" method is not enabled. * reiserfs-write-lock.diff - >>>Fixes two missing reiserfs_write_unlock() calls on error paths that are >>>unrelated to reiserfs-io-error-handling.diff >>> >>>These patches have seen a lot of testing in the SuSE Linux Enterprise Server >>>9 kernel, and are considered ready for mainline. >>> >>>They've received approval[1] from the ReiserFS maintainers also. >>> >>>Andrew - Apologies for the previous format; Please apply. >>> >>>Thanks. >>> >>>-Jeff >>> >>>[1] http://marc.theaimsgroup.com/?l=reiserfs&m=109587254714180 >>> >>>-- Jeff Mahoney SuSE Labs >>> >>> >>> >>> > > >
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |