linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Arnd Bergmann <arnd@arndb.de>
To: Hans Ulli Kroll <ulli.kroll@googlemail.com>
Cc: Catalin Marinas <catalin.marinas@arm.com>,
	linux-arm-kernel@lists.infradead.org,
	Linus Torvalds <torvalds@linux-foundation.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Will Deacon <will.deacon@arm.com>
Subject: Re: [GIT PULL] arm64 updates for 4.4
Date: Sat, 07 Nov 2015 20:23:24 +0100	[thread overview]
Message-ID: <5669629.gUNGHpWjD7@wuerfel> (raw)
In-Reply-To: <alpine.LNX.2.00.1511071112380.9964@T520.local>

On Saturday 07 November 2015 11:56:44 Hans Ulli Kroll wrote:
> On Fri, 6 Nov 2015, Arnd Bergmann wrote:
> > On Friday 06 November 2015 16:04:08 Catalin Marinas wrote:
> > > On Fri, Nov 06, 2015 at 10:57:58AM +0100, Arnd Bergmann wrote:
> > > > On Thursday 05 November 2015 18:27:18 Catalin Marinas wrote:
> > > > > On Wed, Nov 04, 2015 at 02:55:01PM -0800, Linus Torvalds wrote:
> > > > > > On Wed, Nov 4, 2015 at 10:25 AM, Catalin Marinas <catalin.marinas@arm.com> wrote:
> > > > > > It's good for single-process loads - if you do a lot of big fortran
> > > > > > jobs, or a lot of big database loads, and nothing else, you're fine.
> > > > > 
> > > > > These are some of the arguments from the server camp: specific
> > > > > workloads.
> > > > 
> > > > I think (a little overgeneralized), you want 4KB pages for any file
> > > > based mappings,
> > > 
> > > In general, yes, but if the main/only workload on your server is mapping
> > > large db files, the memory usage cost may be amortised.
> > 
> > This will still only do you good for a database that is read into memory
> > once and not written much, and at that point you can as well use hugepages.
> > 
> > The problems for using 64kb page cache on file mappings are
> > 
> > - while you normally want some readahead, the larger pages also result
> >   in read-behind, so you have to actually transfer data from disk into
> >   RAM without ever accessing it.
> > 
> > - When you write the data, you have to write the full 64K page because
> >   that is the granularity of your dirty bit tracking.
> > 
> > So even if you don't care at all about memory consumption, you are
> > still transferring several times more data to and from your drives.
> > As mentioned that can be a win on some storage devices, but usually
> > it's a loss.
> > 
> 
> there is also a maybe a bigger problem.
> I know this from my Zyxel NAS540, this thing is build around the Mindspeed 
> Comcerto 2000 SoC
> 
> Zyxel is currently rolling back to support 4k page sizeses in upcommig 
> 5.10 firmware release, because Minspeed did some stupid thing :
> 
> It's not possible to use some standard ARMv7 toolchain and build your 
> own/needed userspace tools.
> 
> And this in change which causes the pain 
> 
> diff --git a/arch/arm/include/asm/elf.h b/arch/arm/include/asm/elf.h
> -#define ELF_EXEC_PAGESIZE      4096
> +#define ELF_EXEC_PAGESIZE      (PAGE_SIZE)

In ARM32 binutils, ELF_MAXPAGESIZE was changed last year to 64KB, so
binutils-2.25 or higher should support this by default, as long as you
recompile all user binaries.

> The SoC is mostly build from off the shelf IP's
> SATA, NAND, SPI and so on
> The only thing which is completly braindead is MAC
> It's using some kind of VLAN tagging to support tree ports,
> only one descriptor chain for all three interfaces.

You mean they used 64KB logical page sizes to work around a broken
ethernet MAC?

	Arnd

  reply	other threads:[~2015-11-07 19:24 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-04 18:25 [GIT PULL] arm64 updates for 4.4 Catalin Marinas
2015-11-04 22:55 ` Linus Torvalds
2015-11-05 18:27   ` Catalin Marinas
2015-11-06  9:57     ` Arnd Bergmann
2015-11-06 16:04       ` Catalin Marinas
2015-11-06 16:23         ` Arnd Bergmann
2015-11-07 10:56           ` Hans Ulli Kroll
2015-11-07 19:23             ` Arnd Bergmann [this message]
2015-11-09 18:40               ` Hans Ulli Kroll
2015-11-25  4:58         ` Jon Masters

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=5669629.gUNGHpWjD7@wuerfel \
    --to=arnd@arndb.de \
    --cc=catalin.marinas@arm.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=torvalds@linux-foundation.org \
    --cc=ulli.kroll@googlemail.com \
    --cc=will.deacon@arm.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).