FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Ubuntu > Ubuntu Kernel Team

 
 
LinkBack Thread Tools
 
Old 04-04-2011, 07:24 PM
Leann Ogasawara
 
Default UBUNTU: SAUCE: (no-up) Restore VT fonts on switch

Hi Stefan,

When opening the Oneiric repo, the following patch produces a build
failure due to the fact that it's still a consumer of the BKL:

Author: Stefan Bader <stefan.bader@canonical.com>
Date: Tue Mar 4 22:10:36 2008 +0000

UBUNTU: SAUCE: (no-up) Restore VT fonts on switch

Not all X drivers save and restore fonts on text VTs. Add code to the
kernel to explicitly save and restore them on VT switches.

Signed-off-by: Matthew Garrett <mjg59@srcf.ucam.org>
Signed-off-by: Stefan Bader <stefan.bader@canonical.com>
Signed-off-by: Ben Collins <ben.collins@canonical.com>

I'm inclined to just drop this patch completely from Oneiric given that
the patch is 3yrs old, marked as SAUCE: (no-up), and the state of X
drivers has improved since then. We can always re-apply it should
someone scream. I wanted to get your thoughts first?

Thanks,
Leann


--
kernel-team mailing list
kernel-team@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kernel-team
 
Old 04-05-2011, 07:27 AM
Stefan Bader
 
Default UBUNTU: SAUCE: (no-up) Restore VT fonts on switch

On 04/04/2011 09:24 PM, Leann Ogasawara wrote:
> Hi Stefan,
>
> When opening the Oneiric repo, the following patch produces a build
> failure due to the fact that it's still a consumer of the BKL:
>
> Author: Stefan Bader <stefan.bader@canonical.com>
> Date: Tue Mar 4 22:10:36 2008 +0000
>
> UBUNTU: SAUCE: (no-up) Restore VT fonts on switch
>
> Not all X drivers save and restore fonts on text VTs. Add code to the
> kernel to explicitly save and restore them on VT switches.
>
> Signed-off-by: Matthew Garrett <mjg59@srcf.ucam.org>
> Signed-off-by: Stefan Bader <stefan.bader@canonical.com>
> Signed-off-by: Ben Collins <ben.collins@canonical.com>
>
> I'm inclined to just drop this patch completely from Oneiric given that
> the patch is 3yrs old, marked as SAUCE: (no-up), and the state of X
> drivers has improved since then. We can always re-apply it should
> someone scream. I wanted to get your thoughts first?
>
> Thanks,
> Leann
>
I guess it depends a bit what fallbacks we got. If memory serves this was only
for vga framebuffer and only helping if the X driver did not handle the fonts.
So basically helping for the special last resort graphical mode.
Our reasoning before was going along the lines we did for the mystic rfkill
special drivers we hump along. Which is, as long as nobody complains about it
being there and it continues to simply apply, why not keep it just in case.
Given that most users end up with i915, radeon or nouveau frambuffer devices (or
(u)efi on o) it probably does not matter that much. More the old stable
maintainers thinking: As long as there ain't complaints, dun't tuch it.

-Stefan

--
kernel-team mailing list
kernel-team@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kernel-team
 
Old 04-05-2011, 12:50 PM
Tim Gardner
 
Default UBUNTU: SAUCE: (no-up) Restore VT fonts on switch

On 04/05/2011 01:27 AM, Stefan Bader wrote:

On 04/04/2011 09:24 PM, Leann Ogasawara wrote:

Hi Stefan,

When opening the Oneiric repo, the following patch produces a build
failure due to the fact that it's still a consumer of the BKL:

Author: Stefan Bader<stefan.bader@canonical.com>
Date: Tue Mar 4 22:10:36 2008 +0000

UBUNTU: SAUCE: (no-up) Restore VT fonts on switch

Not all X drivers save and restore fonts on text VTs. Add code to the
kernel to explicitly save and restore them on VT switches.

Signed-off-by: Matthew Garrett<mjg59@srcf.ucam.org>
Signed-off-by: Stefan Bader<stefan.bader@canonical.com>
Signed-off-by: Ben Collins<ben.collins@canonical.com>

I'm inclined to just drop this patch completely from Oneiric given that
the patch is 3yrs old, marked as SAUCE: (no-up), and the state of X
drivers has improved since then. We can always re-apply it should
someone scream. I wanted to get your thoughts first?

Thanks,
Leann


I guess it depends a bit what fallbacks we got. If memory serves this was only
for vga framebuffer and only helping if the X driver did not handle the fonts.
So basically helping for the special last resort graphical mode.
Our reasoning before was going along the lines we did for the mystic rfkill
special drivers we hump along. Which is, as long as nobody complains about it
being there and it continues to simply apply, why not keep it just in case.
Given that most users end up with i915, radeon or nouveau frambuffer devices (or
(u)efi on o) it probably does not matter that much. More the old stable
maintainers thinking: As long as there ain't complaints, dun't tuch it.

-Stefan



I'm not really sure what Stefan just said, but I think we should be able
to drop this patch.


--
Tim Gardner tim.gardner@canonical.com

--
kernel-team mailing list
kernel-team@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kernel-team
 
Old 04-05-2011, 07:09 PM
Stefan Bader
 
Default UBUNTU: SAUCE: (no-up) Restore VT fonts on switch

On 04/05/2011 02:50 PM, Tim Gardner wrote:
> On 04/05/2011 01:27 AM, Stefan Bader wrote:
>> On 04/04/2011 09:24 PM, Leann Ogasawara wrote:
>>> Hi Stefan,
>>>
>>> When opening the Oneiric repo, the following patch produces a build
>>> failure due to the fact that it's still a consumer of the BKL:
>>>
>>> Author: Stefan Bader<stefan.bader@canonical.com>
>>> Date: Tue Mar 4 22:10:36 2008 +0000
>>>
>>> UBUNTU: SAUCE: (no-up) Restore VT fonts on switch
>>>
>>> Not all X drivers save and restore fonts on text VTs. Add code to the
>>> kernel to explicitly save and restore them on VT switches.
>>>
>>> Signed-off-by: Matthew Garrett<mjg59@srcf.ucam.org>
>>> Signed-off-by: Stefan Bader<stefan.bader@canonical.com>
>>> Signed-off-by: Ben Collins<ben.collins@canonical.com>
>>>
>>> I'm inclined to just drop this patch completely from Oneiric given that
>>> the patch is 3yrs old, marked as SAUCE: (no-up), and the state of X
>>> drivers has improved since then. We can always re-apply it should
>>> someone scream. I wanted to get your thoughts first?
>>>
>>> Thanks,
>>> Leann
>>>
>> I guess it depends a bit what fallbacks we got. If memory serves this was only
>> for vga framebuffer and only helping if the X driver did not handle the fonts.
>> So basically helping for the special last resort graphical mode.
>> Our reasoning before was going along the lines we did for the mystic rfkill
>> special drivers we hump along. Which is, as long as nobody complains about it
>> being there and it continues to simply apply, why not keep it just in case.
>> Given that most users end up with i915, radeon or nouveau frambuffer devices (or
>> (u)efi on o) it probably does not matter that much. More the old stable
>> maintainers thinking: As long as there ain't complaints, dun't tuch it.
>>
>> -Stefan
>>
>
> I'm not really sure what Stefan just said, but I think we should be able to drop
> this patch.
>

*sigh* Sometimes I wonder why I bother... :-P What I said was that this patch
could still be useful, even if the number of people affected likely is going down.
As long as carrying the patch does not cause problems, why should we remove it?
Just because upstream did not care that much for that detail?

-Stefan

--
kernel-team mailing list
kernel-team@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kernel-team
 

Thread Tools




All times are GMT. The time now is 01:21 PM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org