public inbox for linux-pm@vger.kernel.org
 help / color / mirror / Atom feed
From: "Rafael J. Wysocki" <rjw@sisk.pl>
To: Jiri Slaby <jirislaby@gmail.com>
Cc: linux-pm@lists.linux-foundation.org,
	Nigel Cunningham <ncunningham@crca.org.au>
Subject: Re: [RFC 09/15] PM / Hibernate: user, implement user_ops writer
Date: Thu, 22 Apr 2010 05:43:48 +0200	[thread overview]
Message-ID: <201004220543.48943.rjw@sisk.pl> (raw)
In-Reply-To: <4BCF6C7F.7020807@gmail.com>

On Wednesday 21 April 2010, Jiri Slaby wrote:
> On 03/31/2010 10:29 PM, Rafael J. Wysocki wrote:
> > On Wednesday 31 March 2010, Jiri Slaby wrote:
> >> On 03/30/2010 10:50 PM, Rafael J. Wysocki wrote:
> >>> So, I definitely would like to see numbers.
> >>
> >> With the patch attached, I get similar results for
> >> uncompressed image -> user.c -> s2disk (compress+threads)
> >> compressed image -> user.c -> s2disk (none of compress+threads)
> >>
> >> BUT note that there are differences in pages _copied_ in the "user.c ->
> >> s2disk" phase. Compression was about 40 %. So in the former case 100000
> >> pages went through and in the latter one only ~ 38000. So presumably
> >> there will be savings if the pages are compressed in the kernel in the
> >> save-image phase instead of in userspace.
> >>
> >> I'll test (after I hack that up) also the case of image compression when
> >> saving the image and let you know.
> >>
> >> It was tested on a machine with openSUSE factory with init 5 with KDE 4.
> >> The memory consumption was ~ 400M (100000 pages) after boot every time.
> >> Hibernation lasted 12-13s in both cases.
> > 
> > So that's pretty much the same and I don't think feeding compressed images to
> > s2disk is not worth the effort.  Let's assume s2disk will always receive a raw
> > image (that will make it backwards compatible too).
> 
> Hi.
> 
> Yes, I did it solely for comparison purposes.
> 
> Now, with the patch attached (depending on either COMPRESS_IMAGE or
> COMPRESS_IMAGE1 defined) I tried to compare compression while doing
> atomic copy with compression done when image storage takes place. The
> latter outperformed the former a bit. I did 8 measurements with the
> setup same as above. The average for them is 9684.32 and 10855.07 stored
> raw (uncompressed) pages per second. That is 4.5M/s more.
> 
> There is a sheet with the numbers at:
> http://spreadsheets.google.com/ccc?key=0AvVn7xYA1DnodHFFYzg3Y2tpMUl5NFlURXRtR0xQdmc&hl=en

Thanks for doing that work!

So clearly from performance POV it is better to compress while saving.

Rafael

  reply	other threads:[~2010-04-22  3:43 UTC|newest]

Thread overview: 88+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-03-23 16:17 [RFC 01/15] FS: libfs, implement simple_write_to_buffer Jiri Slaby
2010-03-23 16:17 ` [RFC 02/15] PM / Hibernate: snapshot cleanup Jiri Slaby
2010-03-23 16:17 ` [RFC 03/15] PM / Hibernate: separate block_io Jiri Slaby
2010-03-23 16:17 ` [RFC 04/15] PM / Hibernate: move the first_sector out of swsusp_write Jiri Slaby
2010-03-23 16:17 ` [RFC 05/15] PM / Hibernate: group swap ops Jiri Slaby
2010-03-23 16:17 ` [RFC 06/15] PM / Hibernate: swap, remove swap_map_handle usages Jiri Slaby
2010-03-23 16:17 ` [RFC 07/15] PM / Hibernate: add sws_modules_ops Jiri Slaby
2010-03-23 16:17 ` [RFC 08/15] PM / Hibernate: add user module_ops Jiri Slaby
2010-03-25 22:07   ` Rafael J. Wysocki
     [not found]   ` <201003252307.15232.rjw@sisk.pl>
2010-03-26  9:43     ` Jiri Slaby
2010-03-23 16:17 ` [RFC 09/15] PM / Hibernate: user, implement user_ops writer Jiri Slaby
2010-03-23 16:17 ` [RFC 10/15] PM / Hibernate: user, implement user_ops reader Jiri Slaby
2010-03-23 16:17 ` [RFC 11/15] PM / Hibernate: add chunk i/o support Jiri Slaby
2010-03-23 16:17 ` [RFC 12/15] PM / Hibernate: split snapshot_read_next Jiri Slaby
2010-03-23 16:17 ` [RFC 13/15] PM / Hibernate: split snapshot_write_next Jiri Slaby
2010-03-23 16:17 ` [RFC 14/15] PM / Hibernate: dealign swsusp_info Jiri Slaby
2010-03-23 16:17 ` [RFC 15/15] PM / Hibernate: move non-swap code to snapshot.c Jiri Slaby
2010-03-23 21:51 ` [RFC 01/15] FS: libfs, implement simple_write_to_buffer Rafael J. Wysocki
2010-03-23 22:09 ` Nigel Cunningham
     [not found] ` <1269361063-3341-2-git-send-email-jslaby@suse.cz>
2010-03-24 20:29   ` [RFC 02/15] PM / Hibernate: snapshot cleanup Pavel Machek
2010-03-24 22:35   ` Rafael J. Wysocki
2010-03-25  5:29   ` Pavel Machek
     [not found] ` <1269361063-3341-3-git-send-email-jslaby@suse.cz>
2010-03-24 20:30   ` [RFC 03/15] PM / Hibernate: separate block_io Pavel Machek
     [not found]   ` <20100324203021.GE5798@elf.ucw.cz>
2010-03-24 21:22     ` Jiri Slaby
     [not found]     ` <4BAA829F.7060207@gmail.com>
2010-03-24 22:58       ` Nigel Cunningham
     [not found]       ` <4BAA98FC.9040302@crca.org.au>
2010-03-25  2:35         ` Nigel Cunningham
2010-03-25 14:29         ` Pavel Machek
     [not found]         ` <4BAACC0A.2040009@crca.org.au>
2010-03-25 20:12           ` Rafael J. Wysocki
     [not found]           ` <201003252112.25204.rjw@sisk.pl>
2010-03-25 20:13             ` Nigel Cunningham
     [not found]             ` <4BABC3D1.1050104@crca.org.au>
2010-03-25 20:33               ` Rafael J. Wysocki
     [not found]               ` <201003252133.50206.rjw@sisk.pl>
2010-03-25 20:36                 ` Nigel Cunningham
2010-03-29 13:30             ` Pavel Machek
     [not found] ` <1269361063-3341-4-git-send-email-jslaby@suse.cz>
2010-03-24 20:31   ` [RFC 04/15] PM / Hibernate: move the first_sector out of swsusp_write Pavel Machek
     [not found]   ` <20100324203125.GF5798@elf.ucw.cz>
2010-03-25 21:26     ` Rafael J. Wysocki
     [not found] ` <1269361063-3341-6-git-send-email-jslaby@suse.cz>
2010-03-24 20:33   ` [RFC 06/15] PM / Hibernate: swap, remove swap_map_handle usages Pavel Machek
     [not found]   ` <20100324203329.GG5798@elf.ucw.cz>
2010-03-24 21:29     ` Jiri Slaby
     [not found]     ` <4BAA842A.6060906@gmail.com>
2010-03-25 21:35       ` Rafael J. Wysocki
     [not found] ` <1269361063-3341-9-git-send-email-jslaby@suse.cz>
2010-03-24 20:42   ` [RFC 09/15] PM / Hibernate: user, implement user_ops writer Pavel Machek
     [not found]   ` <20100324204259.GA6423@elf.ucw.cz>
2010-03-24 21:40     ` Jiri Slaby
     [not found]     ` <4BAA86E8.5090108@gmail.com>
2010-03-25 21:36       ` Pavel Machek
2010-03-25 22:14       ` Rafael J. Wysocki
     [not found]       ` <201003252314.33256.rjw@sisk.pl>
2010-03-26  9:34         ` Jiri Slaby
     [not found]         ` <4BAC7FC3.50405@gmail.com>
2010-03-26 22:04           ` Rafael J. Wysocki
2010-03-27  7:02           ` Pavel Machek
     [not found]           ` <201003262304.32118.rjw@sisk.pl>
2010-03-29 15:33             ` Jiri Slaby
     [not found]             ` <4BB0C840.7030802@gmail.com>
2010-03-29 22:09               ` Rafael J. Wysocki
2010-03-30  1:14                 ` Nigel Cunningham
2010-03-30 21:03                   ` Rafael J. Wysocki
2010-03-31  2:31                     ` Nigel Cunningham
2010-03-31 14:47                       ` Pavel Machek
2010-03-31 21:01                         ` Nigel Cunningham
2010-03-31 20:25                       ` Rafael J. Wysocki
2010-03-31 21:06                         ` Nigel Cunningham
2010-03-31 21:36                           ` Rafael J. Wysocki
2010-04-04 23:08                             ` Nigel Cunningham
2010-04-04 23:13                               ` Rafael J. Wysocki
2010-03-31 21:54                           ` Nigel Cunningham
2010-03-30  8:59                 ` Jiri Slaby
2010-03-30 20:50                   ` Rafael J. Wysocki
2010-03-31 14:41                     ` Jiri Slaby
2010-03-31 20:29                       ` Rafael J. Wysocki
2010-04-21 21:22                         ` Jiri Slaby
2010-04-22  3:43                           ` Rafael J. Wysocki [this message]
2010-03-24 22:13 ` [RFC 01/15] FS: libfs, implement simple_write_to_buffer Rafael J. Wysocki
     [not found] ` <1269361063-3341-10-git-send-email-jslaby@suse.cz>
2010-03-25  5:30   ` what the patches do Re: [RFC 10/15] PM / Hibernate: user, implement user_ops reader Pavel Machek
     [not found]   ` <20100325053003.GB12935@elf.ucw.cz>
2010-03-25  5:42     ` Nigel Cunningham
     [not found]     ` <4BAAF7DA.30506@crca.org.au>
2010-03-25  6:12       ` Nigel Cunningham
2010-03-25 20:14       ` Rafael J. Wysocki
     [not found]       ` <201003252114.36605.rjw@sisk.pl>
2010-03-25 20:21         ` Nigel Cunningham
     [not found]         ` <4BABC5B6.2070301@crca.org.au>
2010-03-25 20:29           ` Rafael J. Wysocki
     [not found]           ` <201003252129.10224.rjw@sisk.pl>
2010-03-25 20:35             ` Nigel Cunningham
     [not found]             ` <4BABC926.8060203@crca.org.au>
2010-03-25 21:13               ` Rafael J. Wysocki
2010-03-25 21:26             ` Pavel Machek
     [not found]             ` <20100325212656.GD22902@elf.ucw.cz>
2010-03-25 21:49               ` Nigel Cunningham
     [not found]               ` <4BABDA59.8070107@crca.org.au>
2010-04-02  6:36                 ` Pavel Machek
     [not found]                 ` <20100402063644.GA14941@atrey.karlin.mff.cuni.cz>
2010-04-02 17:03                   ` Rafael J. Wysocki
2010-03-25 22:21     ` Rafael J. Wysocki
     [not found] ` <1269361063-3341-5-git-send-email-jslaby@suse.cz>
2010-03-25 21:28   ` [RFC 05/15] PM / Hibernate: group swap ops Rafael J. Wysocki
     [not found] ` <1269361063-3341-7-git-send-email-jslaby@suse.cz>
2010-03-24 20:36   ` [RFC 07/15] PM / Hibernate: add sws_modules_ops Pavel Machek
     [not found]   ` <20100324203634.GH5798@elf.ucw.cz>
2010-03-24 21:31     ` Jiri Slaby
2010-03-25 22:02   ` Rafael J. Wysocki
     [not found] ` <1269361063-3341-11-git-send-email-jslaby@suse.cz>
2010-03-25 22:38   ` [RFC 11/15] PM / Hibernate: add chunk i/o support Rafael J. Wysocki
     [not found]   ` <201003252338.48513.rjw@sisk.pl>
2010-03-26  9:09     ` Jiri Slaby
     [not found]     ` <4BAC79B4.4040200@gmail.com>
2010-03-26 10:02       ` Nigel Cunningham
     [not found] ` <1269361063-3341-12-git-send-email-jslaby@suse.cz>
2010-03-25 22:44   ` [RFC 12/15] PM / Hibernate: split snapshot_read_next Rafael J. Wysocki
     [not found] ` <1269361063-3341-13-git-send-email-jslaby@suse.cz>
2010-03-25 22:46   ` [RFC 13/15] PM / Hibernate: split snapshot_write_next Rafael J. Wysocki
     [not found] ` <1269361063-3341-14-git-send-email-jslaby@suse.cz>
2010-03-25 22:46   ` [RFC 14/15] PM / Hibernate: dealign swsusp_info Rafael J. Wysocki
     [not found] ` <1269361063-3341-15-git-send-email-jslaby@suse.cz>
2010-03-25 22:50   ` [RFC 15/15] PM / Hibernate: move non-swap code to snapshot.c Rafael J. Wysocki

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=201004220543.48943.rjw@sisk.pl \
    --to=rjw@sisk.pl \
    --cc=jirislaby@gmail.com \
    --cc=linux-pm@lists.linux-foundation.org \
    --cc=ncunningham@crca.org.au \
    /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