[RFC PATCH 1/3] move hvc_console headers around

Milton Miller miltonm at bga.com
Sun Dec 18 10:29:07 EST 2005


On Dec 17, 2005, at 5:38 AM, arnd at arndb.de wrote:

> On Sünnavend 17 Dezember 2005 06:02, Milton Miller wrote:
>>
>> On Dec 16, 2005, at 6:10 PM, Arnd Bergmann wrote:
>>>   */
>>>  #define HVC_ALLOC_TTY_ADAPTERS	8
>>
>> This patch misses the requested cleanup of moving this define for the
>> total number of tty devices registered to the .h
>
> What's the point of moving it to a header file if no other file needs
> to know the value?

That argument applies equally well to both defines.


>> and sanitizing that  against the number of consoles.
>
>> Currently we can recoginse the first 16 devices in the device tree and
>> choose any one of them as the kernel console, but we only allow the
>> first 8 to have a tty (and hence allow /dev/console to be opened).
>
> I understood the problem, but did not want to make functional changes 
> to
> the hvc_vio driver in my cleanup of the cleanup. What do you suggest as
> a fix for this? Should the number be limited at all?

Since one controlls static allocations and the other a runtime 
allocation having them visable makes sense.

I would propose that since no one has aparently complained, that 8 
would be enough for both.

That could be a nieve position.  Perhaps the change should be a 
seperate patch.

milton



More information about the Linuxppc64-dev mailing list