Re: [PATCH v3 1/3] riscv: Set more data to cacheinfo

From: Zong Li
Date: Mon Aug 31 2020 - 03:14:47 EST


On Mon, Aug 31, 2020 at 3:00 PM Pekka Enberg <penberg@xxxxxxxxx> wrote:
>
> Hi,
>
> On Mon, Aug 31, 2020 at 9:15 AM Zong Li <zong.li@xxxxxxxxxx> wrote:
> > If the sets is one, it means that the cache is fully associative, then
> > we don't need to fill the ways number, just keep way number as zero,
> > so here we want to find the fully associative case first and make the
> > if expression fail at the beginning. We might also rewrite it as
> > follow:
>
> [snip]
>
> Thanks for the explanation! The rewritten version is much easier to
> read, at least to me.
>

Let me change it for readability in the next version, thanks.

> - Pekka