[PATCH v8 0/7] device tree mapping

Oleksii Kurochko posted 7 patches 1 week, 1 day ago
Failed in applying to current master (apply log)
There is a newer version of this series
xen/arch/riscv/Kconfig                      |   1 +
xen/arch/riscv/Makefile                     |   2 +
xen/arch/riscv/include/asm/atomic.h         |  11 +-
xen/arch/riscv/include/asm/config.h         |  16 +-
xen/arch/riscv/include/asm/current.h        |  27 +-
xen/arch/riscv/include/asm/fixmap.h         |  46 +++
xen/arch/riscv/include/asm/flushtlb.h       |  15 +
xen/arch/riscv/include/asm/mm.h             |   6 +
xen/arch/riscv/include/asm/page.h           |  99 +++++
xen/arch/riscv/include/asm/pmap.h           |  36 ++
xen/arch/riscv/include/asm/processor.h      |   3 -
xen/arch/riscv/include/asm/riscv_encoding.h |   2 +
xen/arch/riscv/include/asm/sbi.h            |  62 +++
xen/arch/riscv/include/asm/smp.h            |  18 +
xen/arch/riscv/mm.c                         | 101 ++++-
xen/arch/riscv/pt.c                         | 421 ++++++++++++++++++++
xen/arch/riscv/riscv64/asm-offsets.c        |   3 +
xen/arch/riscv/riscv64/head.S               |  14 +
xen/arch/riscv/sbi.c                        | 273 ++++++++++++-
xen/arch/riscv/setup.c                      |  17 +
xen/arch/riscv/smp.c                        |  15 +
xen/arch/riscv/xen.lds.S                    |   2 +-
22 files changed, 1171 insertions(+), 19 deletions(-)
create mode 100644 xen/arch/riscv/include/asm/fixmap.h
create mode 100644 xen/arch/riscv/include/asm/pmap.h
create mode 100644 xen/arch/riscv/pt.c
create mode 100644 xen/arch/riscv/smp.c
[PATCH v8 0/7] device tree mapping
Posted by Oleksii Kurochko 1 week, 1 day ago
Current patch series introduces device tree mapping for RISC-V
and necessary things for that such as:
- Fixmap mapping
- pmap
- Xen page table processing

---
Changes in v8:
 - The following patch was merged to staging:
     [PATCH v5 1/7] xen/riscv: use {read,write}{b,w,l,q}_cpu() to define {read,write}_atomic()
 - All other changes are patch specific so please look at the patch.
---
Changes in v7:
 - Drop the patch "xen/riscv: prevent recursion when ASSERT(), BUG*(), or panic() are called"
 - All other changes are patch specific so please look at the patch.
---
Changes in v6:
 - Add patch to fix recursion when ASSERT(), BUG*(), panic() are called.
 - Add patch to allow write_atomic() to work with  non-scalar types for consistence
   with read_atomic().
 - All other changes are patch specific so please look at the patch. 
---
Changes in v5:
 - The following patch was merged to staging:
     [PATCH v3 3/9] xen/riscv: enable CONFIG_HAS_DEVICE_TREE
 - Drop depedency from "RISCV basic exception handling implementation" as
   it was meged to staging branch.
 - All other changes are patch specific so please look at the patch.
---
Changes in v4:
 - Drop depedency from common devicre tree patch series as it was merged to
   staging.
 - Update the cover letter message.
 - All other changes are patch specific so please look at the patch.
---
Changes in v3:
 - Introduce SBI RFENCE extension support.
 - Introduce and initialize pcpu_info[] and __cpuid_to_hartid_map[] and functionality
   to work with this arrays.
 - Make page table handling arch specific instead of trying to make it generic.
 - All other changes are patch specific so please look at the patch.
---
Changes in v2:
 - Update the cover letter message
 - introduce fixmap mapping
 - introduce pmap
 - introduce CONFIG_GENREIC_PT
 - update use early_fdt_map() after MMU is enabled.
---

Oleksii Kurochko (7):
  xen/riscv: allow write_atomic() to work with non-scalar types
  xen/riscv: set up fixmap mappings
  xen/riscv: introduce asm/pmap.h header
  xen/riscv: introduce functionality to work with CPU info
  xen/riscv: introduce and initialize SBI RFENCE extension
  xen/riscv: page table handling
  xen/riscv: introduce early_fdt_map()

 xen/arch/riscv/Kconfig                      |   1 +
 xen/arch/riscv/Makefile                     |   2 +
 xen/arch/riscv/include/asm/atomic.h         |  11 +-
 xen/arch/riscv/include/asm/config.h         |  16 +-
 xen/arch/riscv/include/asm/current.h        |  27 +-
 xen/arch/riscv/include/asm/fixmap.h         |  46 +++
 xen/arch/riscv/include/asm/flushtlb.h       |  15 +
 xen/arch/riscv/include/asm/mm.h             |   6 +
 xen/arch/riscv/include/asm/page.h           |  99 +++++
 xen/arch/riscv/include/asm/pmap.h           |  36 ++
 xen/arch/riscv/include/asm/processor.h      |   3 -
 xen/arch/riscv/include/asm/riscv_encoding.h |   2 +
 xen/arch/riscv/include/asm/sbi.h            |  62 +++
 xen/arch/riscv/include/asm/smp.h            |  18 +
 xen/arch/riscv/mm.c                         | 101 ++++-
 xen/arch/riscv/pt.c                         | 421 ++++++++++++++++++++
 xen/arch/riscv/riscv64/asm-offsets.c        |   3 +
 xen/arch/riscv/riscv64/head.S               |  14 +
 xen/arch/riscv/sbi.c                        | 273 ++++++++++++-
 xen/arch/riscv/setup.c                      |  17 +
 xen/arch/riscv/smp.c                        |  15 +
 xen/arch/riscv/xen.lds.S                    |   2 +-
 22 files changed, 1171 insertions(+), 19 deletions(-)
 create mode 100644 xen/arch/riscv/include/asm/fixmap.h
 create mode 100644 xen/arch/riscv/include/asm/pmap.h
 create mode 100644 xen/arch/riscv/pt.c
 create mode 100644 xen/arch/riscv/smp.c

-- 
2.46.1
Re: [PATCH v8 0/7] device tree mapping
Posted by Jan Beulich 5 days, 15 hours ago
On 27.09.2024 18:33, Oleksii Kurochko wrote:
> Current patch series introduces device tree mapping for RISC-V
> and necessary things for that such as:
> - Fixmap mapping
> - pmap
> - Xen page table processing
> 
> ---
> Changes in v8:
>  - The following patch was merged to staging:
>      [PATCH v5 1/7] xen/riscv: use {read,write}{b,w,l,q}_cpu() to define {read,write}_atomic()
>  - All other changes are patch specific so please look at the patch.

Except that afaics none of the patches has any revision log.

Jan
Re: [PATCH v8 0/7] device tree mapping
Posted by oleksii.kurochko@gmail.com 5 days, 14 hours ago
On Mon, 2024-09-30 at 09:27 +0200, Jan Beulich wrote:
> On 27.09.2024 18:33, Oleksii Kurochko wrote:
> > Current patch series introduces device tree mapping for RISC-V
> > and necessary things for that such as:
> > - Fixmap mapping
> > - pmap
> > - Xen page table processing
> > 
> > ---
> > Changes in v8:
> >  - The following patch was merged to staging:
> >      [PATCH v5 1/7] xen/riscv: use {read,write}{b,w,l,q}_cpu() to
> > define {read,write}_atomic()
> >  - All other changes are patch specific so please look at the
> > patch.
> 
> Except that afaics none of the patches has any revision log.
Would it be helpful if I will send revision log as a reply to each
patch? Or it would be better just to re-send the patch series?

~ Oleksii
Re: [PATCH v8 0/7] device tree mapping
Posted by Jan Beulich 5 days, 14 hours ago
On 30.09.2024 10:14, oleksii.kurochko@gmail.com wrote:
> On Mon, 2024-09-30 at 09:27 +0200, Jan Beulich wrote:
>> On 27.09.2024 18:33, Oleksii Kurochko wrote:
>>> Current patch series introduces device tree mapping for RISC-V
>>> and necessary things for that such as:
>>> - Fixmap mapping
>>> - pmap
>>> - Xen page table processing
>>>
>>> ---
>>> Changes in v8:
>>>  - The following patch was merged to staging:
>>>      [PATCH v5 1/7] xen/riscv: use {read,write}{b,w,l,q}_cpu() to
>>> define {read,write}_atomic()
>>>  - All other changes are patch specific so please look at the
>>> patch.
>>
>> Except that afaics none of the patches has any revision log.
> Would it be helpful if I will send revision log as a reply to each
> patch? Or it would be better just to re-send the patch series?

To me the one for 6/7 is relevant, to aid review. Sending that as reply
will be okay I guess. Patches 1-5 went in a few minutes ago anyway.

Jan

Re: [PATCH v8 0/7] device tree mapping
Posted by Jan Beulich 5 days, 14 hours ago
On 27.09.2024 18:33, Oleksii Kurochko wrote:
> Current patch series introduces device tree mapping for RISC-V
> and necessary things for that such as:
> - Fixmap mapping
> - pmap
> - Xen page table processing

While nothing is being said here towards a dependency, ...

> ---
> Changes in v8:
>  - The following patch was merged to staging:
>      [PATCH v5 1/7] xen/riscv: use {read,write}{b,w,l,q}_cpu() to define {read,write}_atomic()
>  - All other changes are patch specific so please look at the patch.
> ---
> Changes in v7:
>  - Drop the patch "xen/riscv: prevent recursion when ASSERT(), BUG*(), or panic() are called"
>  - All other changes are patch specific so please look at the patch.
> ---
> Changes in v6:
>  - Add patch to fix recursion when ASSERT(), BUG*(), panic() are called.
>  - Add patch to allow write_atomic() to work with  non-scalar types for consistence
>    with read_atomic().
>  - All other changes are patch specific so please look at the patch. 
> ---
> Changes in v5:
>  - The following patch was merged to staging:
>      [PATCH v3 3/9] xen/riscv: enable CONFIG_HAS_DEVICE_TREE
>  - Drop depedency from "RISCV basic exception handling implementation" as
>    it was meged to staging branch.
>  - All other changes are patch specific so please look at the patch.
> ---
> Changes in v4:
>  - Drop depedency from common devicre tree patch series as it was merged to
>    staging.
>  - Update the cover letter message.
>  - All other changes are patch specific so please look at the patch.
> ---
> Changes in v3:
>  - Introduce SBI RFENCE extension support.
>  - Introduce and initialize pcpu_info[] and __cpuid_to_hartid_map[] and functionality
>    to work with this arrays.
>  - Make page table handling arch specific instead of trying to make it generic.
>  - All other changes are patch specific so please look at the patch.
> ---
> Changes in v2:
>  - Update the cover letter message
>  - introduce fixmap mapping
>  - introduce pmap
>  - introduce CONFIG_GENREIC_PT
>  - update use early_fdt_map() after MMU is enabled.
> ---
> 
> Oleksii Kurochko (7):
>   xen/riscv: allow write_atomic() to work with non-scalar types
>   xen/riscv: set up fixmap mappings
>   xen/riscv: introduce asm/pmap.h header
>   xen/riscv: introduce functionality to work with CPU info
>   xen/riscv: introduce and initialize SBI RFENCE extension
>   xen/riscv: page table handling
>   xen/riscv: introduce early_fdt_map()
> 
>  xen/arch/riscv/Kconfig                      |   1 +
>  xen/arch/riscv/Makefile                     |   2 +
>  xen/arch/riscv/include/asm/atomic.h         |  11 +-
>  xen/arch/riscv/include/asm/config.h         |  16 +-
>  xen/arch/riscv/include/asm/current.h        |  27 +-
>  xen/arch/riscv/include/asm/fixmap.h         |  46 +++
>  xen/arch/riscv/include/asm/flushtlb.h       |  15 +
>  xen/arch/riscv/include/asm/mm.h             |   6 +
>  xen/arch/riscv/include/asm/page.h           |  99 +++++
>  xen/arch/riscv/include/asm/pmap.h           |  36 ++
>  xen/arch/riscv/include/asm/processor.h      |   3 -
>  xen/arch/riscv/include/asm/riscv_encoding.h |   2 +
>  xen/arch/riscv/include/asm/sbi.h            |  62 +++
>  xen/arch/riscv/include/asm/smp.h            |  18 +
>  xen/arch/riscv/mm.c                         | 101 ++++-
>  xen/arch/riscv/pt.c                         | 421 ++++++++++++++++++++
>  xen/arch/riscv/riscv64/asm-offsets.c        |   3 +
>  xen/arch/riscv/riscv64/head.S               |  14 +
>  xen/arch/riscv/sbi.c                        | 273 ++++++++++++-
>  xen/arch/riscv/setup.c                      |  17 +

... I had to fiddle with three of the patches touching this file, to
accommodate for an apparent debugging patch you have in your tree.
Please can you make sure to submit patches against plain staging, or
to clearly state dependencies?

Jan
Re: [PATCH v8 0/7] device tree mapping
Posted by oleksii.kurochko@gmail.com 5 days, 14 hours ago
On Mon, 2024-09-30 at 10:17 +0200, Jan Beulich wrote:
> On 27.09.2024 18:33, Oleksii Kurochko wrote:
> > Current patch series introduces device tree mapping for RISC-V
> > and necessary things for that such as:
> > - Fixmap mapping
> > - pmap
> > - Xen page table processing
> 
> While nothing is being said here towards a dependency, ...
> 
> > ---
> > Changes in v8:
> >  - The following patch was merged to staging:
> >      [PATCH v5 1/7] xen/riscv: use {read,write}{b,w,l,q}_cpu() to
> > define {read,write}_atomic()
> >  - All other changes are patch specific so please look at the
> > patch.
> > ---
> > Changes in v7:
> >  - Drop the patch "xen/riscv: prevent recursion when ASSERT(),
> > BUG*(), or panic() are called"
> >  - All other changes are patch specific so please look at the
> > patch.
> > ---
> > Changes in v6:
> >  - Add patch to fix recursion when ASSERT(), BUG*(), panic() are
> > called.
> >  - Add patch to allow write_atomic() to work with  non-scalar types
> > for consistence
> >    with read_atomic().
> >  - All other changes are patch specific so please look at the
> > patch. 
> > ---
> > Changes in v5:
> >  - The following patch was merged to staging:
> >      [PATCH v3 3/9] xen/riscv: enable CONFIG_HAS_DEVICE_TREE
> >  - Drop depedency from "RISCV basic exception handling
> > implementation" as
> >    it was meged to staging branch.
> >  - All other changes are patch specific so please look at the
> > patch.
> > ---
> > Changes in v4:
> >  - Drop depedency from common devicre tree patch series as it was
> > merged to
> >    staging.
> >  - Update the cover letter message.
> >  - All other changes are patch specific so please look at the
> > patch.
> > ---
> > Changes in v3:
> >  - Introduce SBI RFENCE extension support.
> >  - Introduce and initialize pcpu_info[] and __cpuid_to_hartid_map[]
> > and functionality
> >    to work with this arrays.
> >  - Make page table handling arch specific instead of trying to make
> > it generic.
> >  - All other changes are patch specific so please look at the
> > patch.
> > ---
> > Changes in v2:
> >  - Update the cover letter message
> >  - introduce fixmap mapping
> >  - introduce pmap
> >  - introduce CONFIG_GENREIC_PT
> >  - update use early_fdt_map() after MMU is enabled.
> > ---
> > 
> > Oleksii Kurochko (7):
> >   xen/riscv: allow write_atomic() to work with non-scalar types
> >   xen/riscv: set up fixmap mappings
> >   xen/riscv: introduce asm/pmap.h header
> >   xen/riscv: introduce functionality to work with CPU info
> >   xen/riscv: introduce and initialize SBI RFENCE extension
> >   xen/riscv: page table handling
> >   xen/riscv: introduce early_fdt_map()
> > 
> >  xen/arch/riscv/Kconfig                      |   1 +
> >  xen/arch/riscv/Makefile                     |   2 +
> >  xen/arch/riscv/include/asm/atomic.h         |  11 +-
> >  xen/arch/riscv/include/asm/config.h         |  16 +-
> >  xen/arch/riscv/include/asm/current.h        |  27 +-
> >  xen/arch/riscv/include/asm/fixmap.h         |  46 +++
> >  xen/arch/riscv/include/asm/flushtlb.h       |  15 +
> >  xen/arch/riscv/include/asm/mm.h             |   6 +
> >  xen/arch/riscv/include/asm/page.h           |  99 +++++
> >  xen/arch/riscv/include/asm/pmap.h           |  36 ++
> >  xen/arch/riscv/include/asm/processor.h      |   3 -
> >  xen/arch/riscv/include/asm/riscv_encoding.h |   2 +
> >  xen/arch/riscv/include/asm/sbi.h            |  62 +++
> >  xen/arch/riscv/include/asm/smp.h            |  18 +
> >  xen/arch/riscv/mm.c                         | 101 ++++-
> >  xen/arch/riscv/pt.c                         | 421
> > ++++++++++++++++++++
> >  xen/arch/riscv/riscv64/asm-offsets.c        |   3 +
> >  xen/arch/riscv/riscv64/head.S               |  14 +
> >  xen/arch/riscv/sbi.c                        | 273 ++++++++++++-
> >  xen/arch/riscv/setup.c                      |  17 +
> 
> ... I had to fiddle with three of the patches touching this file, to
> accommodate for an apparent debugging patch you have in your tree.
> Please can you make sure to submit patches against plain staging, or
> to clearly state dependencies?
I am always trying not to forget to rebase on top of staging for this
patch series:

65c49e7aa2 (HEAD -> riscv-dt-support-v8, origin/riscv-dt-support-v8)
xen/riscv: introduce early_fdt_map()
ead52f68ce xen/riscv: page table handling
c3aba0520f xen/riscv: introduce and initialize SBI RFENCE extension
3ffb3ffd38 xen/riscv: introduce functionality to work with CPU info
4bfd2bfdb2 xen/riscv: introduce asm/pmap.h header
87bc91db10 xen/riscv: set up fixmap mappings
09b925f973 xen/riscv: allow write_atomic() to work with non-scalar
types
625ee7650c xen/README: add compiler and binutils versions for RISC-V64
5379a23ad7 xen/riscv: test basic exception handling stuff
2b6fb9f3c4 (origin/staging, origin/HEAD, staging) blkif: Fix a couple
of typos
6e73a16230 blkif: Fix alignment description for discard request
0111c86bfa x86/boot: Refactor BIOS/PVH start

Only some patches have been merged today to staging on top of "blkif:
Fix a couple of typos".

It shouldn't be any issue with applying patches from these patch
series.

~ Oleksii
Re: [PATCH v8 0/7] device tree mapping
Posted by Jan Beulich 5 days, 14 hours ago
On 30.09.2024 10:24, oleksii.kurochko@gmail.com wrote:
> On Mon, 2024-09-30 at 10:17 +0200, Jan Beulich wrote:
>> On 27.09.2024 18:33, Oleksii Kurochko wrote:
>>> Current patch series introduces device tree mapping for RISC-V
>>> and necessary things for that such as:
>>> - Fixmap mapping
>>> - pmap
>>> - Xen page table processing
>>
>> While nothing is being said here towards a dependency, ...
>>
>>> ---
>>> Changes in v8:
>>>  - The following patch was merged to staging:
>>>      [PATCH v5 1/7] xen/riscv: use {read,write}{b,w,l,q}_cpu() to
>>> define {read,write}_atomic()
>>>  - All other changes are patch specific so please look at the
>>> patch.
>>> ---
>>> Changes in v7:
>>>  - Drop the patch "xen/riscv: prevent recursion when ASSERT(),
>>> BUG*(), or panic() are called"
>>>  - All other changes are patch specific so please look at the
>>> patch.
>>> ---
>>> Changes in v6:
>>>  - Add patch to fix recursion when ASSERT(), BUG*(), panic() are
>>> called.
>>>  - Add patch to allow write_atomic() to work with  non-scalar types
>>> for consistence
>>>    with read_atomic().
>>>  - All other changes are patch specific so please look at the
>>> patch. 
>>> ---
>>> Changes in v5:
>>>  - The following patch was merged to staging:
>>>      [PATCH v3 3/9] xen/riscv: enable CONFIG_HAS_DEVICE_TREE
>>>  - Drop depedency from "RISCV basic exception handling
>>> implementation" as
>>>    it was meged to staging branch.
>>>  - All other changes are patch specific so please look at the
>>> patch.
>>> ---
>>> Changes in v4:
>>>  - Drop depedency from common devicre tree patch series as it was
>>> merged to
>>>    staging.
>>>  - Update the cover letter message.
>>>  - All other changes are patch specific so please look at the
>>> patch.
>>> ---
>>> Changes in v3:
>>>  - Introduce SBI RFENCE extension support.
>>>  - Introduce and initialize pcpu_info[] and __cpuid_to_hartid_map[]
>>> and functionality
>>>    to work with this arrays.
>>>  - Make page table handling arch specific instead of trying to make
>>> it generic.
>>>  - All other changes are patch specific so please look at the
>>> patch.
>>> ---
>>> Changes in v2:
>>>  - Update the cover letter message
>>>  - introduce fixmap mapping
>>>  - introduce pmap
>>>  - introduce CONFIG_GENREIC_PT
>>>  - update use early_fdt_map() after MMU is enabled.
>>> ---
>>>
>>> Oleksii Kurochko (7):
>>>   xen/riscv: allow write_atomic() to work with non-scalar types
>>>   xen/riscv: set up fixmap mappings
>>>   xen/riscv: introduce asm/pmap.h header
>>>   xen/riscv: introduce functionality to work with CPU info
>>>   xen/riscv: introduce and initialize SBI RFENCE extension
>>>   xen/riscv: page table handling
>>>   xen/riscv: introduce early_fdt_map()
>>>
>>>  xen/arch/riscv/Kconfig                      |   1 +
>>>  xen/arch/riscv/Makefile                     |   2 +
>>>  xen/arch/riscv/include/asm/atomic.h         |  11 +-
>>>  xen/arch/riscv/include/asm/config.h         |  16 +-
>>>  xen/arch/riscv/include/asm/current.h        |  27 +-
>>>  xen/arch/riscv/include/asm/fixmap.h         |  46 +++
>>>  xen/arch/riscv/include/asm/flushtlb.h       |  15 +
>>>  xen/arch/riscv/include/asm/mm.h             |   6 +
>>>  xen/arch/riscv/include/asm/page.h           |  99 +++++
>>>  xen/arch/riscv/include/asm/pmap.h           |  36 ++
>>>  xen/arch/riscv/include/asm/processor.h      |   3 -
>>>  xen/arch/riscv/include/asm/riscv_encoding.h |   2 +
>>>  xen/arch/riscv/include/asm/sbi.h            |  62 +++
>>>  xen/arch/riscv/include/asm/smp.h            |  18 +
>>>  xen/arch/riscv/mm.c                         | 101 ++++-
>>>  xen/arch/riscv/pt.c                         | 421
>>> ++++++++++++++++++++
>>>  xen/arch/riscv/riscv64/asm-offsets.c        |   3 +
>>>  xen/arch/riscv/riscv64/head.S               |  14 +
>>>  xen/arch/riscv/sbi.c                        | 273 ++++++++++++-
>>>  xen/arch/riscv/setup.c                      |  17 +
>>
>> ... I had to fiddle with three of the patches touching this file, to
>> accommodate for an apparent debugging patch you have in your tree.
>> Please can you make sure to submit patches against plain staging, or
>> to clearly state dependencies?
> I am always trying not to forget to rebase on top of staging for this
> patch series:
> 
> 65c49e7aa2 (HEAD -> riscv-dt-support-v8, origin/riscv-dt-support-v8)
> xen/riscv: introduce early_fdt_map()
> ead52f68ce xen/riscv: page table handling
> c3aba0520f xen/riscv: introduce and initialize SBI RFENCE extension
> 3ffb3ffd38 xen/riscv: introduce functionality to work with CPU info
> 4bfd2bfdb2 xen/riscv: introduce asm/pmap.h header
> 87bc91db10 xen/riscv: set up fixmap mappings
> 09b925f973 xen/riscv: allow write_atomic() to work with non-scalar
> types
> 625ee7650c xen/README: add compiler and binutils versions for RISC-V64
> 5379a23ad7 xen/riscv: test basic exception handling stuff
> 2b6fb9f3c4 (origin/staging, origin/HEAD, staging) blkif: Fix a couple
> of typos
> 6e73a16230 blkif: Fix alignment description for discard request
> 0111c86bfa x86/boot: Refactor BIOS/PVH start

This looks to be a mix of several series. And "xen/riscv: test basic
exception handling stuff" looks to be what the problem was with, as that
wasn't committed yet (and imo also shouldn't be committed, as expressed
before; of course you can try to find someone else to approve it).

Jan

Re: [PATCH v8 0/7] device tree mapping
Posted by oleksii.kurochko@gmail.com 5 days, 14 hours ago
On Mon, 2024-09-30 at 10:32 +0200, Jan Beulich wrote:
> On 30.09.2024 10:24, oleksii.kurochko@gmail.com wrote:
> > On Mon, 2024-09-30 at 10:17 +0200, Jan Beulich wrote:
> > > On 27.09.2024 18:33, Oleksii Kurochko wrote:
> > > > Current patch series introduces device tree mapping for RISC-V
> > > > and necessary things for that such as:
> > > > - Fixmap mapping
> > > > - pmap
> > > > - Xen page table processing
> > > 
> > > While nothing is being said here towards a dependency, ...
> > > 
> > > > ---
> > > > Changes in v8:
> > > >  - The following patch was merged to staging:
> > > >      [PATCH v5 1/7] xen/riscv: use {read,write}{b,w,l,q}_cpu()
> > > > to
> > > > define {read,write}_atomic()
> > > >  - All other changes are patch specific so please look at the
> > > > patch.
> > > > ---
> > > > Changes in v7:
> > > >  - Drop the patch "xen/riscv: prevent recursion when ASSERT(),
> > > > BUG*(), or panic() are called"
> > > >  - All other changes are patch specific so please look at the
> > > > patch.
> > > > ---
> > > > Changes in v6:
> > > >  - Add patch to fix recursion when ASSERT(), BUG*(), panic()
> > > > are
> > > > called.
> > > >  - Add patch to allow write_atomic() to work with  non-scalar
> > > > types
> > > > for consistence
> > > >    with read_atomic().
> > > >  - All other changes are patch specific so please look at the
> > > > patch. 
> > > > ---
> > > > Changes in v5:
> > > >  - The following patch was merged to staging:
> > > >      [PATCH v3 3/9] xen/riscv: enable CONFIG_HAS_DEVICE_TREE
> > > >  - Drop depedency from "RISCV basic exception handling
> > > > implementation" as
> > > >    it was meged to staging branch.
> > > >  - All other changes are patch specific so please look at the
> > > > patch.
> > > > ---
> > > > Changes in v4:
> > > >  - Drop depedency from common devicre tree patch series as it
> > > > was
> > > > merged to
> > > >    staging.
> > > >  - Update the cover letter message.
> > > >  - All other changes are patch specific so please look at the
> > > > patch.
> > > > ---
> > > > Changes in v3:
> > > >  - Introduce SBI RFENCE extension support.
> > > >  - Introduce and initialize pcpu_info[] and
> > > > __cpuid_to_hartid_map[]
> > > > and functionality
> > > >    to work with this arrays.
> > > >  - Make page table handling arch specific instead of trying to
> > > > make
> > > > it generic.
> > > >  - All other changes are patch specific so please look at the
> > > > patch.
> > > > ---
> > > > Changes in v2:
> > > >  - Update the cover letter message
> > > >  - introduce fixmap mapping
> > > >  - introduce pmap
> > > >  - introduce CONFIG_GENREIC_PT
> > > >  - update use early_fdt_map() after MMU is enabled.
> > > > ---
> > > > 
> > > > Oleksii Kurochko (7):
> > > >   xen/riscv: allow write_atomic() to work with non-scalar types
> > > >   xen/riscv: set up fixmap mappings
> > > >   xen/riscv: introduce asm/pmap.h header
> > > >   xen/riscv: introduce functionality to work with CPU info
> > > >   xen/riscv: introduce and initialize SBI RFENCE extension
> > > >   xen/riscv: page table handling
> > > >   xen/riscv: introduce early_fdt_map()
> > > > 
> > > >  xen/arch/riscv/Kconfig                      |   1 +
> > > >  xen/arch/riscv/Makefile                     |   2 +
> > > >  xen/arch/riscv/include/asm/atomic.h         |  11 +-
> > > >  xen/arch/riscv/include/asm/config.h         |  16 +-
> > > >  xen/arch/riscv/include/asm/current.h        |  27 +-
> > > >  xen/arch/riscv/include/asm/fixmap.h         |  46 +++
> > > >  xen/arch/riscv/include/asm/flushtlb.h       |  15 +
> > > >  xen/arch/riscv/include/asm/mm.h             |   6 +
> > > >  xen/arch/riscv/include/asm/page.h           |  99 +++++
> > > >  xen/arch/riscv/include/asm/pmap.h           |  36 ++
> > > >  xen/arch/riscv/include/asm/processor.h      |   3 -
> > > >  xen/arch/riscv/include/asm/riscv_encoding.h |   2 +
> > > >  xen/arch/riscv/include/asm/sbi.h            |  62 +++
> > > >  xen/arch/riscv/include/asm/smp.h            |  18 +
> > > >  xen/arch/riscv/mm.c                         | 101 ++++-
> > > >  xen/arch/riscv/pt.c                         | 421
> > > > ++++++++++++++++++++
> > > >  xen/arch/riscv/riscv64/asm-offsets.c        |   3 +
> > > >  xen/arch/riscv/riscv64/head.S               |  14 +
> > > >  xen/arch/riscv/sbi.c                        | 273
> > > > ++++++++++++-
> > > >  xen/arch/riscv/setup.c                      |  17 +
> > > 
> > > ... I had to fiddle with three of the patches touching this file,
> > > to
> > > accommodate for an apparent debugging patch you have in your
> > > tree.
> > > Please can you make sure to submit patches against plain staging,
> > > or
> > > to clearly state dependencies?
> > I am always trying not to forget to rebase on top of staging for
> > this
> > patch series:
> > 
> > 65c49e7aa2 (HEAD -> riscv-dt-support-v8, origin/riscv-dt-support-
> > v8)
> > xen/riscv: introduce early_fdt_map()
> > ead52f68ce xen/riscv: page table handling
> > c3aba0520f xen/riscv: introduce and initialize SBI RFENCE extension
> > 3ffb3ffd38 xen/riscv: introduce functionality to work with CPU info
> > 4bfd2bfdb2 xen/riscv: introduce asm/pmap.h header
> > 87bc91db10 xen/riscv: set up fixmap mappings
> > 09b925f973 xen/riscv: allow write_atomic() to work with non-scalar
> > types
> > 625ee7650c xen/README: add compiler and binutils versions for RISC-
> > V64
> > 5379a23ad7 xen/riscv: test basic exception handling stuff
> > 2b6fb9f3c4 (origin/staging, origin/HEAD, staging) blkif: Fix a
> > couple
> > of typos
> > 6e73a16230 blkif: Fix alignment description for discard request
> > 0111c86bfa x86/boot: Refactor BIOS/PVH start
> 
> This looks to be a mix of several series. And "xen/riscv: test basic
> exception handling stuff" looks to be what the problem was with, as
> that
> wasn't committed yet (and imo also shouldn't be committed, as
> expressed
> before; of course you can try to find someone else to approve it).
Oh, you are right. I will put it to separate branch to not breaking the
things. Thanks.

~ Oleksii