All of lore.kernel.org
 help / color / mirror / Atom feed
From: Leonid Yegoshin <Leonid.Yegoshin@imgtec.com>
To: Justin Chen <justinpopo6@gmail.com>, <linux-mips@linux-mips.org>
Cc: <bcm-kernel-feedback-list@broadcom.com>, <f.fainelli@gmail.com>,
	"Justin Chen" <justin.chen@broadcom.com>
Subject: Re: [RFC] MIPS: Add cacheinfo support
Date: Thu, 8 Dec 2016 15:26:59 -0800	[thread overview]
Message-ID: <5849EC43.2070802@imgtec.com> (raw)
In-Reply-To: <20161208011626.20885-1-justinpopo6@gmail.com>

On 12/07/2016 05:16 PM, Justin Chen wrote:
> From: Justin Chen <justin.chen@broadcom.com>
>
> Add cacheinfo support for MIPS architectures.
>
> Use information from the cpuinfo_mips struct to populate the
> cacheinfo struct. This allows an architecture agnostic approach,
> however this also means if cache information is not properly
> populated within the cpuinfo_mips struct, there is nothing
> we can do. (I.E. c-r3k.c)
>
>

Justin, for architecture agnostic approach of work with caches the 
cpuinfo_mips lacks more information:

1.  L1I-L1D coherency status (is L1I coherent to L1D)
2.  L1D-L2 coherency status (is L1D coherent to L2)
3.  L1I-L2 coherency status (is L1I coherent to L2)
4.  L1I refill from L1D (snoops L1D) or L2 (may require L1D flush)?
5.  L1D cache aliasing disabled by HW
6.  L1I cache aliasing disabled by HW
7.  Barrier existence and it's type between various cache flushes as is 
as between cache flush and final (completed) state.
8.  Cache ownership (current: assumed that sibling CPUs share L1 - may 
be shared_cpu_map?)
9.  Is address cache flush operation global in system or pure local?
10. Is index cache flush operation global in system or pure local?

Without that the proposed cpuinfo_mips struct is basically useless.

Regards,
- Leonid

WARNING: multiple messages have this Message-ID (diff)
From: Leonid Yegoshin <Leonid.Yegoshin@imgtec.com>
To: Justin Chen <justinpopo6@gmail.com>, linux-mips@linux-mips.org
Cc: bcm-kernel-feedback-list@broadcom.com, f.fainelli@gmail.com,
	Justin Chen <justin.chen@broadcom.com>
Subject: Re: [RFC] MIPS: Add cacheinfo support
Date: Thu, 8 Dec 2016 15:26:59 -0800	[thread overview]
Message-ID: <5849EC43.2070802@imgtec.com> (raw)
Message-ID: <20161208232659.5mYUKlDzt_Ws1pTHIHWjd9s5SRt6xFEiJ6TRy7CTZpI@z> (raw)
In-Reply-To: <20161208011626.20885-1-justinpopo6@gmail.com>

On 12/07/2016 05:16 PM, Justin Chen wrote:
> From: Justin Chen <justin.chen@broadcom.com>
>
> Add cacheinfo support for MIPS architectures.
>
> Use information from the cpuinfo_mips struct to populate the
> cacheinfo struct. This allows an architecture agnostic approach,
> however this also means if cache information is not properly
> populated within the cpuinfo_mips struct, there is nothing
> we can do. (I.E. c-r3k.c)
>
>

Justin, for architecture agnostic approach of work with caches the 
cpuinfo_mips lacks more information:

1.  L1I-L1D coherency status (is L1I coherent to L1D)
2.  L1D-L2 coherency status (is L1D coherent to L2)
3.  L1I-L2 coherency status (is L1I coherent to L2)
4.  L1I refill from L1D (snoops L1D) or L2 (may require L1D flush)?
5.  L1D cache aliasing disabled by HW
6.  L1I cache aliasing disabled by HW
7.  Barrier existence and it's type between various cache flushes as is 
as between cache flush and final (completed) state.
8.  Cache ownership (current: assumed that sibling CPUs share L1 - may 
be shared_cpu_map?)
9.  Is address cache flush operation global in system or pure local?
10. Is index cache flush operation global in system or pure local?

Without that the proposed cpuinfo_mips struct is basically useless.

Regards,
- Leonid

  reply	other threads:[~2016-12-08 23:27 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-12-08  1:16 [RFC] MIPS: Add cacheinfo support justinpopo6
2016-12-08 23:26 ` Leonid Yegoshin [this message]
2016-12-08 23:26   ` Leonid Yegoshin
2016-12-09  0:28   ` Justin Chen
2016-12-09  1:01     ` Leonid Yegoshin
2016-12-09  1:01       ` Leonid Yegoshin
2016-12-12 18:24       ` Florian Fainelli
2016-12-12 20:45         ` Leonid Yegoshin
2016-12-12 20:45           ` Leonid Yegoshin
2016-12-12 21:57           ` Florian Fainelli
2016-12-13  0:19             ` Leonid Yegoshin
2016-12-13  0:19               ` Leonid Yegoshin
2016-12-13 11:09               ` Maciej W. Rozycki
2016-12-13 11:09                 ` Maciej W. Rozycki
2016-12-13 19:18                 ` Leonid Yegoshin
2016-12-13 19:18                   ` Leonid Yegoshin
2016-12-13 20:01                   ` Justin Chen
2016-12-13 20:05                     ` Florian Fainelli
2016-12-13 20:15                       ` Leonid Yegoshin
2016-12-13 20:15                         ` Leonid Yegoshin
2016-12-13 21:53           ` Ralf Baechle
2016-12-13 21:48       ` Ralf Baechle

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=5849EC43.2070802@imgtec.com \
    --to=leonid.yegoshin@imgtec.com \
    --cc=bcm-kernel-feedback-list@broadcom.com \
    --cc=f.fainelli@gmail.com \
    --cc=justin.chen@broadcom.com \
    --cc=justinpopo6@gmail.com \
    --cc=linux-mips@linux-mips.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 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.