From owner-svn-src-head@freebsd.org Thu Nov 30 20:47:30 2017 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 40D19DB9D75; Thu, 30 Nov 2017 20:47:30 +0000 (UTC) (envelope-from jkim@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2610:1c1:1:6074::16:84]) by mx1.freebsd.org (Postfix) with ESMTP id ADD7F749C6; Thu, 30 Nov 2017 20:47:29 +0000 (UTC) (envelope-from jkim@FreeBSD.org) Subject: Re: svn commit: r326383 - head/sys/x86/cpufreq To: cem@freebsd.org Cc: src-committers , svn-src-all@freebsd.org, svn-src-head@freebsd.org References: <201711300140.vAU1e7dC001292@repo.freebsd.org> From: Jung-uk Kim Message-ID: <31ee136d-164e-bb9b-3a90-71bb7f55595a@FreeBSD.org> Date: Thu, 30 Nov 2017 15:47:24 -0500 User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:52.0) Gecko/20100101 Thunderbird/52.5.0 MIME-Version: 1.0 In-Reply-To: Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="mjM0GTAu8uddWmuwoHR3JvW1kOfGtFD3N" X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.25 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 30 Nov 2017 20:47:30 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --mjM0GTAu8uddWmuwoHR3JvW1kOfGtFD3N Content-Type: multipart/mixed; boundary="EPKt3dBLGuuwoOdQabVxqOen535SjfDbL"; protected-headers="v1" From: Jung-uk Kim To: cem@freebsd.org Cc: src-committers , svn-src-all@freebsd.org, svn-src-head@freebsd.org Message-ID: <31ee136d-164e-bb9b-3a90-71bb7f55595a@FreeBSD.org> Subject: Re: svn commit: r326383 - head/sys/x86/cpufreq References: <201711300140.vAU1e7dC001292@repo.freebsd.org> In-Reply-To: --EPKt3dBLGuuwoOdQabVxqOen535SjfDbL Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: quoted-printable On 11/30/2017 15:31, Conrad Meyer wrote: > On Thu, Nov 30, 2017 at 12:08 PM, Jung-uk Kim wrote:= >> On 11/30/2017 14:32, Conrad Meyer wrote: >>> Hi, >>> >>> I don't think this answers the second question about the conditional.= >>> It seems like PCPU_GET() for the initial CPU should be pulled out of >>> the loop, which binds the thread to a different CPU every iteration. >> >> Ah, good catch. I'll fix it soon. Sorry. >=20 > Thanks! :-) >=20 >>> Also, as a side effect of disabling verification, you have fixed PR >>> 221621, 219213, and probably 218262. >> >> Probably. However, I am just trying to fix my FX-8350 and A10-6800 an= d >> I don't have Zen processors to verify the MSRs are actually working on= >> those CPUs. >=20 > I have one, I can verify if needed (although the change looks good to > me). On some Zen systems (including mine) it seems that the hardware > can successfully set a P-state, but will fail to read it back. For me > it is P1 but other users have reported P0. That's the root issue of > all of those PRs. If reading back isn't required, maybe that's a > solution to the issue. Reading back was not really necessary (aka "fire-and-forget"). However, we wanted to make sure all cores are in the same P-state before returning to the caller. Back in the days, it wasn't a big deal because we only had few cores to deal with and never expected to see complex topologies such as the Threadripper, I guess. :-/ Jung-uk Kim --EPKt3dBLGuuwoOdQabVxqOen535SjfDbL-- --mjM0GTAu8uddWmuwoHR3JvW1kOfGtFD3N Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAEBCAAdFiEEl1bqgKaRyqfWXu/CfJ+WJvzb8UYFAlogblwACgkQfJ+WJvzb 8UYgEgf/a8TAc1wVVJqD5RL9i6cRCnRkSm/yQZ3yE+hllhv2zMm3aauw9WuLlEa/ 028sNj4AOEemT9/9fb5z6LfWWLo9o5pMuFd8Z2nv7ROQ67ymwPlOzn1TAYWHqXsx SW4z3IL/KodAMMB1x61MFXFrGcZAiBU8VJcmQipgPHDBjPodLD60cwXGLj/ESipu 9GxOaYx3ajm4Lp6ua6UeuMatEDUcAkvoVSzXNBg4oSzDXeimeaHmnzr/olvI95l8 gNjy44GT4m3r4x3WoKrbuRyXI1GFAyWpZbuM7+8B57vECFsb+6w29tZr7xhsZiDA waixoyUWvrC08GDFnxapg9RjBGxhRQ== =M9xj -----END PGP SIGNATURE----- --mjM0GTAu8uddWmuwoHR3JvW1kOfGtFD3N--