Received: by 2002:a25:4158:0:0:0:0:0 with SMTP id o85csp3155572yba; Mon, 22 Apr 2019 21:31:58 -0700 (PDT) X-Google-Smtp-Source: APXvYqwVoBn2E8r0ZbidppRl7dbrGEIAvAFW8WyV1OrU6ffa00ZtJYQAu3QqjFMu7iPhxU+OH2+Q X-Received: by 2002:a17:902:c1:: with SMTP id a59mr23220357pla.176.1555993917979; Mon, 22 Apr 2019 21:31:57 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1555993917; cv=none; d=google.com; s=arc-20160816; b=Y/c2H9X+kSIxmWTOIKNL8VkyIdJU0jZRvs092kKNlvcG+Ylwx8L7bZUyI8bW77gAB2 zDSZd9+RlM+jG5ZZH8sP/k/awdvheJftECizhsVrDlHA5WKD+AGoV3tfyTuql9Y0Ay5J MgndnhL9p2CJjgU5BO58Jwj3RPhRhs1VkBYldmfiavpVaBb4qsMthpbNvlA1VQ/hnUUt vQsH7pey/v2eYdEWsXMzcaJuLFU1yZYBqTC8vrkEtTzSlQw0nPYxS9zOC4Ei0g/hPxWg MqwsP0n73cEnJs8RYC2WTX/wyPu7pyZdeiYKAugifAprxmtrEjJGS+NQFxlnC5jJVGXr /kNA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature; bh=iCDEEjaXFN2ttbzzYVmz/c+842djhuXLkuUAp4+5DKk=; b=TQ+yt5/I9CDsRvT406lBthKlVDqq4lFe+mRvD1jVZJA29zpI1O7Zk3kELzivftt1MB Blvzdn7gh5F5qshAEh4VRq84lGpweJWR3pP/XB2zLV1TUGC5OWdKUSoaWtGrqZMfnMeH VJG18m9hv6ywy9eeKx8O/QjKDnCgD5X12WHeS1XTOdzaECg1qR+dUAN4w3aRS2nG1XW1 CCEXSMksf4UAoCfn5u5hnhG6nSnttzdZlHrpGmqJ9oHJBg/BXNwegB40R+xEhvRD/sZK BDREHEuWgZBiKEldEBEKUa+l9MnU3nWJjLY+iyZ0CS+4wWrmL2z5unabj/S9RjtuZZWf 0SuQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@endlessm-com.20150623.gappssmtp.com header.s=20150623 header.b=YjSoeKBW; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id k1si1626007pgo.81.2019.04.22.21.31.42; Mon, 22 Apr 2019 21:31:57 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@endlessm-com.20150623.gappssmtp.com header.s=20150623 header.b=YjSoeKBW; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1731291AbfDWDGt (ORCPT + 99 others); Mon, 22 Apr 2019 23:06:49 -0400 Received: from mail-qt1-f195.google.com ([209.85.160.195]:44878 "EHLO mail-qt1-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728527AbfDWDGt (ORCPT ); Mon, 22 Apr 2019 23:06:49 -0400 Received: by mail-qt1-f195.google.com with SMTP id s10so3720397qtc.11 for ; Mon, 22 Apr 2019 20:06:48 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=endlessm-com.20150623.gappssmtp.com; s=20150623; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=iCDEEjaXFN2ttbzzYVmz/c+842djhuXLkuUAp4+5DKk=; b=YjSoeKBWJEPWfxHSo+UsMp7qTB0PdsBkOtCvHq4zFY0IrPa5vOPy8ywxS7QSjyShWc Ol1vf1qT4sbjel1DHXiEabJQOgcVwR9CkrWr7tCuc8q2HB9voTU/odLAZuCoLd9ou5bO 3X3b+QPB9Y71pejtNWb6E7QBm5oDbXfnKxy1A8094TMuBg/bzdeHpIIcMvQYkxEfO5Z7 OPgdZqx7hPVB7sPbMZnQCi8J70GjcpiLIK2mP5WVeOHC6PYwfCjztPOgV+Ui3idkSwP+ lpG7KbVIFVOFEr1inbqFRgw/wnlVe44B7yVRAuofuPXOclQIWX8xCgfN7nK0wlpRvMSV kEZA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=iCDEEjaXFN2ttbzzYVmz/c+842djhuXLkuUAp4+5DKk=; b=ISvZUDAt/v3A2mazBUmV+uh0jhYf1FopAvfS2OCO+UgfGeF6k/FU0r/aZWKRjy0gxu 5M2hxtmXnJSj08WdBeX3S0mAZDVk89qHektwoIccJhFBXlM24IZG32voC3QTEkD1m+z7 FdNElmiBKZuv8TfISUfAJopeOSy25L7u2ObFinxPLo7EfR1nDAlRChp920+v8wWOl2UT cBwWbwC2O+56IVT9111foXQBq8vRZh5lCBHQw5x2NE5F5dHjge/0yxUv9VpbjCh7kn1s AQZWihBw2eMJQYuF+0CdZlEZbsUqu4IcTYQ6vBrtM3dUno2BTbIRLHOCpkDPHvwHrQnL DAsQ== X-Gm-Message-State: APjAAAXQgbxCchSMDL/heMBtcdyInxGaQrMBR7IlMNzuTCUP/ipwG0zk dJI2v83UuYNhlo1pOvoDi5uEDTbE9FD8b65YO8figg== X-Received: by 2002:a0c:9945:: with SMTP id i5mr18382547qvd.159.1555988807902; Mon, 22 Apr 2019 20:06:47 -0700 (PDT) MIME-Version: 1.0 References: <20190422101526.28415-1-drake@endlessm.com> <20190422101526.28415-2-drake@endlessm.com> <20190422120451.GA9499@gmail.com> In-Reply-To: <20190422120451.GA9499@gmail.com> From: Daniel Drake Date: Tue, 23 Apr 2019 11:06:36 +0800 Message-ID: Subject: Re: [PATCH 2/2] x86/tsc: set LAPIC timer frequency to crystal clock frequency To: Ingo Molnar Cc: Thomas Gleixner , Ingo Molnar , Borislav Petkov , hpa@zytor.com, x86@kernel.org, Linux Kernel , Len Brown , "Wysocki, Rafael J" , Linux Upstreaming Team Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Apr 22, 2019 at 8:04 PM Ingo Molnar wrote: > Minor style nit: the parentheses are unnecessary, integer expressions > like this are evaluated left to right and multiplication and division has > the same precedence. Fair point, although the same could be said for cpu_khz_from_msr(). > But it might also make sense to actually store crystal_mhz instead of > crystal_khz, because both CPUID 15H and 16H provides MHz values. > > That way the above expression would simplify to: > > lapic_timer_frequency = crystal_mhz / HZ; Wouldn't it be lapic_timer_frequency = crystal_mhz * 1000000 / HZ; ? Thanks Daniel