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]) by mx.groups.io with SMTP id smtpd.web08.4818.1641893389053109552 for ; Tue, 11 Jan 2022 01:29:49 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=fXNA50cG; spf=pass (domain: redhat.com, ip: 170.10.133.124, mailfrom: kraxel@redhat.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1641893388; 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=SwPNNeUAl/DjcDDh2syPgAdJ9nIvTnEyScXubtEqltU=; b=fXNA50cGT9uYVbKpGWuOVun2e18/dedsOy06KLi2iWDXxasxdn7tavErFt47JPLQl1IGBs uMWnZK86Z4xPqJIhyEHT70J8mA7Kkc7ep6nc19cVaFTXMG/08ktNOVykiPd/ThV8uAwNBu sUrvAdy1yhCV0/ZZYf/a9p8VbdtXqms= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-194--mKbqgl9NXCw9-__IXPxIQ-1; Tue, 11 Jan 2022 04:29:44 -0500 X-MC-Unique: -mKbqgl9NXCw9-__IXPxIQ-1 Received: from smtp.corp.redhat.com (int-mx07.intmail.prod.int.phx2.redhat.com [10.5.11.22]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id CD5EB1898293; Tue, 11 Jan 2022 09:29:43 +0000 (UTC) Received: from sirius.home.kraxel.org (unknown [10.39.193.24]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 8E8A51059118; Tue, 11 Jan 2022 09:29:43 +0000 (UTC) Received: by sirius.home.kraxel.org (Postfix, from userid 1000) id 28631180009C; Tue, 11 Jan 2022 10:29:41 +0100 (CET) Date: Tue, 11 Jan 2022 10:29:41 +0100 From: "Gerd Hoffmann" To: sebastien.boeuf@intel.com Cc: devel@edk2.groups.io, jiewen.yao@intel.com, jordan.l.justen@intel.com Subject: Re: [PATCH v2 02/10] OvmfPkg/CloudHv: Replace legacy 8254 PIT with local APIC timer Message-ID: <20220111092941.u5oe56inqhvdhuhu@sirius.home.kraxel.org> References: <493c86d77ae56fa9a62c93e37adad9447fafae05.1641836781.git.sebastien.boeuf@intel.com> MIME-Version: 1.0 In-Reply-To: <493c86d77ae56fa9a62c93e37adad9447fafae05.1641836781.git.sebastien.boeuf@intel.com> X-Scanned-By: MIMEDefang 2.84 on 10.5.11.22 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=kraxel@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Mon, Jan 10, 2022 at 06:47:22PM +0100, sebastien.boeuf@intel.com wrote: > From: Sebastien Boeuf > > Cloud Hypervisor doesn't emulate the legacy 8254 PIT, which is why > instead of relying on it as the timer UEFI services, rely on the > XenTimerDxe implementation. This is not Xen specific, as it simply uses > the local APIC timer triggering interrupts on the vector 32. I think you must set PcdFSBClock for this to work properly. When using kvm (which cloudhv does as far I know) this is simple as the frequency is fixed then, see microvm dsc file. With xen this is dynamic so the xen platform init has some code to figure the frequency and set the pcd. take care, Gerd