[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 09/11] include/exec: added functions to the stubs in exec-all
From: |
David Gibson |
Subject: |
Re: [PATCH 09/11] include/exec: added functions to the stubs in exec-all.h |
Date: |
Mon, 17 May 2021 13:55:53 +1000 |
On Thu, May 13, 2021 at 11:03:24AM -0300, Lucas Mateus Martins Araujo e Castro
wrote:
>
> On 12/05/2021 15:34, Richard Henderson wrote:
> > On 5/12/21 9:08 AM, Bruno Larsen (billionai) wrote:
> > > From: "Lucas Mateus Castro (alqotel)"<lucas.araujo@eldorado.org.br>
> > >
> > > Added tlb_set_page and tlb_set_page_with_attrs to the
> > > stubbed functions in exec-all.h as it is needed
> > > in some functions when compiling without TCG
> > >
> > > Signed-off-by: Lucas Mateus Castro
> > > (alqotel)<lucas.araujo@eldorado.org.br>
> > > ---
> > > include/exec/exec-all.h | 10 ++++++++++
> > > 1 file changed, 10 insertions(+)
> >
> > No, the caller is tcg-specific already.
> >
> >
> > r~
>
> tlb_set_page is called by many ppc_hash64_handle_mmu_fault,
> ppc_radix64_handle_mmu_fault and ppc_hash32_handle_mmu_fault, all of which
> from what I've seen are only used inside #if defined(CONFIG_SOFTMMU). So
> what is the best way to deal with these tlb_set_page calls? Should these
> part of the _handle_mmu_fault functions never be reached or should
> these
The handle_mmu_fault() functions per se shouldn't be included in a
!SOFTMMU build. We might have to extract some of their internal logic
for the gdb path, though.
> functions never be called?
>
> If it's the latter then should we change the #if defined to #if
> defined(CONFIG_SOFTMMU) && (CONFIG_TCG)?
That definitely doesn't make sense. In practice CONFIG_SOFTMMU == CONFIG_TCG.
>
>
> P.S: There was a miscommunication between me and Bruno, this should've been
> a RFC.
>
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature
- Re: [RFC PATCH 08/11] target/ppc: wrapped some TCG only logic with ifdefs, (continued)
- [PATCH 09/11] include/exec: added functions to the stubs in exec-all.h, Bruno Larsen (billionai), 2021/05/12
- Re: [PATCH 09/11] include/exec: added functions to the stubs in exec-all.h, Richard Henderson, 2021/05/12
- Re: [PATCH 09/11] include/exec: added functions to the stubs in exec-all.h, Lucas Mateus Martins Araujo e Castro, 2021/05/13
- Re: [PATCH 09/11] include/exec: added functions to the stubs in exec-all.h, Richard Henderson, 2021/05/13
- Re: [PATCH 09/11] include/exec: added functions to the stubs in exec-all.h, David Gibson, 2021/05/17
- Re: [PATCH 09/11] include/exec: added functions to the stubs in exec-all.h, Lucas Mateus Martins Araujo e Castro, 2021/05/17
- Re: [PATCH 09/11] include/exec: added functions to the stubs in exec-all.h, Richard Henderson, 2021/05/17
- Re: [PATCH 09/11] include/exec: added functions to the stubs in exec-all.h, Bruno Piazera Larsen, 2021/05/18
- Re: [PATCH 09/11] include/exec: added functions to the stubs in exec-all.h, David Gibson, 2021/05/23
- Re: [PATCH 09/11] include/exec: added functions to the stubs in exec-all.h,
David Gibson <=
- Re: [PATCH 09/11] include/exec: added functions to the stubs in exec-all.h, Bruno Piazera Larsen, 2021/05/17
- Re: [PATCH 09/11] include/exec: added functions to the stubs in exec-all.h, David Gibson, 2021/05/23
[RFC PATCH 10/11] target/ppc: created tcg-stub.c file, Bruno Larsen (billionai), 2021/05/12
[PATCH 11/11] target/ppc: updated meson.build to support disable-tcg, Bruno Larsen (billionai), 2021/05/12