[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <498nql587Wz9sSJ@ozlabs.org>
Date: Sun, 26 Apr 2020 09:51:15 +1000 (AEST)
From: Michael Ellerman <patch-notifications@...erman.id.au>
To: Chris Packham <chris.packham@...iedtelesis.co.nz>,
benh@...nel.crashing.org, christophe.leroy@....fr,
tglx@...utronix.de, paulus@...ba.org, cai@....pw, oss@...error.net
Cc: Chris Packham <chris.packham@...iedtelesis.co.nz>,
linuxppc-dev@...ts.ozlabs.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3] powerpc/setup_64: Set cache-line-size based on cache-block-size
On Thu, 2020-04-16 at 22:19:08 UTC, Chris Packham wrote:
> If {i,d}-cache-block-size is set and {i,d}-cache-line-size is not, use
> the block-size value for both. Per the devicetree spec cache-line-size
> is only needed if it differs from the block size.
>
> Originally the code would fallback from block size to line size. An
> error message was printed if both properties were missing.
>
> Later the code was refactored to use clearer names and logic but it
> inadvertently made line size a required property. This caused the
> default values to be used and in turn leads to Power9 systems using the
> wrong size.
>
> Fixes: bd067f83b084 ("powerpc/64: Fix naming of cache block vs. cache lin=
> e")
> Signed-off-by: Chris Packham <chris.packham@...iedtelesis.co.nz>
Applied to powerpc fixes, thanks.
https://git.kernel.org/powerpc/c/94c0b013c98583614e1ad911e8795ca36da34a85
cheers
Powered by blists - more mailing lists