linux-kernel - Re: [PATCH 3/9] vt: delete unneeded functions register_con_driver|take_over

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 8 May 2013 13:45:00 -0700
From:	Greg KH <gregkh@...uxfoundation.org>
To:	Peter Hurley <peter@...leysoftware.com>
Cc:	Wang YanQing <udknight@...il.com>, airlied@...hat.com,
	linux-kernel@...r.kernel.org, jslaby@...e.cz, alan@...ux.intel.com
Subject: Re: [PATCH 3/9] vt: delete unneeded functions
 register_con_driver|take_over_console

On Wed, May 08, 2013 at 04:37:14PM -0400, Peter Hurley wrote:
> On 05/08/2013 04:14 PM, Greg KH wrote:
> >On Wed, May 08, 2013 at 03:56:49PM -0400, Peter Hurley wrote:
> >>On 05/08/2013 02:13 PM, Wang YanQing wrote:
> >>>Now there is no place use register_con_driver|take_over_console,
> >>>and we can achieve their function with do_register_con_driver|
> >>>do_take_over_console easily, so just delete them to reduce code duplication.
> >>>
> >>>Signed-off-by: Wang YanQing <udknight@...il.com>
> >>>---
> >>>  drivers/tty/vt/vt.c     | 45 ---------------------------------------------
> >>>  include/linux/console.h |  2 --
> >>>  2 files changed, 47 deletions(-)
> >>>
> >>>diff --git a/drivers/tty/vt/vt.c b/drivers/tty/vt/vt.c
> >>>index fbd447b..852d470 100644
> >>>--- a/drivers/tty/vt/vt.c
> >>>+++ b/drivers/tty/vt/vt.c
> >>>@@ -3576,26 +3576,6 @@ err:
> >>>  	return retval;
> >>>  }
> >>>
> >>>-/**
> >>>- * register_con_driver - register console driver to console layer
> >>>- * @csw: console driver
> >>>- * @first: the first console to take over, minimum value is 0
> >>>- * @last: the last console to take over, maximum value is MAX_NR_CONSOLES -1
> >>>- *
> >>>- * DESCRIPTION: This function registers a console driver which can later
> >>>- * bind to a range of consoles specified by @first and @last. It will
> >>>- * also initialize the console driver by calling con_startup().
> >>>- */
> >>>-int register_con_driver(const struct consw *csw, int first, int last)
> >>>-{
> >>>-	int retval;
> >>>-
> >>>-	console_lock();
> >>>-	retval = do_register_con_driver(csw, first, last);
> >>>-	console_unlock();
> >>>-	return retval;
> >>>-}
> >>>-EXPORT_SYMBOL(register_con_driver);
> >>
> >>Maybe register_con_driver() should print a deprecated warning
> >>for a release or two before we remove it.
> >
> >If there are no in-kernel users, why do we need to keep it around?
> 
> Well, it's not my position that we *need* to keep previously exported
> symbols around, but I do think a release or two is enough of a
> heads-up for out-of-tree drivers to transition.

I don't care about out-of-tree drivers, as we have no insight into what
they do, but they can see everything we can do, it's up to them to keep
up with us.

Or even better yet, get merged into the tree, and then these types of
things aren't even an issue.

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/