[PATCH] RISC-V: move riscv_cbom_block_size to the correct #ifdef block

Palmer Dabbelt palmer at rivosinc.com
Thu Sep 15 10:09:19 PDT 2022


On Wed, 14 Sep 2022 15:19:51 PDT (-0700), heiko at sntech.de wrote:
> Hi,
>
> Am Mittwoch, 14. September 2022, 21:32:01 CEST schrieb Conor.Dooley at microchip.com:
>> On 14/09/2022 15:36, Heiko Stuebner wrote:
>> > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>> >
>> > riscv_cbom_block_size is used by all current non-coherent dma operations,
>> > not only the zicbom variant. So move it over the block also containing
>> > the riscv_noncoherent_supported() prototype.
>> >
>> > Fixes: 8f7e001e0325 ("RISC-V: Clean up the Zicbom block size probing")
>> > Reported-by: kernel test robot <lkp at intel.com>
>> > Signed-off-by: Heiko Stuebner <heiko at sntech.de>
>>
>> I accidentally fetched palmers repo rather than riscv & noticed he
>> pushed a fix there for this too:
>> https://git.kernel.org/pub/scm/linux/kernel/git/palmer/linux.git/commit/?h=zicbom-fix
>>
>> I think Palmer's solution is slightly nicer, but to me either makes
>> little difference, just getting things squared away for 6.0 is all
>> I care about at this point.
>
> yeah, Palmer's solution is nice and incidentially should also
> not only make LKP happier but maybe also Anup in one go :-) .
>
> So I'd guess disregard this patch and move Palmer's patch over?

OK, I'll go post mine and then merge it -- I was writing it at Plumbers, 
I guess I forgot to send it.

>
>
> Heiko
>
>>
>> Either this or the one on Palmer's branch is:
>> Reviewed-by: Conor Dooley <conor.dooley at microchip.com>
>>
>> > ---
>> >  arch/riscv/include/asm/cacheflush.h | 2 +-
>> >  1 file changed, 1 insertion(+), 1 deletion(-)
>> >
>> > diff --git a/arch/riscv/include/asm/cacheflush.h b/arch/riscv/include/asm/cacheflush.h
>> > index a89c005b4bbf..5c16d901d3da 100644
>> > --- a/arch/riscv/include/asm/cacheflush.h
>> > +++ b/arch/riscv/include/asm/cacheflush.h
>> > @@ -43,13 +43,13 @@ void flush_icache_mm(struct mm_struct *mm, bool local);
>> >  #endif /* CONFIG_SMP */
>> >
>> >  #ifdef CONFIG_RISCV_ISA_ZICBOM
>> > -extern unsigned int riscv_cbom_block_size;
>> >  void riscv_init_cbom_blocksize(void);
>> >  #else
>> >  static inline void riscv_init_cbom_blocksize(void) { }
>> >  #endif
>> >
>> >  #ifdef CONFIG_RISCV_DMA_NONCOHERENT
>> > +extern unsigned int riscv_cbom_block_size;
>> >  void riscv_noncoherent_supported(void);
>> >  #endif
>> >
>> > --
>> > 2.35.1
>> >
>>
>>



More information about the linux-riscv mailing list