All of lore.kernel.org
 help / color / mirror / Atom feed
From: Piotr Neuman <sikkh@wp.pl>
To: Clifford Beshers <clifford.beshers@linspireinc.com>
Cc: Alex Zarochentsev <zam@namesys.com>, reiserfs-list@namesys.com
Subject: Re: Reiser4 journal replay stats
Date: Wed, 20 Oct 2004 23:31:08 +0200	[thread overview]
Message-ID: <200410202331.08984.sikkh@wp.pl> (raw)
In-Reply-To: <41757DD8.5020105@linspireinc.com>

> Hans Reiser wrote:
> > Piotr Neuman wrote:
> >> Hi
> >>
> >> This small patch is will print out nuber of transartions replied from
> >> logs after a crash, quite like reiserfs 3.6 does:
> >>
> >> http://kem.p.lodz.pl/~peter/reiser4_replay_stats.patch
> >>
> >> Thanks to Nikita Danilov for guidance.
> >
> > Do the members of the list view this as useful?  I worry that to
> > average persons, who don't know what a transaction is, this is vaguely
> > scary.
> >
> > Clifford, I cc'd you because your intuition is better than mine for this.
> >
> > Perhaps it should say something reassuring, like "All went well in the
> > filesystem aspect of recovering from what seems to have been an
> > improperly shutdown or crashed system."

[cut]

> I don't know if this text already exists, but I would make sure there
> are messages that indicate:
>
>     * that the file system was improperly shutdown;
>     * that an improper shutdown is no big deal for Reiser4 and we'll
>       happily clean things up;
>     * give an indication that no data was lost (if you can determine
>       that) and that all is well with the world.
>
> Probably a longer answer than you wanted, but I'm not going to take the
> time to edit it down. ;-)

Updated patch to reflect above suggestions. There is only one technical thing, 
since reiser4 uses wandering logs I find it very hard to crash in such a way 
that it will reply even a single transaction, so I don't really know if this 
patch is usefull ;-).

Download:

http://kem.p.lodz.pl/~peter/reiser4_replay_stats-2.patch

Regards

Piotr Neuman

      parent reply	other threads:[~2004-10-20 21:31 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-10-16 20:09 Reiser4 journal replay stats Piotr Neuman
2004-10-19 20:21 ` Hans Reiser
     [not found]   ` <41757DD8.5020105@linspireinc.com>
2004-10-20 21:31     ` Piotr Neuman [this message]

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=200410202331.08984.sikkh@wp.pl \
    --to=sikkh@wp.pl \
    --cc=clifford.beshers@linspireinc.com \
    --cc=reiserfs-list@namesys.com \
    --cc=zam@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.