All of lore.kernel.org
 help / color / mirror / Atom feed
From: Hans Reiser <reiser@namesys.com>
To: Chris Mason <mason@suse.com>
Cc: berthiaume_wayne@emc.com, reiserfs-list@namesys.com, green@namesys.com
Subject: Re: fsync() Performance Issue
Date: Sat, 04 May 2002 06:05:32 +0400	[thread overview]
Message-ID: <3CD341EC.9080906@namesys.com> (raw)
In-Reply-To: 1020463205.3946.252.camel@tiny

Chris Mason wrote:

>On Fri, 2002-05-03 at 16:35, berthiaume_wayne@emc.com wrote:
>  
>
>>	Chris, I have some quick preliminary results for you. I have
>>additional testing to perform and haven't run debugreiserfs() yet. If you
>>have a preference for which tests to run debugreiserfs() let me know.
>>	Base testing was done against 2.4.13 built on RH 7.1 using the
>>test_writes.c code I forwarded to you. The system is a Tyan with single
>>PIII, IDE Promise 20269, Maxtor 160GB drive - write cache disabled. All
>>numbers are with fsync() and 1KB files. As I said, more testing, i.e.
>>filesizes, need to be performed.
>>    
>>
>
>  
>
>>2.4.19-pre7 speedup, data logging, write barrier / no options
>>	=> 47.1ms/file
>>    
>>
>
>Hi Wayne, thanks for sending these along.
>
>I expected a slight improvement over the 2.4.13 code even with the data
>logging turned off.  I'm curious to see how it does with the IDE cache
>turned on.  With scsi, I see 10-15% better without any options than an
>unpatched kernel.
>
>  
>
>>2.4.19-pre7 speedup, data logging, write barrier / data=journal
>>	=> 25.2ms/file
>>2.4.19-pre7 speedup, data logging, write barrier / data=journal,barrier=none
>>	=> 27.8ms/file
>>    
>>
>
>The barrier option doesn't make much difference because the write cache
>is off.  With write cache on, the barrier code should allow you to be
>faster than with the caching off, but without risking the data (Jens and
>I are working on final fsync safety issues though).
>
>Hans, data=journal turns on the data journaling.  The data journaling
>
and the reason it is faster is....

>patches also include optimizations to write metadata back to disk in
>bigger chunks for tiny transactions (the current method is to write one
>transaction's worth back, when a transaction has 3 blocks, this is
>pretty slow).
>
is this a lazy fsync?  If so, then everything makes sense to me, if not, 
I remain uneducated and looking to receive your wisdom;-)

>
>I've put these patches up on:
>
>ftp.suse.com/pub/people/mason/patches/data-logging
>
>  
>
>>	One question is will these patches be going into the 2.4 tree and
>>when?
>>    
>>
>
>The data logging patches are a huge change, but the good news is they
>are based on the nesting patches that have been stable for a long time
>in the quota code.  I'll probably want a month or more of heavy testing
>before I think about submitting them.
>
>-chris
>
>
>
>
>  
>




  reply	other threads:[~2002-05-04  2:05 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-05-03 20:35 fsync() Performance Issue berthiaume_wayne
2002-05-03 22:00 ` Chris Mason
2002-05-04  2:05   ` Hans Reiser [this message]
2002-05-04  5:41     ` Valdis.Kletnieks
2002-05-04 13:11     ` Chris Mason
2002-05-04 14:59       ` Hans Reiser
2002-05-06 12:40         ` Chris Mason
2002-05-06 13:02           ` Hans Reiser
2002-05-06 21:21           ` Hans Reiser
2002-05-06 22:57             ` Chris Mason
2002-05-06 23:41               ` Hans Reiser
2002-05-07  1:17               ` Manuel Krause
2002-05-07  2:04                 ` Chris Mason
2002-05-07 20:26                   ` Manuel Krause
2002-05-08  1:22                     ` Chris Mason
  -- strict thread matches above, loose matches on Subject: below --
2002-05-06 12:54 berthiaume_wayne
2002-04-30 14:45 berthiaume_wayne
2002-04-29 17:26 berthiaume_wayne
2002-04-26 20:28 berthiaume_wayne
2002-04-29 16:20 ` Russell Coker
2002-04-29 16:30   ` Chris Mason
2002-04-29 16:32   ` Toby Dickenson
2002-04-29 16:45     ` Chris Mason
2002-04-29 17:56     ` Matthias Andree
2002-04-29 18:58       ` Valdis.Kletnieks
2002-04-29 18:56         ` Hans Reiser
2002-04-30 14:20 ` Oleg Drokin
2002-04-30 14:27   ` Chris Mason
2002-05-02  5:07   ` Christian Stuke
2002-05-02  6:20     ` Oleg Drokin

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=3CD341EC.9080906@namesys.com \
    --to=reiser@namesys.com \
    --cc=berthiaume_wayne@emc.com \
    --cc=green@namesys.com \
    --cc=mason@suse.com \
    --cc=reiserfs-list@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.