All of lore.kernel.org
 help / color / mirror / Atom feed
From: Peng Fan <van.freenix@gmail.com>
To: Stefano Stabellini <sstabellini@kernel.org>
Cc: julien.grall@arm.com, xen-devel@lists.xen.org
Subject: Re: [PATCH v4 0/7] unsafe big.LITTLE support
Date: Thu, 8 Mar 2018 14:15:10 +0800	[thread overview]
Message-ID: <20180308061458.GA25007@shlinux2> (raw)
In-Reply-To: <alpine.DEB.2.10.1803021103130.4239@sstabellini-ThinkPad-X260>

Hi Stefano,
On Fri, Mar 02, 2018 at 11:05:54AM -0800, Stefano Stabellini wrote:
>Hi all,
>
>This series changes the initialization of two virtual registers to make
>sure they match the value of the underlying physical cpu.
>
>It also disables cpus different from the boot cpu, unless a newly
>introduced command line option is specified. In that case, it explains
>how to setup the system to avoid corruptions, which involves manually
>specifying the cpu affinity of all domains, because the scheduler still
>lacks big.LITTLE support.
>
>In the uncommon case of a system where the cacheline sizes are different
>across cores, it disables all cores that have a different dcache line size
>from the boot cpu. In fact, it is not sufficient to use the dcache line
>size of the current cpu, it would be necessary to use the minimum across
>all dcache line sizes of all cores.  Given that it is actually uncommon
>even in big.LITTLE systems, just disable cpus for now.
>
>The first patch in the series is a fix for the way we read the dcache
>line size.

I am trying the patchset, but I meet issue that Guest Big/Little with
vcpu not working properly. As my current hardware has an issue
which has fix in Kernel, https://source.codeaurora.org/external/imx/linux-imx/commit/?h=imx_4.9.51_imx8_beta2&id=917cc3a8db2f3609ef8e2f59e7bcd31aa2cd4e59

I am not sure whether this issue cause DomU big/Little not work.
So wonder has this patchset been tested on Big/Little Hardware?

Thanks,
Peng.

>
>Cheers,
>
>Stefano
>
>
>Julien Grall (1):
>      xen/arm: Park CPUs with a MIDR different from the boot CPU.
>
>Stefano Stabellini (6):
>      xen/arm: Read the dcache line size from CTR register
>      xen/arm: make processor a per cpu variable
>      xen/arm: read ACTLR on the pcpu where the vcpu will run
>      xen/arm: set VPIDR based on the MIDR value of the underlying pCPU
>      xen/arm: update the docs about heterogeneous computing
>      xen/arm: disable CPUs with different dcache line sizes
>
> docs/misc/arm/big.LITTLE.txt        | 46 +++++++++++++++++++++++++++++++++++++
> docs/misc/xen-command-line.markdown | 15 ++++++++++++
> xen/arch/arm/arm32/head.S           |  2 +-
> xen/arch/arm/arm64/head.S           |  2 +-
> xen/arch/arm/domain.c               | 15 ++++++------
> xen/arch/arm/processor.c            |  8 +++----
> xen/arch/arm/setup.c                | 17 ++------------
> xen/arch/arm/smpboot.c              | 39 +++++++++++++++++++++++++++++++
> xen/arch/arm/vcpreg.c               |  4 ++--
> xen/include/asm-arm/cpregs.h        |  2 ++
> xen/include/asm-arm/domain.h        |  3 ---
> xen/include/asm-arm/page.h          | 27 +++++++++++++++-------
> 12 files changed, 138 insertions(+), 42 deletions(-)
> create mode 100644 docs/misc/arm/big.LITTLE.txt
>
>_______________________________________________
>Xen-devel mailing list
>Xen-devel@lists.xenproject.org
>https://lists.xenproject.org/mailman/listinfo/xen-devel

-- 

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel

  parent reply	other threads:[~2018-03-08  6:15 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-03-02 19:05 [PATCH v4 0/7] unsafe big.LITTLE support Stefano Stabellini
2018-03-02 19:06 ` [PATCH v4 1/7] xen/arm: Read the dcache line size from CTR register Stefano Stabellini
2018-03-02 19:06   ` [PATCH v4 2/7] xen/arm: Park CPUs with a MIDR different from the boot CPU Stefano Stabellini
2018-03-02 19:06   ` [PATCH v4 3/7] xen/arm: make processor a per cpu variable Stefano Stabellini
2018-03-02 19:06   ` [PATCH v4 4/7] xen/arm: read ACTLR on the pcpu where the vcpu will run Stefano Stabellini
2018-03-02 19:06   ` [PATCH v4 5/7] xen/arm: set VPIDR based on the MIDR value of the underlying pCPU Stefano Stabellini
2018-03-02 19:06   ` [PATCH v4 6/7] xen/arm: update the docs about heterogeneous computing Stefano Stabellini
2018-03-02 19:06   ` [PATCH v4 7/7] xen/arm: disable CPUs with different dcache line sizes Stefano Stabellini
2018-03-06 10:59     ` Julien Grall
2018-03-06 19:41       ` Stefano Stabellini
2018-03-06 10:46   ` [PATCH v4 1/7] xen/arm: Read the dcache line size from CTR register Julien Grall
2018-03-08  6:15 ` Peng Fan [this message]
2018-03-08 11:03   ` [PATCH v4 0/7] unsafe big.LITTLE support Julien Grall
2018-03-08 12:23     ` Peng Fan
2018-03-08 12:30       ` Julien Grall
2018-03-08 12:43         ` Peng Fan
2018-03-08 15:13           ` Julien Grall
2018-03-09  9:05             ` Peng Fan
2018-03-09 10:22               ` Julien Grall
2018-03-09 13:30                 ` Peng Fan
2018-03-09 14:40                   ` Julien Grall
2018-03-10  1:09                     ` Stefano Stabellini
2018-03-12  2:57                       ` Peng Fan
2018-03-12 11:02                         ` Julien Grall
2018-03-12  2:32                     ` Peng Fan
2018-03-12 10:34                       ` Julien Grall

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=20180308061458.GA25007@shlinux2 \
    --to=van.freenix@gmail.com \
    --cc=julien.grall@arm.com \
    --cc=sstabellini@kernel.org \
    --cc=xen-devel@lists.xen.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.