From: Elliott Mitchell <ehem+xen@m5p.com>
To: Neowutran <xen@neowutran.ovh>
Cc: xen-devel@lists.xenproject.org, "Jan Beulich" <jbeulich@suse.com>,
"Andrew Cooper" <andrew.cooper3@citrix.com>,
"Roger Pau Monné" <roger.pau@citrix.com>, "Wei Liu" <wl@xen.org>
Subject: Re: [PATCH v2] Bug fix - Integer overflow when cpu frequency > u32 max value.
Date: Sun, 18 Dec 2022 13:47:44 -0800 [thread overview]
Message-ID: <Y5+KgBxxmlYfq7e2@mattapan.m5p.com> (raw)
In-Reply-To: <d018f50f439c4d8dff8add022d28698a2af3c320.1671365645.git.xen@neowutran.ovh>
On Sun, Dec 18, 2022 at 01:14:07PM +0100, Neowutran wrote:
> xen/x86: prevent overflow with high frequency TSCs
>
> Promote tsc_khz to a 64-bit type before multiplying by 1000 to avoid a
> 'overflow before widen' bug.
> Otherwise just above 4.294GHz the value will overflow.
> Processors with clocks this high are now in production and require this to work
> correctly.
>
> Signed-off-by: Neowutran <xen@neowutran.ovh>
Needing a bit of word-wrapping, but that can be adjusted during commit to
the Xen tree.
This strikes me as urgent for an updated Xen release. In the past 3
months, both large manufacturers of desktop processors have released
processors with sufficient clock speed to require this patch. This WILL
trigger *massive* numbers of bug reports very soon.
Similarly I suggest all Linux distributions which distribute Xen will
want to issue updates urgently.
--
(\___(\___(\______ --=> 8-) EHM <=-- ______/)___/)___/)
\BS ( | ehem+sigmsg@m5p.com PGP 87145445 | ) /
\_CS\ | _____ -O #include <stddisclaimer.h> O- _____ | / _/
8A19\___\_|_/58D2 7E3D DDF4 7BA6 <-PGP-> 41D1 B375 37D0 8714\_|_/___/5445
next prev parent reply other threads:[~2022-12-18 21:48 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-12-18 12:14 [PATCH v2] Bug fix - Integer overflow when cpu frequency > u32 max value Neowutran
2022-12-18 21:47 ` Elliott Mitchell [this message]
2022-12-19 8:00 ` Jan Beulich
2022-12-19 8:55 ` Neowutran
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=Y5+KgBxxmlYfq7e2@mattapan.m5p.com \
--to=ehem+xen@m5p.com \
--cc=andrew.cooper3@citrix.com \
--cc=jbeulich@suse.com \
--cc=roger.pau@citrix.com \
--cc=wl@xen.org \
--cc=xen-devel@lists.xenproject.org \
--cc=xen@neowutran.ovh \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.