[PATCH v9 0/3] target/arm: add support for FEAT_TLBIRANGE and FEAT_TLBIOS

Rebecca Cran posted 3 patches 2 years, 11 months ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20210512182337.18563-1-rebecca@nuviainc.com
Maintainers: Peter Maydell <peter.maydell@linaro.org>
target/arm/cpu.h    |  10 +
target/arm/cpu64.c  |   1 +
target/arm/helper.c | 324 ++++++++++++++++++++
3 files changed, 335 insertions(+)
[PATCH v9 0/3] target/arm: add support for FEAT_TLBIRANGE and FEAT_TLBIOS
Posted by Rebecca Cran 2 years, 11 months ago
Testing: booted Linux 5.12. Ran "make test" and checkpatch.pl.

Changes from v8 to v9:

Dropped patch 1/4, because Richard's 
'accel/tcg: Add tlb_flush interface for a range of pages'
patch replaces it.

Fixed page_shift calculation.

Added do_rvae_write and vae2_tlbmask functions.

Moved RVAE*OS entries from the tlbios to tlbirange table.

Rebecca Cran (3):
  target/arm: Add support for FEAT_TLBIRANGE
  target/arm: Add support for FEAT_TLBIOS
  target/arm: set ID_AA64ISAR0.TLB to 2 for max AARCH64 CPU type

 target/arm/cpu.h    |  10 +
 target/arm/cpu64.c  |   1 +
 target/arm/helper.c | 324 ++++++++++++++++++++
 3 files changed, 335 insertions(+)

-- 
2.26.2


Re: [PATCH v9 0/3] target/arm: add support for FEAT_TLBIRANGE and FEAT_TLBIOS
Posted by Alex Bennée 2 years, 11 months ago
Rebecca Cran <rebecca@nuviainc.com> writes:

> Testing: booted Linux 5.12. Ran "make test" and checkpatch.pl.
>
> Changes from v8 to v9:
>
> Dropped patch 1/4, because Richard's 
> 'accel/tcg: Add tlb_flush interface for a range of pages'
> patch replaces it.

But it's needed right? It's perfectly fine to pull someone elses patch
into your series if it's needed for the review. The alternative is to
add a "Based-on" tag to your cover letter so it's easy to find the patch
or series we need to apply before yours.

-- 
Alex Bennée

Re: [PATCH v9 0/3] target/arm: add support for FEAT_TLBIRANGE and FEAT_TLBIOS
Posted by Rebecca Cran 2 years, 11 months ago
On 5/12/21 1:32 PM, Alex Bennée wrote:

> But it's needed right? It's perfectly fine to pull someone elses patch
> into your series if it's needed for the review. The alternative is to
> add a "Based-on" tag to your cover letter so it's easy to find the patch
> or series we need to apply before yours.
> 

Yes, it's needed. Sorry, I didn't realize there was an established 
process for this.

 From previous patches, I see the line should be:

Based-on: 20210508201640.1045808-1-richard.henderson@linaro.org


-- 
Rebecca Cran

Re: [PATCH v9 0/3] target/arm: add support for FEAT_TLBIRANGE and FEAT_TLBIOS
Posted by Peter Maydell 2 years, 11 months ago
On Wed, 12 May 2021 at 19:26, Rebecca Cran <rebecca@nuviainc.com> wrote:
>
> Testing: booted Linux 5.12. Ran "make test" and checkpatch.pl.
>
> Changes from v8 to v9:
>
> Dropped patch 1/4, because Richard's
> 'accel/tcg: Add tlb_flush interface for a range of pages'
> patch replaces it.
>
> Fixed page_shift calculation.
>
> Added do_rvae_write and vae2_tlbmask functions.
>
> Moved RVAE*OS entries from the tlbios to tlbirange table.



Applied to target-arm.next, thanks.

-- PMM