From: Russell King <rmk+lkml@arm.linux.org.uk>
To: David Miller <davem@davemloft.net>
Cc: linux-kernel@vger.kernel.org, linuxppc-dev@ozlabs.org,
paulus@samba.org, dwmw2@infradead.org, alan@lxorguk.ukuu.org.uk
Subject: Re: [PATCH] Stop pmac_zilog from abusing 8250's device numbers.
Date: Wed, 4 Apr 2007 08:47:36 +0100 [thread overview]
Message-ID: <20070404074736.GA3440@flint.arm.linux.org.uk> (raw)
In-Reply-To: <20070403.182125.77058566.davem@davemloft.net>
On Tue, Apr 03, 2007 at 06:21:25PM -0700, David Miller wrote:
> From: Alan Cox <alan@lxorguk.ukuu.org.uk>
> Date: Wed, 4 Apr 2007 02:19:24 +0100
>
> > > I totally agree with Paul, the onboard serial device should get
> > > ttyS0 regardless of what hardare is used to drive it.
> >
> > Thats between you and udev.
>
> That might be true when udev exists, but it doesn't for the console
> specification on the kernel command line.
FACT: you can only have one struct console with one name.
FACT: If you have two struct consoles using the same name, the driver
which wins the priviledge of console is unpredictable.
FACT: there is no way to know at any particular driver registration time
how many "generic" serial ports will be available - as required for the
chardev and tty layers.
There are very real issues that need fixing deep within the kernel
before you can even think about the possibility of *PROPERLY* supporting
all serial ports beneath one namespace.
Before someone whinges "we did it in 2.4" my answer is: yes, you hacked
around the problem in 2.4 creating an utter mess in the process.
If you want to see what an utter mess pushing all serial ports towards
ttyS* creates in 2.5 kernels (which will be similar for 2.6), look at
these patches:
http://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=1427/1
http://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=1428/1
http://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=1429/1
http://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=1431/1
http://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=1432/1
http://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=1433/1
http://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=1435/2
http://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=1436/2
Moreover, note that the serial core is statically configured with the
*maximum* number of ports it will ever support. Tough luck if you're
a distro and wanting to allow users to support multi-port serial cards.
Since we've been where we are for years and years, my take on this
subject has always been: we should *not* abuse any of the existing
allocations. If we want to create a single namespace for serial ports
we should obtain a *new* allocation and migrate drivers over to that
namespace, avoiding any bastardisation of existing drivers. Modulo
fixing the very real issues mentioned above first.
Feel free to continue whinging about it for a few more years.
--
Russell King
Linux kernel 2.6 ARM Linux - http://www.arm.linux.org.uk/
maintainer of:
next prev parent reply other threads:[~2007-04-04 7:47 UTC|newest]
Thread overview: 99+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-04-03 14:25 [PATCH] Stop pmac_zilog from abusing 8250's device numbers David Woodhouse
2007-04-03 20:37 ` Brad Boyer
2007-04-03 22:58 ` David Woodhouse
2007-04-03 23:14 ` Paul Mackerras
2007-04-03 23:28 ` David Woodhouse
2007-04-03 21:29 ` Brad Boyer
2007-04-03 23:57 ` David Woodhouse
[not found] ` <200704032210 02.GA13210@cynthia.pants.nu>
2007-04-03 22:10 ` Brad Boyer
2007-04-04 0:54 ` David Woodhouse
2007-04-03 23:09 ` Brad Boyer
2007-04-04 1:38 ` Alan Cox
2007-04-04 2:10 ` Valdis.Kletnieks
2007-04-04 1:57 ` David Lang
2007-04-04 8:51 ` Russell King
2007-04-04 9:48 ` Geert Uytterhoeven
2007-04-04 16:09 ` H. Peter Anvin
2007-04-04 16:22 ` Linas Vepstas
2007-04-04 16:34 ` David Woodhouse
2007-04-04 17:38 ` Linas Vepstas
2007-04-04 1:16 ` David Miller
2007-04-04 1:28 ` Alan Cox
2007-04-04 1:07 ` David Lang
2007-04-04 1:44 ` Alan Cox
2007-04-04 1:23 ` David Lang
2007-04-04 1:32 ` David Lang
2007-04-04 16:07 ` H. Peter Anvin
2007-04-04 17:50 ` Linas Vepstas
2007-04-04 1:50 ` David Miller
2007-04-04 11:14 ` Alan Cox
2007-04-04 12:20 ` Olivier Galibert
2007-04-04 1:48 ` David Miller
2007-04-04 3:07 ` Gerhard Mack
2007-04-11 16:30 ` Benny Amorsen
2007-04-12 15:34 ` Roland Dreier
2007-04-13 2:45 ` Gerhard Mack
2007-04-13 2:24 ` David Lang
2007-04-13 8:09 ` Geert Uytterhoeven
2007-04-04 0:02 ` David Miller
2007-04-04 0:12 ` David Woodhouse
2007-04-03 22:14 ` Brad Boyer
2007-04-04 1:13 ` Benjamin Herrenschmidt
2007-04-04 1:17 ` David Miller
2007-04-04 1:32 ` Benjamin Herrenschmidt
2007-04-04 0:01 ` [PATCH v2] " David Woodhouse
2007-04-04 1:25 ` [PATCH] " Alan Cox
2007-04-04 4:20 ` Paul Mackerras
2007-04-04 5:19 ` David Woodhouse
2007-04-04 5:50 ` David Miller
2007-04-04 7:04 ` Geert Uytterhoeven
2007-04-04 13:43 ` Maciej W. Rozycki
2007-04-04 7:52 ` Russell King
2007-04-04 8:41 ` David Miller
2007-04-04 5:53 ` Paul Mackerras
2007-04-04 6:03 ` David Woodhouse
2007-04-04 6:31 ` Paul Mackerras
2007-04-04 14:19 ` [PATCH v3] Stop pmac_zilog from abusing 8250's device numbers; optionally David Woodhouse
2007-04-04 23:48 ` Paul Mackerras
2007-04-05 12:45 ` David Woodhouse
2007-04-05 22:53 ` Paul Mackerras
2007-04-05 22:59 ` David Woodhouse
2007-04-05 14:31 ` Atsushi Nemoto
2007-04-05 14:47 ` David Woodhouse
2007-04-05 15:09 ` Atsushi Nemoto
2007-04-05 15:18 ` David Woodhouse
2007-04-05 15:49 ` Atsushi Nemoto
2007-04-05 15:56 ` Geert Uytterhoeven
2007-04-04 8:12 ` [PATCH] Stop pmac_zilog from abusing 8250's device numbers David Miller
2007-04-04 8:38 ` Russell King
2007-04-04 8:43 ` David Miller
2007-04-04 9:00 ` Russell King
2007-04-04 19:58 ` David Miller
2007-04-04 12:21 ` Theodore Tso
2007-04-04 16:10 ` Russell King
2007-04-04 16:16 ` H. Peter Anvin
2007-04-04 16:25 ` Alan Cox
2007-04-04 17:41 ` Theodore Tso
2007-04-04 18:15 ` Russell King
2007-04-04 19:50 ` David Woodhouse
2007-04-05 6:05 ` Paul Mackerras
2007-04-05 8:50 ` Russell King
2007-04-05 16:15 ` Olivier Galibert
2007-04-05 21:41 ` Pavel Machek
2007-04-03 23:56 ` David Miller
2007-04-04 1:13 ` Benjamin Herrenschmidt
2007-04-04 6:58 ` Paul Mackerras
2007-04-04 14:16 ` David Woodhouse
2007-04-03 23:55 ` David Miller
2007-04-04 1:19 ` Alan Cox
2007-04-04 1:21 ` David Miller
2007-04-04 1:36 ` Alan Cox
2007-04-04 1:47 ` David Miller
2007-04-04 7:47 ` Russell King [this message]
2007-04-04 9:55 ` Paul Mackerras
2007-04-04 16:06 ` Russell King
2007-04-04 6:58 ` Geert Uytterhoeven
2007-08-14 11:49 ` Olaf Hering
2007-08-14 11:53 ` David Woodhouse
2007-08-14 16:24 ` Olaf Hering
2007-04-04 7:55 ` Olaf Hering
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20070404074736.GA3440@flint.arm.linux.org.uk \
--to=rmk+lkml@arm.linux.org.uk \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=davem@davemloft.net \
--cc=dwmw2@infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linuxppc-dev@ozlabs.org \
--cc=paulus@samba.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).