Google Groups no longer supports new Usenet posts or subscriptions. Historical content remains viewable.
Dismiss

Re: SOLVED: Time Clock Stops in FreeBSD 9.0 guest running under ESXi 5.0

74 views
Skip to first unread message

Adam Strohl

unread,
Aug 3, 2012, 3:38:44 AM8/3/12
to
Just a heads up on the original issue, which is FreeBSD's timer/clock
stopping under ESXi 5.0 and some later versions of VMware Workstation.

I've gotten a few direct messages that this thread ranks high on Google
but people are missing the solution. A few months ago I found this
forum posting (I believe this was linked in this thread already)
http://unix.derkeiler.com/Mailing-Lists/FreeBSD/stable/2012-03/msg00201.html


The long and short of it is that changing the kern.timecounter sysctl
value to ACPI-fast or (ACPI-safe if you're not running 9.x yet) fixes
the hanging issue so far for us.

To temporarily enable it under 9.x:
sysctl kern.timecounter.hardware=ACPI-fast

Pre 9.x (which doesn't have the ACPI-fast mode):
sysctl kern.timecounter.hardware=ACPI-safe

To make this persist across reboots and be enabled by default add this
line to your /etc/sysctl.conf

Under 9.x:
kern.timecounter.hardware=ACPI-fast

Pre 9.x:
kern.timecounter.hardware=ACPI-safe

Hope this helps anyone running across this issue.

--
Adam Strohl
http://www.ateamsystems.com/

_______________________________________________
freebsd...@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-stable
To unsubscribe, send any mail to "freebsd-stabl...@freebsd.org"

Adam Strohl

unread,
Aug 3, 2012, 3:59:10 AM8/3/12
to
Doh, correct URL for the forum post is:
http://forums.freebsd.org/showthread.php?t=31929&page=2

Marcelo Gondim

unread,
Aug 3, 2012, 8:06:19 AM8/3/12
to
Hi all,

I sent a PR [1] but I decided to also send the problem here.
If you try to destroy a geom_virstor that does not exist, this causes a
kernel panic immediately.

Just try:

gvirstor load
gvirstor destroy tatata

# uname -a
FreeBSD zeus.xxxx.xxx.br 9.1-PRERELEASE FreeBSD 9.1-PRERELEASE #27: Mon
Jul 16 01:41:24 BRT 2012
ro...@zeus.xxxx.xxx.br:/usr/obj/usr/src/sys/GONDIM amd64

[1] http://www.freebsd.org/cgi/query-pr.cgi?pr=170199

Best regards,
Gondim

Mark Saad

unread,
Aug 3, 2012, 8:29:20 AM8/3/12
to


---
> _______________________________________________
> freebsd...@freebsd.org mailing list
> http://lists.freebsd.org/mailman/listinfo/freebsd-stable
> To unsubscribe, send any mail to "freebsd-stabl...@freebsd.org"

Did you ask VMware to update this kb ?

http://kb.vmware.com/selfservice/microsites/search.do?language=en_US&cmd=displayKC&externalId=1006427

They need to have a FreeBSD section .

It's slightly sad VMware could. It figure this out . They have notes on disabling hpet due to screwy clock issues under Linux , wonder why they never tried to disable it under FreeBSD ?

---
Mark saad | mark...@longcount.org

Jim Harris

unread,
Aug 3, 2012, 12:49:41 PM8/3/12
to
On Fri, Aug 3, 2012 at 5:06 AM, Marcelo Gondim <gon...@bsdinfo.com.br> wrote:
> Hi all,
>
> I sent a PR [1] but I decided to also send the problem here.
> If you try to destroy a geom_virstor that does not exist, this causes a
> kernel panic immediately.
>
> Just try:
>
> gvirstor load
> gvirstor destroy tatata
>
> # uname -a
> FreeBSD zeus.xxxx.xxx.br 9.1-PRERELEASE FreeBSD 9.1-PRERELEASE #27: Mon Jul
> 16 01:41:24 BRT 2012 ro...@zeus.xxxx.xxx.br:/usr/obj/usr/src/sys/GONDIM
> amd64
>
> [1] http://www.freebsd.org/cgi/query-pr.cgi?pr=170199
>
> Best regards,
> Gondim
>

Hi Gondim,

Can you test the following patch?

Index: sys/geom/virstor/g_virstor.c
===================================================================
--- sys/geom/virstor/g_virstor.c (revision 238909)
+++ sys/geom/virstor/g_virstor.c (working copy)
@@ -235,6 +235,12 @@
return;
}
sc = virstor_find_geom(cp, name);
+ if (sc == NULL) {
+ gctl_error(req, "Don't know anything about '%s'", name);
+ g_topology_unlock();
+ return;
+ }
+
LOG_MSG(LVL_INFO, "Stopping %s by the userland command",
sc->geom->name);
update_metadata(sc);

Thanks,

-Jim

Marcelo Gondim

unread,
Aug 3, 2012, 1:04:05 PM8/3/12
to
Em 03/08/2012 13:49, Jim Harris escreveu:
> ===================================================================
> --- sys/geom/virstor/g_virstor.c (revision 238909)
> +++ sys/geom/virstor/g_virstor.c (working copy)
> @@ -235,6 +235,12 @@
> return;
> }
> sc = virstor_find_geom(cp, name);
> + if (sc == NULL) {
> + gctl_error(req, "Don't know anything about '%s'", name);
> + g_topology_unlock();
> + return;
> + }
> +
> LOG_MSG(LVL_INFO, "Stopping %s by the userland command",
> sc->geom->name);
> update_metadata(sc);
Hi Jim,

When I applied the patch gave this error:

# patch < /root/patch.diff
Hmm... Looks like a unified diff to me...
The text leading up to this was:
--------------------------
|--- sys/geom/virstor/g_virstor.c (revision 238909)
|+++ sys/geom/virstor/g_virstor.c (working copy)
--------------------------
Patching file sys/geom/virstor/g_virstor.c using Plan A...
Hunk #1 failed at 235.
1 out of 1 hunks failed--saving rejects to sys/geom/virstor/g_virstor.c.rej
done




# cat sys/geom/virstor/g_virstor.c.rej
***************
*** 235,240 ****
return;
}
sc = virstor_find_geom(cp, name);
LOG_MSG(LVL_INFO, "Stopping %s by the userland command",
sc->geom->name);
update_metadata(sc);
--- 235,246 ----
return;
}
sc = virstor_find_geom(cp, name);
+ if (sc == NULL) {
+ gctl_error(req, "Don't know anything about
'%s'", name);
+ g_topology_unlock();
+ return;
+ }
+
LOG_MSG(LVL_INFO, "Stopping %s by the userland command",
sc->geom->name);
update_metadata(sc);

Marcelo Gondim

unread,
Aug 3, 2012, 1:47:18 PM8/3/12
to
Em 03/08/2012 14:22, Jim Harris escreveu:
> On Fri, Aug 3, 2012 at 10:04 AM, Marcelo Gondim <gon...@bsdinfo.com.br> wrote:
>> Hi Jim,
>>
>> When I applied the patch gave this error:
>>
>> # patch < /root/patch.diff
>> Hmm... Looks like a unified diff to me...
>> The text leading up to this was:
>> --------------------------
>>
>> |--- sys/geom/virstor/g_virstor.c (revision 238909)
>> |+++ sys/geom/virstor/g_virstor.c (working copy)
>> --------------------------
>> Patching file sys/geom/virstor/g_virstor.c using Plan A...
>> Hunk #1 failed at 235.
>> 1 out of 1 hunks failed--saving rejects to sys/geom/virstor/g_virstor.c.rej
>> done
>>
>>
>>
> Strange. It applies with no issues on my checkout.
>
> Let's try an attachment instead. If this doesn't work, could you
> kindly apply the patch by hand?
>
> Thanks,
>
> -Jim
Hi Jim,

# patch < /root/patch.diff
Hmm... Looks like a unified diff to me...
The text leading up to this was:
--------------------------
|Index: sys/geom/virstor/g_virstor.c
|===================================================================
|--- sys/geom/virstor/g_virstor.c (revision 239019)
|+++ sys/geom/virstor/g_virstor.c (working copy)
--------------------------
Patching file sys/geom/virstor/g_virstor.c using Plan A...
Hunk #1 succeeded at 234.
done

I will now compile and test.
I will return with the result. :)

Marcelo Gondim

unread,
Aug 3, 2012, 4:12:48 PM8/3/12
to
Em 03/08/2012 14:22, Jim Harris escreveu:
> On Fri, Aug 3, 2012 at 10:04 AM, Marcelo Gondim <gon...@bsdinfo.com.br> wrote:
>> Hi Jim,
>>
>> When I applied the patch gave this error:
>>
>> # patch < /root/patch.diff
>> Hmm... Looks like a unified diff to me...
>> The text leading up to this was:
>> --------------------------
>>
>> |--- sys/geom/virstor/g_virstor.c (revision 238909)
>> |+++ sys/geom/virstor/g_virstor.c (working copy)
>> --------------------------
>> Patching file sys/geom/virstor/g_virstor.c using Plan A...
>> Hunk #1 failed at 235.
>> 1 out of 1 hunks failed--saving rejects to sys/geom/virstor/g_virstor.c.rej
>> done
>>
>>
>>
> Strange. It applies with no issues on my checkout.
>
> Let's try an attachment instead. If this doesn't work, could you
> kindly apply the patch by hand?
>
> Thanks,
>
> -Jim
Hi Jim,

Perfect!!!

# gvirstor destroy tudo
gvirstor: Don't know anything about 'tudo'

Jim Harris

unread,
Aug 3, 2012, 4:36:11 PM8/3/12
to
On Fri, Aug 3, 2012 at 1:12 PM, Marcelo Gondim <gon...@bsdinfo.com.br> wrote:
>
> Hi Jim,
>
> Perfect!!!
>
> # gvirstor destroy tudo
> gvirstor: Don't know anything about 'tudo'
>
>
>

Patch applied to head as r239021. I have requested approval from re@
to merge to stable/9.

Thank you for confirming the patch on your end.

Regards,

-Jim
0 new messages