Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 23 Oct 2008 18:44:18 GMT
From:      Ed Schouten <ed@FreeBSD.org>
To:        Perforce Change Reviews <perforce@FreeBSD.org>
Subject:   PERFORCE change 151816 for review
Message-ID:  <200810231844.m9NIiIsb060300@repoman.freebsd.org>

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

Change 151816 by ed@ed_dull on 2008/10/23 18:43:33

	Commit some WIP patches/tests.

Affected files ...

.. //depot/projects/mpsafetty/sys/conf/files#19 edit
.. //depot/projects/mpsafetty/sys/dev/dcons/dcons_os.c#3 edit
.. //depot/projects/mpsafetty/sys/dev/uart/uart_tty.c#4 edit
.. //depot/projects/mpsafetty/sys/kern/subr_clist.c#7 delete
.. //depot/projects/mpsafetty/sys/sys/clist.h#5 delete

Differences ...

==== //depot/projects/mpsafetty/sys/conf/files#19 (text+ko) ====

@@ -1662,7 +1662,6 @@
 kern/subr_autoconf.c		standard
 kern/subr_blist.c		standard
 kern/subr_bus.c			standard
-kern/subr_clist.c		standard
 kern/subr_clock.c		standard
 kern/subr_devstat.c		standard
 kern/subr_disk.c		standard

==== //depot/projects/mpsafetty/sys/dev/dcons/dcons_os.c#3 (text+ko) ====

@@ -126,11 +126,13 @@
 extern struct gdb_dbgport *gdb_cur;
 #endif
 
-static tsw_outwakeup_t dcons_outwakeup;
+static tsw_outwakeup_t	dcons_outwakeup;
+static tsw_free_t	dcons_free;
 
 static struct ttydevsw dcons_ttydevsw = {
 	.tsw_flags      = TF_NOPREFIX,
 	.tsw_outwakeup  = dcons_outwakeup,
+	.tsw_free	= dcons_free,
 };
 
 #if (defined(GDB) || defined(DDB)) && defined(ALT_BREAK_TO_DEBUGGER)
@@ -219,6 +221,16 @@
 }
 
 static void
+dcons_free(void *arg)
+{
+
+	/*
+	 * XXX: dcons(4) should not reuse the device name before this
+	 * function has been called!
+	 */
+}
+
+static void
 dcons_timeout(void *v)
 {
 	struct	tty *tp;

==== //depot/projects/mpsafetty/sys/dev/uart/uart_tty.c#4 (text+ko) ====

@@ -327,6 +327,16 @@
 	tty_unlock(tp);
 }
 
+static void
+uart_tty_free(void *arg)
+{
+
+	/*
+	 * XXX: uart(4) could reuse the device unit number before it is
+	 * being freed by the TTY layer!
+	 */
+}
+
 static struct ttydevsw uart_tty_class = {
 	.tsw_flags	= TF_INITLOCK|TF_CALLOUT,
 	.tsw_open	= uart_tty_open,
@@ -335,6 +345,7 @@
 	.tsw_ioctl	= uart_tty_ioctl,
 	.tsw_param	= uart_tty_param,
 	.tsw_modem	= uart_tty_modem,
+	.tsw_free	= uart_tty_free,
 };
 
 int



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