Linux LVM users
 help / color / mirror / Atom feed
From: Patrick Caulfield <patrick@tykepenguin.com>
To: linux-lvm@sistina.com
Subject: [linux-lvm] reiserfs panic
Date: Fri Nov 30 04:22:01 2001	[thread overview]
Message-ID: <20011130102423.E1452@tykepenguin.com> (raw)
In-Reply-To: <20011129164915.I1408@tykepenguin.com>

I'm posting this here partly because I hope Chris Mason is reading(!) and partly
because it may be of some use to others on the list.

There seems to be a problem when reading from a deactivated snapshot of a
reiserfs volume that causes a panic.

The problem is caused by the fact that when a snapshot becomes full, LVM
returns I/O errors for the volume and when reiserfs hits an I/O error it panics.

The solution is, of course, to make your snapshots large enough that this never
happens :-) but just in case this is not possible the following patch may be of
use:


diff -u fs/reiserfs/namei.c.orig fs/reiserfs/namei.c
--- fs/reiserfs/namei.c.orig    Thu Nov 29 16:13:28 2001
+++ fs/reiserfs/namei.c Thu Nov 29 16:34:49 2001
@@ -320,9 +320,10 @@
 
     while (1) {
        retval = search_by_entry_key (dir->i_sb, &key_to_search, path_to_entry, de);
-       if (retval == IO_ERROR)
+       if (retval == IO_ERROR) 
+           return NAME_NOT_FOUND;
            // FIXME: still has to be dealt with
-           reiserfs_panic (dir->i_sb, "zam-7001: io error in " __FUNCTION__ "\n");
+           // reiserfs_panic (dir->i_sb, "zam-7001: io error in " __FUNCTION__ "\n");
 
        /* compare names for all entries having given hash value */
        retval = linear_search_in_dir_item (&key_to_search, de, name, namelen);



patrick

       reply	other threads:[~2001-11-30  4:22 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20011129164915.I1408@tykepenguin.com>
2001-11-30  4:22 ` Patrick Caulfield [this message]
2001-11-30  7:55   ` [linux-lvm] reiserfs panic Chris Mason
2001-11-30  8:04     ` Patrick Caulfield
2001-11-30 10:31     ` Andreas Dilger
2001-12-29 22:31       ` [linux-lvm] Fun little horror story -- please add to FAQ if it isn't already documented Chris Worley
2001-12-30  5:05         ` Ben Holness
2001-12-30  9:29           ` Steven Lembark
2001-11-30 17:59   ` [linux-lvm] reiserfs panic Ed Tomlinson
2001-11-30 18:51     ` Andreas Dilger

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20011130102423.E1452@tykepenguin.com \
    --to=patrick@tykepenguin.com \
    --cc=linux-lvm@sistina.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox