All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arnd Bergmann <arnd@arndb.de>
To: Minchan Kim <minchan@kernel.org>
Cc: 정효진 <syr.jeong@samsung.com>,
	"'Alex Lemberg'" <Alex.Lemberg@sandisk.com>,
	linaro-kernel@lists.linaro.org,
	"'Rik van Riel'" <riel@redhat.com>,
	linux-mmc@vger.kernel.org, linux-kernel@vger.kernel.org,
	"'Luca Porzio (lporzio)'" <lporzio@micron.com>,
	linux-mm@kvack.org, kernel-team@android.com,
	"'Yejin Moon'" <yejin.moon@samsung.com>,
	"'Hugh Dickins'" <hughd@google.com>,
	"'Yaniv Iarovici'" <Yaniv.Iarovici@sandisk.com>,
	cpgs@samsung.com
Subject: Re: swap on eMMC and other flash
Date: Mon, 9 Apr 2012 13:00:34 +0000	[thread overview]
Message-ID: <201204091300.34304.arnd@arndb.de> (raw)
In-Reply-To: <4F8299B4.5090909@kernel.org>

On Monday 09 April 2012, Minchan Kim wrote:
> > 
> > Regarding swap page size:
> > Actually, I can't guarantee the optimal size of different eMMC in the industry, because it depends on NAND page size an firmware implementation inside eMMC. In case of SAMSUNG eMMC, 8KB page size and 512KB block size(erase unit) is current implementation.
> > I think that the multiple of 8KB page size align with 512KB is good for SAMSUNG eMMC.
> > If swap system use 512KB page and issue Discard/Trim align with 512KB, eMMC make best performance as of today. However, large page size in swap partition may not best way in Linux system level.
> > I'm not sure that the best page size between Swap system and eMMC device.
> 
> 
> The variety is one of challenges for removing GC generally. ;-(.
> I don't like manual setting through /sys/block/xxx because it requires
> that user have to know nand page size and erase block size but it's not
> easy to know to normal user.
> Arnd. What's your plan to support various flash storages effectively?

My preference would be to build the logic to detect the sizes into mkfs
and mkswap and encode them in the superblock in new fields. I don't think
we can trust any data that a device reports right now because operating
systems have ignored it in the past and either someone has forgotten to
update the fields after moving to new technology (eMMC), or the data can
not be encoded correctly according to the spec (SD, USB).

System builders for embedded systems can then make sure that they get
it right for the hardware they use, and we can try our best to help
that process.

	Ard

WARNING: multiple messages have this Message-ID (diff)
From: Arnd Bergmann <arnd@arndb.de>
To: Minchan Kim <minchan@kernel.org>
Cc: 정효진 <syr.jeong@samsung.com>,
	"'Alex Lemberg'" <Alex.Lemberg@sandisk.com>,
	linaro-kernel@lists.linaro.org,
	"'Rik van Riel'" <riel@redhat.com>,
	linux-mmc@vger.kernel.org, linux-kernel@vger.kernel.org,
	"'Luca Porzio (lporzio)'" <lporzio@micron.com>,
	linux-mm@kvack.org, kernel-team@android.com,
	"'Yejin Moon'" <yejin.moon@samsung.com>,
	"'Hugh Dickins'" <hughd@google.com>,
	"'Yaniv Iarovici'" <Yaniv.Iarovici@sandisk.com>,
	cpgs@samsung.com
Subject: Re: swap on eMMC and other flash
Date: Mon, 9 Apr 2012 13:00:34 +0000	[thread overview]
Message-ID: <201204091300.34304.arnd@arndb.de> (raw)
In-Reply-To: <4F8299B4.5090909@kernel.org>

On Monday 09 April 2012, Minchan Kim wrote:
> > 
> > Regarding swap page size:
> > Actually, I can't guarantee the optimal size of different eMMC in the industry, because it depends on NAND page size an firmware implementation inside eMMC. In case of SAMSUNG eMMC, 8KB page size and 512KB block size(erase unit) is current implementation.
> > I think that the multiple of 8KB page size align with 512KB is good for SAMSUNG eMMC.
> > If swap system use 512KB page and issue Discard/Trim align with 512KB, eMMC make best performance as of today. However, large page size in swap partition may not best way in Linux system level.
> > I'm not sure that the best page size between Swap system and eMMC device.
> 
> 
> The variety is one of challenges for removing GC generally. ;-(.
> I don't like manual setting through /sys/block/xxx because it requires
> that user have to know nand page size and erase block size but it's not
> easy to know to normal user.
> Arnd. What's your plan to support various flash storages effectively?

My preference would be to build the logic to detect the sizes into mkfs
and mkswap and encode them in the superblock in new fields. I don't think
we can trust any data that a device reports right now because operating
systems have ignored it in the past and either someone has forgotten to
update the fields after moving to new technology (eMMC), or the data can
not be encoded correctly according to the spec (SD, USB).

System builders for embedded systems can then make sure that they get
it right for the hardware they use, and we can try our best to help
that process.

	Ard

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

  reply	other threads:[~2012-04-09 13:00 UTC|newest]

Thread overview: 89+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-30 17:44 swap on eMMC and other flash Arnd Bergmann
2012-03-30 17:44 ` Arnd Bergmann
2012-03-30 18:50 ` Arnd Bergmann
2012-03-30 18:50   ` Arnd Bergmann
2012-03-30 22:08   ` Zach Pfeffer
2012-03-30 22:08     ` Zach Pfeffer
2012-03-31  9:24     ` Arnd Bergmann
2012-03-31  9:24       ` Arnd Bergmann
2012-04-03 18:17       ` Zach Pfeffer
2012-04-03 18:17         ` Zach Pfeffer
2012-03-31 20:29   ` Hugh Dickins
2012-03-31 20:29     ` Hugh Dickins
2012-03-31 20:29     ` Hugh Dickins
2012-04-02 11:45     ` Arnd Bergmann
2012-04-02 11:45       ` Arnd Bergmann
2012-04-02 14:41       ` Hugh Dickins
2012-04-02 14:41         ` Hugh Dickins
2012-04-02 14:55         ` Arnd Bergmann
2012-04-02 14:55           ` Arnd Bergmann
2012-04-05  0:17           ` 정효진
2012-04-05  0:17             ` 정효진
2012-04-09 12:50             ` Arnd Bergmann
2012-04-09 12:50               ` Arnd Bergmann
2012-04-08 13:50           ` Alex Lemberg
2012-04-08 13:50             ` Alex Lemberg
2012-04-09  2:14             ` Minchan Kim
2012-04-09  2:14               ` Minchan Kim
2012-04-09  7:37               ` 정효진
2012-04-09  7:37                 ` 정효진
2012-04-09  8:11                 ` Minchan Kim
2012-04-09  8:11                   ` Minchan Kim
2012-04-09  8:11                   ` Minchan Kim
2012-04-09 13:00                   ` Arnd Bergmann [this message]
2012-04-09 13:00                     ` Arnd Bergmann
2012-04-10  1:10                     ` Minchan Kim
2012-04-10  1:10                       ` Minchan Kim
2012-04-10  8:40                       ` Arnd Bergmann
2012-04-10  8:40                         ` Arnd Bergmann
2012-04-12  8:32                         ` Luca Porzio (lporzio)
2012-04-12  8:32                           ` Luca Porzio (lporzio)
2012-04-09 12:54                 ` Arnd Bergmann
2012-04-09 12:54                   ` Arnd Bergmann
2012-04-02 12:52     ` Luca Porzio (lporzio)
2012-04-02 12:52       ` Luca Porzio (lporzio)
2012-04-02 14:58       ` Hugh Dickins
2012-04-02 14:58         ` Hugh Dickins
2012-04-02 16:51         ` Rik van Riel
2012-04-02 16:51           ` Rik van Riel
2012-04-04 12:21   ` Adrian Hunter
2012-04-04 12:21     ` Adrian Hunter
2012-04-04 12:47     ` Arnd Bergmann
2012-04-04 12:47       ` Arnd Bergmann
2012-04-11 10:28       ` Adrian Hunter
2012-04-11 10:28         ` Adrian Hunter
2012-07-16 13:29         ` Pavel Machek
2012-07-16 13:29           ` Pavel Machek
2012-04-06  7:15 ` Minchan Kim
2012-04-06 16:16   ` Arnd Bergmann
2012-04-06 16:16     ` Arnd Bergmann
2012-04-09  2:06     ` Minchan Kim
2012-04-09  2:06       ` Minchan Kim
2012-04-09  2:06       ` Minchan Kim
2012-04-09 12:35       ` Arnd Bergmann
2012-04-09 12:35         ` Arnd Bergmann
2012-04-09 12:35         ` Arnd Bergmann
2012-04-10  0:57         ` Minchan Kim
2012-04-10  0:57           ` Minchan Kim
2012-04-10  0:57           ` Minchan Kim
2012-04-10  8:32           ` Arnd Bergmann
2012-04-10  8:32             ` Arnd Bergmann
2012-04-10  8:32             ` Arnd Bergmann
2012-04-11  9:54             ` Minchan Kim
2012-04-11  9:54               ` Minchan Kim
2012-04-11 15:57               ` Arnd Bergmann
2012-04-11 15:57                 ` Arnd Bergmann
2012-04-12  2:36                 ` Minchan Kim
2012-04-12  2:36                   ` Minchan Kim
2012-04-16 18:22                 ` Stephan Uphoff
2012-04-16 18:22                   ` Stephan Uphoff
2012-04-16 18:59                   ` Arnd Bergmann
2012-04-16 18:59                     ` Arnd Bergmann
2012-04-16 21:12                     ` Stephan Uphoff
2012-04-16 21:12                       ` Stephan Uphoff
2012-04-17  2:18                       ` Minchan Kim
2012-04-17  2:18                         ` Minchan Kim
2012-04-17  2:05                     ` Minchan Kim
2012-04-17  2:05                       ` Minchan Kim
2012-04-27  7:34                   ` Luca Porzio (lporzio)
2012-04-27  7:34                     ` Luca Porzio (lporzio)

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=201204091300.34304.arnd@arndb.de \
    --to=arnd@arndb.de \
    --cc=Alex.Lemberg@sandisk.com \
    --cc=Yaniv.Iarovici@sandisk.com \
    --cc=cpgs@samsung.com \
    --cc=hughd@google.com \
    --cc=kernel-team@android.com \
    --cc=linaro-kernel@lists.linaro.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=linux-mmc@vger.kernel.org \
    --cc=lporzio@micron.com \
    --cc=minchan@kernel.org \
    --cc=riel@redhat.com \
    --cc=syr.jeong@samsung.com \
    --cc=yejin.moon@samsung.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.