lkml.org 
[lkml]   [2023]   [Sep]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH v20 28/30] LoongArch: KVM: Enable kvm config and add the makefile
From
On 8/31/23 16:30, Tianrui Zhao wrote:
> [snip]
> +
> +config AS_HAS_LVZ_EXTENSION
> + def_bool $(as-instr,hvcl 0)

Upon closer look it looks like this piece could use some improvement as
well: while presence of "hvcl" indeed always implies full support for
LVZ instructions, "hvcl" however isn't used anywhere in the series, so
it's not trivially making sense. It may be better to test for an
instruction actually going to be used like "gcsrrd". What do you think?

--
WANG "xen0n" Xuerui

Linux/LoongArch mailing list: https://lore.kernel.org/loongarch/

\
 
 \ /
  Last update: 2023-09-11 09:30    [W:0.366 / U:0.260 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site