All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yury Umanets <umka@namesys.com>
To: Szakacsits Szabolcs <szaka@sienet.hu>
Cc: Alan Chu <alanchu@MIT.EDU>, Vitaly Fertman <vitaly@namesys.com>,
	reiserfs-list@namesys.com, bug-parted@gnu.org
Subject: Re: Gnu Parted error, Kernel Panic, Reiserfs
Date: Fri, 16 Apr 2004 16:05:27 +0300	[thread overview]
Message-ID: <1082120726.7920.17.camel@firefly> (raw)
In-Reply-To: <Pine.LNX.4.21.0404161253580.16938-100000@mlf.linux.rulez.org>

On Fri, 2004-04-16 at 14:30, Szakacsits Szabolcs wrote:
> On Fri, 16 Apr 2004, Yury Umanets wrote:
> > 
> > I'm on-like, but seems missed mentioned emails somehow.
> 
> You answered the same. I've never resized reiserfs so I don't know. I'm
> just seeing the issue popping up everywhere. 

What I can say, I have not seen any details, sorry. Only claims that
there are some problems and promises that details will be presented. 

But anyway, send me emails to the following addresses:

umka@namesys.com
umka@clustrefs.com
torque@ukrpost.ent

I hope at least one should be okay..

>  
> > But I should say that I have no any details and moreover, nobody was so
> > kind to send me at least something but complains that it does not work,
> 
> Is there a test suite for resizing?


>  Just running it against recent/latest
> reiserfs could catch potential compatibility problems.

reiserfsck is perfect tool for testing resizer ;)

Yes, but I can't reproduce the problem. I tried to do it recently.
Problem may be the following:

(1) inconsistent reiserfs partition _before_ resizing. This will lead
for errors for sure. It is due the fact, that progsreiserfs resize code
has no code for checking tree validness before resizing. 

But at least node levels should be checked, because they are used for
making decision what to do on particular level of tree. And if node
level invalid resizer will get out with complains that invalid node is
found and tree will be left in invalid state.

So, as you can see, this is quite possible to be stumbled here.

(2) it is quite possible that there is a bug in resizer code, which can
only be visible on _big_ partitions. And I have not such a big
partitions to check it.


> 
> > What I need to know is the following:
> > 
> > (1) was reiserfs consistent before resizing or not (what last reiserfsck
> > said before resizing).
> 
> Running fsck before resizing is a very good practice. But most people
> don't do it so some consistent check should be done in the resizer. IMHO
> at least you must check you can account for all blocks in use. If not then
> you must decline to resize otherwise you will probably trash the fs.
Of course there are some checks. But they are not sufficient. They only
check if filesystem is consistent or not (flag in super block).

> 
> ntfsresize had this from day 0 and caught a lot of inconsistent
> filesystems (and bugs during development) and refused to resize. 
> After people run chkdsk everything worked fine.
> 
> Partition Magic didn't have this or it's optional (I don't have it but
> people said so) and I suspect that's one of the reasons it also trashes
> filesystems occasionally.
> 
> > (2) what version of reiserfs was used.
> > (3) sequence of actions or/and detailed description of what was done.
> > (4) is it visible using both parted and qtparted?
> 
> I've seen it reported with both tools. E.g. here are some qtparted ones.
> 
> 	http://qtparted.sourceforge.net/forums/viewtopic.php?t=62
> 
>     Szaka
Okay, I see what we (me) have to do. First of all I will add some code,
which will check tree for consistency. This is very simple check, not so
sophisticated as reiserfsck does. I will only check node levels, node
pointer items and probably validness of indirect items.

And if check will not pass user will be asked to run fsck first.
Actually this functionality already exists in parted. Only thing to be
done is tree check. 

Will do it in week and then will see what happen. Okay?

Thanks.

-- 
umka


  reply	other threads:[~2004-04-16 13:05 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-04-10 20:01 Gnu Parted error, Kernel Panic, Reiserfs Alan Chu
2004-04-10 22:32 ` Rudy L. Zijlstra
2004-04-10 23:16   ` Alan Chu
2004-04-11  0:11     ` Rudy L. Zijlstra
2004-04-12  9:06 ` Vitaly Fertman
2004-04-16  6:05   ` Alan Chu
2004-04-16  8:57     ` Szakacsits Szabolcs
2004-04-16 10:00       ` Yury Umanets
2004-04-16 11:30         ` Szakacsits Szabolcs
2004-04-16 13:05           ` Yury Umanets [this message]
2004-04-16 13:13             ` Yury Umanets
2004-04-16 14:06               ` Szakacsits Szabolcs
2004-04-16 14:14                 ` Yury Umanets
2004-04-16 14:24                   ` Jason Stubbs
2004-04-19 11:38                 ` Yury Umanets
2004-04-19 12:23                   ` Jason Stubbs
2004-04-16 13:23             ` Yury Umanets
2004-04-16 13:49             ` Yury Umanets

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=1082120726.7920.17.camel@firefly \
    --to=umka@namesys.com \
    --cc=alanchu@MIT.EDU \
    --cc=bug-parted@gnu.org \
    --cc=reiserfs-list@namesys.com \
    --cc=szaka@sienet.hu \
    --cc=vitaly@namesys.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.