From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9411C1F428A for ; Tue, 21 Jan 2025 16:29:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=170.10.133.124 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1737476974; cv=none; b=VsjIqUx1iTSNocT5VVNkU/OYiMpIOf42i2piqsGBrswNfqJwcVvmXV5z8FF+wip0qiwYZPk7hq1x21MX0cZVMvlOH5j6KkHdDRZK/TEUMUQubUq/8FRvTqwEUYWb2AhEgkfcwDtXH3VbjhybSSPZ5gwgVPl4o4GzGICraqjYsRQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1737476974; c=relaxed/simple; bh=lvhAgMr1qLIw0C+KjhXTlOOlZ4f8ZXmXmQaQg4vArmk=; h=From:To:Cc:Subject:In-Reply-To:References:Date:Message-ID: MIME-Version:Content-Type; b=O8PSIXwSxnG6gZmYTIaTAadDj36+Y29uJVRnGzxHZR+hdEX0DmoRwXI22b+qD3skPMGB54p1bx0OkBpquhSmZ3o627zn+vH0ukXmqPu1iONHc7xGgOL62A24MRIebvE+QsUuSSL3jnwqldR4PoT1YCKwv7jz5iM00ttIiHjthBg= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=redhat.com; spf=pass smtp.mailfrom=redhat.com; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b=PBaYFN5R; arc=none smtp.client-ip=170.10.133.124 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=redhat.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=redhat.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="PBaYFN5R" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1737476970; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=FmmBH81f6hVHBomfe2UGheR/yM/UTiXDoOohLBQeYlA=; b=PBaYFN5RxEA6XY28zOcvdPhhdcDxpQOgIkeBHRlqx3Q6K0JCWKEwuFzmmDB0VrT0BoXj0R ofGkvLBgBKK19JPQtSPtPp+0McI9rBYA6hFC5aEF6/Am3RumzWDu5SgOu3Ltlrusf4oWsQ gWZ606Xy8vapTNAcjE8dvQzj2KYpT5A= Received: from mail-wr1-f71.google.com (mail-wr1-f71.google.com [209.85.221.71]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-60-V4G3VtLoMySQYUb-Ks52-Q-1; Tue, 21 Jan 2025 11:29:29 -0500 X-MC-Unique: V4G3VtLoMySQYUb-Ks52-Q-1 X-Mimecast-MFC-AGG-ID: V4G3VtLoMySQYUb-Ks52-Q Received: by mail-wr1-f71.google.com with SMTP id ffacd0b85a97d-38a35a65575so4802216f8f.1 for ; Tue, 21 Jan 2025 08:29:28 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1737476968; x=1738081768; h=mime-version:message-id:date:references:in-reply-to:subject:cc:to :from:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=FmmBH81f6hVHBomfe2UGheR/yM/UTiXDoOohLBQeYlA=; b=Wqeoyg79MVbfRBi0LP6/FFGfQSBQvME9y6xESHgbJ3z27ph9/IyMwoL1ukpCjx4ZfX 9M2wc0JewJKQkvzI2jE9+7Gp250tcZx4oCrbkTWAlMVFIaj+apxSJh6HleMoSAtvQbS8 sApb/aLF/r4NTTxIjt0tDoiLR5jYofgeR/QlIbLOVB3hdt2GYGc5kXC+woNZf3PqW4K3 YdliwcNvuUPGFSCz/DozrWI0jNSLAf9/p2AcxvhhWOhT8qL6UFYKrP1LHjyjL/Xt3WTt W94YiBKap30AaiksI2bBdVT23y5PDAHhv3eOMPRZN7YcGLeHGIghnpCWor9oke8W5YoU dfSA== X-Forwarded-Encrypted: i=1; AJvYcCUKnZAeWn1vQqBxBBB1NBdks+ar5cwJCzGMukkGJFrXWPO6l2eK1Re0LC7uWPL2prkScO+1WDNjfV2qeac=@vger.kernel.org X-Gm-Message-State: AOJu0Yz0e+TQb/MDjbq4BHuK1rAOXc0O1zGFASYN9NgoBDeeVMz+bWV2 QKTMyRnczy+t62EdtZJ3I11aK7r8cTWiyhSGNB3AWUyU2F/W2aPORJvZZdqKzzLO1sex9laWqEE pU4gSHSQkdzqDqz51OXoOIHaMsVel4DWhipkOYHp5LZqCghxy2AQX/3yxgIZ8vw== X-Gm-Gg: ASbGnctagihM3hV8ou5gWLWluS3JiHTb2Wr3SLN215n5fWqlgs2Rveu9jxZQ+HSVaav o4YVSnIyalGKm2wz2H8QxHSWarLlcMDYZcabzNxkhjAmSqkp+AGJa6FnNbw/7SboDIHvyiLihxp 924Ck5by5PwlAwUtYtRy6k40MTs8UuGKCpEjL8NJK9OBrpKvUA4qP2lYtfEhTvkjBGpisokalnV q9SpYGcBmYJqno8yJc+uE3VLy5q5C8Jtdp6gqrkqUVXxX4Pr1OPvfUNiDbMkkQZ X-Received: by 2002:adf:e8d1:0:b0:38b:da34:5915 with SMTP id ffacd0b85a97d-38bf566f5e5mr13683321f8f.23.1737476967738; Tue, 21 Jan 2025 08:29:27 -0800 (PST) X-Google-Smtp-Source: AGHT+IFK0MXcaZRWIzl2bfA4+MZMepukExVAz8baqmZ1gj8sc3lVxxkt4XOwKAvBR2JEk6noYQJ9yA== X-Received: by 2002:adf:e8d1:0:b0:38b:da34:5915 with SMTP id ffacd0b85a97d-38bf566f5e5mr13683271f8f.23.1737476966326; Tue, 21 Jan 2025 08:29:26 -0800 (PST) Received: from fedora (g2.ign.cz. [91.219.240.8]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-4389040854bsm190088195e9.7.2025.01.21.08.29.22 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 21 Jan 2025 08:29:23 -0800 (PST) From: Vitaly Kuznetsov To: Sean Christopherson Cc: Paolo Bonzini , kvm@vger.kernel.org, linux-kernel@vger.kernel.org, Dongjie Zou Subject: Re: [PATCH v2 4/4] KVM: selftests: Add CPUID tests for Hyper-V features that need in-kernel APIC In-Reply-To: References: <20250118003454.2619573-1-seanjc@google.com> <20250118003454.2619573-5-seanjc@google.com> <877c6p8t35.fsf@redhat.com> Date: Tue, 21 Jan 2025 17:29:22 +0100 Message-ID: <87wmeo6sgt.fsf@redhat.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain Sean Christopherson writes: > On Mon, Jan 20, 2025, Vitaly Kuznetsov wrote: >> > diff --git a/tools/testing/selftests/kvm/x86_64/hyperv_cpuid.c b/tools/testing/selftests/kvm/x86_64/hyperv_cpuid.c >> > index 3188749ec6e1..8f26130dc30d 100644 >> > --- a/tools/testing/selftests/kvm/x86_64/hyperv_cpuid.c >> > +++ b/tools/testing/selftests/kvm/x86_64/hyperv_cpuid.c >> > @@ -43,6 +43,7 @@ static bool smt_possible(void) >> > >> > static void test_hv_cpuid(struct kvm_vcpu *vcpu, bool evmcs_expected) >> > { >> > + const bool has_irqchip = !vcpu || vcpu->vm->has_irqchip; >> > const struct kvm_cpuid2 *hv_cpuid_entries; >> > int i; >> > int nent_expected = 10; >> > @@ -85,12 +86,19 @@ static void test_hv_cpuid(struct kvm_vcpu *vcpu, bool evmcs_expected) >> > entry->eax, evmcs_expected >> > ); >> > break; >> > + case 0x40000003: >> > + TEST_ASSERT(has_irqchip || !(entry->edx & BIT(19)), >> > + "Synthetic Timers should require in-kernel APIC"); >> >> Nitpick: BIT(19) of CPUID.0x40000003(EDX) advertises 'direct' mode >> for Synthetic timers and that's what we have paired with >> lapic_in_kernel() check. Thus, we may want to be a bit more specific and >> say >> >> "Direct Synthetic timers should require in-kernel APIC" >> (personally, I'd prefer "Synthetic timers in 'direct' mode" name but >> that's not how TLFS calls them) > > What about adding quotes to try and communicate that it's a property of Syntehtic > Timers? E.g. > > "\"Direct\" Synthetic Timers should require in-kernel APIC"); Sounds good to me) -- Vitaly