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 10-28-2009, 09:36 PM
Stefan Bader
 
Default SRU: Re-review of dangling Karmic patches

This is a multi-part message in MIME format.There are currently 3 Karmic patches (beside of others yet to come) which have
been applied to the tree in the gray time in between Karmic final and pre-SRU.
So I re-send those, so we can get a more formal review of those.

1. SRU Justification:

Impact: The default of the synaptics driver would take a too high repeat
rate and cause problems on the Toshiba Protege M300.

Fix: Add a quirk which checks for this model and does the right thing.

2. SRU justification:

Impact: Some USB devices claim to support long sense data but when sense
is requested, they will fail to do so. But they work when only short sense
is requested.

Fix: Try long format first but if that fails fall back to short sense data.
This patch is accepted in upstream stable.

3. SRU justification:

Impact: The nx protection patch incorrectly would tell users that it is enabled
even on hardware that does not support it.

Fix: Adding a case for unsupported hw and print a warning when this is encountered.
 
Old 10-29-2009, 09:19 AM
Andy Whitcroft
 
Default SRU: Re-review of dangling Karmic patches

On Wed, Oct 28, 2009 at 10:36:17PM +0000, Stefan Bader wrote:
> There are currently 3 Karmic patches (beside of others yet to come) which have
> been applied to the tree in the gray time in between Karmic final and pre-SRU.
> So I re-send those, so we can get a more formal review of those.
>
> 1. SRU Justification:
>
> Impact: The default of the synaptics driver would take a too high repeat
> rate and cause problems on the Toshiba Protege M300.
>
> Fix: Add a quirk which checks for this model and does the right thing.

Looks reasonable. This is a model specific quirk with near zero
regression possibility. This is upstream (not stable).

Acked-by: Andy Whitcroft <apw@canonical.com>

> 2. SRU justification:
>
> Impact: Some USB devices claim to support long sense data but when sense
> is requested, they will fail to do so. But they work when only short sense
> is requested.
>
> Fix: Try long format first but if that fails fall back to short sense data.
> This patch is accepted in upstream stable.

This introduces a retry on fail which fixes a number of devices.
Although the retry is generally applied it does look pretty safe.

Acked-by: Andy Whitcroft <apw@canonical.com>

>
> 3. SRU justification:
>
> Impact: The nx protection patch incorrectly would tell users that it is enabled
> even on hardware that does not support it.
>
> Fix: Adding a case for unsupported hw and print a warning when this is encountered.
>

This is only a reporting change no functionally is changed by this fix.
Looks safe.

Acked-by: Andy Whitcroft <apw@canonical.com>

-apw

--
kernel-team mailing list
kernel-team@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kernel-team
 
Old 10-29-2009, 09:22 AM
Colin Ian King
 
Default SRU: Re-review of dangling Karmic patches

On Wed, 2009-10-28 at 22:36 +0000, Stefan Bader wrote:
> There are currently 3 Karmic patches (beside of others yet to come) which have
> been applied to the tree in the gray time in between Karmic final and pre-SRU.
> So I re-send those, so we can get a more formal review of those.
>
> 1. SRU Justification:
>
> Impact: The default of the synaptics driver would take a too high repeat
> rate and cause problems on the Toshiba Protege M300.
>
> Fix: Add a quirk which checks for this model and does the right thing.

ACK

>
> 2. SRU justification:
>
> Impact: Some USB devices claim to support long sense data but when sense
> is requested, they will fail to do so. But they work when only short sense
> is requested.
>
> Fix: Try long format first but if that fails fall back to short sense data.
> This patch is accepted in upstream stable.

ACK

>
> 3. SRU justification:
>
> Impact: The nx protection patch incorrectly would tell users that it is enabled
> even on hardware that does not support it.
>
> Fix: Adding a case for unsupported hw and print a warning when this is encountered.
>
ACK

--
Colin King <colin.king@canonical.com>
Hardware Enable Team - Kernel Team - Canonical
Ubuntu - Linux for human beings | www.ubuntu.com | www.canonical.com


--
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 03:07 PM.

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