public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: "H. Peter Anvin" <hpa@zytor.com>
To: David Miller <davem@davemloft.net>
Cc: anton@samba.org, hch@infradead.org, arjan@infradead.org,
	zippel@linux-m68k.org, dmlb2000@gmail.com,
	linux-kernel@vger.kernel.org, gdb@sourceware.org
Subject: Re: PAGE_SIZE Availability Inconsistency
Date: Thu, 08 Mar 2007 20:31:05 -0800	[thread overview]
Message-ID: <45F0E309.70403@zytor.com> (raw)
In-Reply-To: <20070308.202718.68040662.davem@davemloft.net>

David Miller wrote:
> From: "H. Peter Anvin" <hpa@zytor.com>
> Date: Thu, 08 Mar 2007 20:18:28 -0800
> 
>> Anton Blanchard wrote:
>>> The other option is to create a v3 swap format that doesnt use any
>>> PAGE_SIZE parameters.
>> The best thing to do would be to look for the magic both at PAGE_SIZE 
>> (for compatibility) and MIN_PAGE_SIZE (for sanity.)
> 
> That might work, but a large part of me says to go for v3
> and do it cleanly.

The advantage would be that it wouldn't require a v3 for platforms for 
which MIN_PAGE_SIZE == PAGE_SIZE, which accounts for a very large 
percentage of systems.

You still have to look for the darn magic in two places, so there is no 
reason for it to be different.

	-hpa

  reply	other threads:[~2007-03-09  4:32 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-03-05 23:55 PAGE_SIZE Availability Inconsistency David Brown
2007-03-05 23:59 ` Eric Dumazet
2007-03-06  0:01 ` Randy Dunlap
2007-03-06  0:03 ` David Miller
2007-03-06  0:04   ` David Brown
2007-03-06  0:26     ` David Miller
2007-03-06  2:21       ` H. Peter Anvin
2007-03-08 21:08         ` Avi Kivity
2007-03-08 22:21           ` H. Peter Anvin
2007-03-19 19:39             ` Eric W. Biederman
2007-03-06  9:29 ` Christoph Hellwig
2007-03-08  2:18   ` Roman Zippel
2007-03-08  5:28     ` David Brown
2007-03-08  8:32       ` Christoph Hellwig
2007-03-08  9:00     ` Christoph Hellwig
2007-03-08 15:53       ` Arjan van de Ven
2007-03-08 16:08         ` Christoph Hellwig
2007-03-08 16:21           ` Daniel Jacobowitz
2007-03-08 17:05           ` H. Peter Anvin
2007-03-08 17:12             ` Christoph Hellwig
2007-03-08 17:57             ` Anton Blanchard
2007-03-08 18:04               ` H. Peter Anvin
2007-03-08 21:42                 ` Anton Blanchard
2007-03-08 21:46                   ` Anton Blanchard
2007-03-08 21:48                   ` David Miller
2007-03-09  2:43                     ` Anton Blanchard
2007-03-09  4:18                       ` H. Peter Anvin
2007-03-09  4:27                         ` David Miller
2007-03-09  4:31                           ` H. Peter Anvin [this message]
2007-03-09  4:36                             ` David Miller
2007-03-21  2:12                             ` Anton Blanchard
2007-03-21  2:48                               ` H. Peter Anvin
2007-03-08 22:22                   ` H. Peter Anvin
2007-03-08 21:03               ` David Brown

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=45F0E309.70403@zytor.com \
    --to=hpa@zytor.com \
    --cc=anton@samba.org \
    --cc=arjan@infradead.org \
    --cc=davem@davemloft.net \
    --cc=dmlb2000@gmail.com \
    --cc=gdb@sourceware.org \
    --cc=hch@infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=zippel@linux-m68k.org \
    /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