Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 28 Dec 2007 21:08:27 GMT
From:      John Birrell <jb@FreeBSD.org>
To:        Perforce Change Reviews <perforce@freebsd.org>
Subject:   PERFORCE change 131902 for review
Message-ID:  <200712282108.lBSL8R7R085515@repoman.freebsd.org>

next in thread | raw e-mail | index | archive | help
http://perforce.freebsd.org/chv.cgi?CH=131902

Change 131902 by jb@jb_freebsd1 on 2007/12/28 21:07:59

	The resolution passed into cyclic_init is only used in Solaris,
	so we don't need to bother setting it.

Affected files ...

.. //depot/projects/dtrace/src/sys/cddl/amd64/cyclic_machdep.c#4 edit

Differences ...

==== //depot/projects/dtrace/src/sys/cddl/amd64/cyclic_machdep.c#4 (text+ko) ====

@@ -54,7 +54,6 @@
 	NULL		/* cyb_arg_t cyb_arg */
 };
 
-static hrtime_t resolution;
 static void	*cyclic_lock_ih;
 static void	*cyclic_low_ih;
 
@@ -100,9 +99,6 @@
 static void
 cyclic_machdep_init(void)
 {
-	/* Default the resolution. */
-	resolution = ticks * 1000000;
-
 	/*
 	 * Add a software interrupt handlers for low priority cyclic
 	 * events.
@@ -112,8 +108,11 @@
 	swi_add(&clk_intr_event, "lock cyclic", cyclic_swi_lock, NULL,
 	    SWI_TQ_FAST, 0, &cyclic_lock_ih);
 
-	/* Register the cyclic backend. */
-	cyclic_init(&be, resolution);
+	/*
+	 * Register the cyclic backend. Note that the resolution argument
+	 * is only used on Solaris, so we don't bother with that.
+	 */
+	cyclic_init(&be, 0);
 }
 
 static void



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200712282108.lBSL8R7R085515>