Date: Sun, 29 Oct 2017 19:30:25 -0700 From: Peter Grehan <grehan@freebsd.org> To: Ian Lepore <ian@freebsd.org>, src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r325108 - head/sys/amd64/vmm/io Message-ID: <66f496cb-c2c4-3044-333e-84540bdd64c2@freebsd.org> In-Reply-To: <1509330161.21609.86.camel@freebsd.org> References: <201710292050.v9TKo3j5058456@repo.freebsd.org> <1509310495.21609.60.camel@freebsd.org> <3b1ffc68-def3-04f6-2a1e-af44d6f3b4a5@freebsd.org> <1509330161.21609.86.camel@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Hi Ian, > In fact I did check maintainers, and I just now checked it again, and > seeing nothing on-point, wrote a really snarky reply. >=20 > Luckily, before hitting send I realized you wouldn't have said anything > unless there was an entry in there, so I checked like 4 more times > before I noticed the line that begins vmm(4). =C2=A0I had been looking = for a > line that began with sys/amd64. >=20 > So... sorry about that. =C2=A0Should I revert it and open a review? No, it's fine. It's a good idea in general for code in that area since=20 there are often unintended side-effects with guest behaviour, and=20 putting up a review can at least flush out folk who might have more=20 insight into that. I'll update the maintainer's file to include the directory. > Longer-term, I was hoping to find some time over the next few weeks to > further explore the roundoff errors in the timers and attempt a better > fix, and I was definitely intending to put that up for review > (especially because it involves math, and I'm always better off asking > for help with that). I know Rod Grimes is going to be doing some testing this week with=20 clock drift in guests, and I'll do some dtrace work on HPET usage with=20 non-FreeBSD guests. Timer accuracy has long been an issue with bhyve so any help in this=20 area is much appreciated. later, Peter.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?66f496cb-c2c4-3044-333e-84540bdd64c2>