Mailing List Archive: Linux: Kernel

Reverting 5d423 fixes loading of ath9k on Acer Extensa 7630EZ

Index | Next | Previous | View Threaded

mcgrof at gmail

Nov 4, 2009, 3:40 PM

Post #1 of 11 (79 views)
Permalink

Reverting 5d423 fixes loading of ath9k on Acer Extensa 7630EZ

Ingo,

reverting the commit 5d423, titled "x86/pci: remove rounding quirk
from e820_setup_gap()" fixes loading of ath9k on an Acer Extensa
7630EZ. We've troubleshooted the issue with the user, Bernhard, who
eventually did a full bisect between 2.6.30 and 2.6.31 and found 5d423
was the culprit. For details please feel free to check:

http://bugzilla.kernel.org/show_bug.cgi?id=14402

Can you please consider reviewing this issue and help determine if
this indeed needs to be reverted for 2.6.32 and the next 2.6.31.y.

I am curious if other devices would work by reverting this as well.

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

torvalds at linux-foundation

Nov 4, 2009, 3:57 PM

Post #2 of 11 (76 views)
Permalink

Re: Reverting 5d423 fixes loading of ath9k on Acer Extensa 7630EZ [In reply to]

On Wed, 4 Nov 2009, Luis R. Rodriguez wrote:
>
> Can you please consider reviewing this issue and help determine if
> this indeed needs to be reverted for 2.6.32 and the next 2.6.31.y.
>
> I am curious if other devices would work by reverting this as well.
> [ ... ] For details please feel free to check:
>
> http://bugzilla.kernel.org/show_bug.cgi?id=14402

That same commit was the cause for

http://bugzilla.kernel.org/show_bug.cgi?id=13940

and we just increased the rounding to make it go away (see commit
15b812f1). But that was a hack.

And if that didn't help the ath9k case, then we should just revert
entirely.

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

mcgrof at gmail

Nov 4, 2009, 4:16 PM

Post #3 of 11 (76 views)
Permalink

Re: Reverting 5d423 fixes loading of ath9k on Acer Extensa 7630EZ [In reply to]

On Wed, Nov 4, 2009 at 3:57 PM, Linus Torvalds
<torvalds[at]linux-foundation.org> wrote:
>
>
> On Wed, 4 Nov 2009, Luis R. Rodriguez wrote:
>>
>> Can you please consider reviewing this issue and help determine if
>> this indeed needs to be reverted for 2.6.32 and the next 2.6.31.y.
>>
>> I am curious if other devices would work by reverting this as well.
>> [ ... ] For details please feel free to check:
>>
>> http://bugzilla.kernel.org/show_bug.cgi?id=14402
>
> That same commit was the cause for
>
> http://bugzilla.kernel.org/show_bug.cgi?id=13940
>
> and we just increased the rounding to make it go away (see commit
> 15b812f1). But that was a hack.
>
> And if that didn't help the ath9k case, then we should just revert
> entirely.

Alright, will see if Bernhard is up for testing the 15b812f1 hack and
report back.

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

yinghai at kernel

Nov 4, 2009, 5:30 PM

Post #4 of 11 (70 views)
Permalink

Re: Reverting 5d423 fixes loading of ath9k on Acer Extensa 7630EZ [In reply to]

Luis R. Rodriguez wrote:
> On Wed, Nov 4, 2009 at 3:57 PM, Linus Torvalds
> <torvalds[at]linux-foundation.org> wrote:
>>
>> On Wed, 4 Nov 2009, Luis R. Rodriguez wrote:
>>> Can you please consider reviewing this issue and help determine if
>>> this indeed needs to be reverted for 2.6.32 and the next 2.6.31.y.
>>>
>>> I am curious if other devices would work by reverting this as well.
>>> [ ... ] For details please feel free to check:
>>>
>>> http://bugzilla.kernel.org/show_bug.cgi?id=14402
>> That same commit was the cause for
>>
>> http://bugzilla.kernel.org/show_bug.cgi?id=13940
>>
>> and we just increased the rounding to make it go away (see commit
>> 15b812f1). But that was a hack.
>>
>> And if that didn't help the ath9k case, then we should just revert
>> entirely.
>
> Alright, will see if Bernhard is up for testing the 15b812f1 hack and
> report back.

can you post whole boot log with debug?

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

mcgrof at gmail

Nov 4, 2009, 5:40 PM

Post #5 of 11 (69 views)
Permalink

Re: Reverting 5d423 fixes loading of ath9k on Acer Extensa 7630EZ [In reply to]

On Wed, Nov 4, 2009 at 5:30 PM, Yinghai Lu <yinghai[at]kernel.org> wrote:
> Luis R. Rodriguez wrote:
>> On Wed, Nov 4, 2009 at 3:57 PM, Linus Torvalds
>> <torvalds[at]linux-foundation.org> wrote:
>>>
>>> On Wed, 4 Nov 2009, Luis R. Rodriguez wrote:
>>>> Can you please consider reviewing this issue and help determine if
>>>> this indeed needs to be reverted for 2.6.32 and the next 2.6.31.y.
>>>>
>>>> I am curious if other devices would work by reverting this as well.
>>>> [ ... ] For details please feel free to check:
>>>>
>>>> http://bugzilla.kernel.org/show_bug.cgi?id=14402
>>> That same commit was the cause for
>>>
>>> http://bugzilla.kernel.org/show_bug.cgi?id=13940
>>>
>>> and we just increased the rounding to make it go away (see commit
>>> 15b812f1). But that was a hack.
>>>
>>> And if that didn't help the ath9k case, then we should just revert
>>> entirely.
>>
>> Alright, will see if Bernhard is up for testing the 15b812f1 hack and
>> report back.
>
> can you post whole boot log with debug?

You can find the logs without 15b812f1 on the bug report. I'll poke
Bernhard to poke logs with 15b812f1 applied.

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

mingo at elte

Nov 4, 2009, 11:14 PM

Post #6 of 11 (66 views)
Permalink

Re: Reverting 5d423 fixes loading of ath9k on Acer Extensa 7630EZ [In reply to]

* Linus Torvalds <torvalds[at]linux-foundation.org> wrote:

> On Wed, 4 Nov 2009, Luis R. Rodriguez wrote:
> >
> > Can you please consider reviewing this issue and help determine if
> > this indeed needs to be reverted for 2.6.32 and the next 2.6.31.y.
> >
> > I am curious if other devices would work by reverting this as well.
> > [ ... ] For details please feel free to check:
> >
> > http://bugzilla.kernel.org/show_bug.cgi?id=14402
>
> That same commit was the cause for
>
> http://bugzilla.kernel.org/show_bug.cgi?id=13940
>
> and we just increased the rounding to make it go away (see commit
> 15b812f1). But that was a hack.
>
> And if that didn't help the ath9k case, then we should just revert
> entirely.

Agreed - below is the combo 15b812f1 + 5d423ccd revert. (Would be nice
to get the boot log of the latest post-15b812f1 kernel that Yinghai
asked for before we revert, in the hope of better understanding the
problem.)

Thanks,

Ingo

Signed-off-by: Ingo Molnar <mingo[at]elte.hu>
---
diff --git a/arch/x86/kernel/e820.c b/arch/x86/kernel/e820.c
index d17d482..b322e30 100644
--- a/arch/x86/kernel/e820.c
+++ b/arch/x86/kernel/e820.c
@@ -617,7 +617,7 @@ __init int e820_search_gap(unsigned long *gapstart, unsigned long *gapsize,
*/
__init void e820_setup_gap(void)
{
- unsigned long gapstart, gapsize;
+ unsigned long gapstart, gapsize, round;
int found;

gapstart = 0x10000000;
@@ -634,9 +634,14 @@ __init void e820_setup_gap(void)
#endif

/*
- * e820_reserve_resources_late protect stolen RAM already
+ * See how much we want to round up: start off with
+ * rounding to the next 1MB area.
*/
- pci_mem_start = gapstart;
+ round = 0x100000;
+ while ((gapsize >> 4) > round)
+ round += round;
+ /* Fun with two's complement */
+ pci_mem_start = (gapstart + round) & -round;

printk(KERN_INFO
"Allocating PCI resources starting at %lx (gap: %lx:%lx)\n",
@@ -1378,8 +1383,8 @@ static unsigned long ram_alignment(resource_size_t pos)
if (mb < 16)
return 1024*1024;

- /* To 64MB for anything above that */
- return 64*1024*1024;
+ /* To 32MB for anything above that */
+ return 32*1024*1024;
}

#define MAX_RESOURCE_SIZE ((resource_size_t)-1)
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo[at]vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

mcgrof at gmail

Nov 8, 2009, 11:35 AM

Post #7 of 11 (52 views)
Permalink

Re: Reverting 5d423 fixes loading of ath9k on Acer Extensa 7630EZ [In reply to]

On Wed, Nov 4, 2009 at 11:14 PM, Ingo Molnar <mingo[at]elte.hu> wrote:
>
> * Linus Torvalds <torvalds[at]linux-foundation.org> wrote:
>
>> On Wed, 4 Nov 2009, Luis R. Rodriguez wrote:
>> >
>> > Can you please consider reviewing this issue and help determine if
>> > this indeed needs to be reverted for 2.6.32 and the next 2.6.31.y.
>> >
>> > I am curious if other devices would work by reverting this as well.
>> > [ ... ] For details please feel free to check:
>> >
>> > http://bugzilla.kernel.org/show_bug.cgi?id=14402
>>
>> That same commit was the cause for
>>
>> http://bugzilla.kernel.org/show_bug.cgi?id=13940
>>
>> and we just increased the rounding to make it go away (see commit
>> 15b812f1). But that was a hack.
>>
>> And if that didn't help the ath9k case, then we should just revert
>> entirely.
>
> Agreed - below is the combo 15b812f1 + 5d423ccd revert. (Would be nice
> to get the boot log of the latest post-15b812f1 kernel that Yinghai
> asked for before we revert, in the hope of better understanding the
> problem.)

Bernhard has confirmed the new patch fixes this issue.

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

mingo at elte

Nov 8, 2009, 11:15 PM

Post #8 of 11 (48 views)
Permalink

Re: Reverting 5d423 fixes loading of ath9k on Acer Extensa 7630EZ [In reply to]

* Luis R. Rodriguez <mcgrof[at]gmail.com> wrote:

> On Wed, Nov 4, 2009 at 11:14 PM, Ingo Molnar <mingo[at]elte.hu> wrote:
> >
> > * Linus Torvalds <torvalds[at]linux-foundation.org> wrote:
> >
> >> On Wed, 4 Nov 2009, Luis R. Rodriguez wrote:
> >> >
> >> > Can you please consider reviewing this issue and help determine if
> >> > this indeed needs to be reverted for 2.6.32 and the next 2.6.31.y.
> >> >
> >> > I am curious if other devices would work by reverting this as well.
> >> > [ ... ] For details please feel free to check:
> >> >
> >> > http://bugzilla.kernel.org/show_bug.cgi?id=14402
> >>
> >> That same commit was the cause for
> >>
> >> ?? ?? ?? http://bugzilla.kernel.org/show_bug.cgi?id=13940
> >>
> >> and we just increased the rounding to make it go away (see commit
> >> 15b812f1). But that was a hack.
> >>
> >> And if that didn't help the ath9k case, then we should just revert
> >> entirely.
> >
> > Agreed - below is the combo 15b812f1 + 5d423ccd revert. (Would be nice
> > to get the boot log of the latest post-15b812f1 kernel that Yinghai
> > asked for before we revert, in the hope of better understanding the
> > problem.)
>
> Bernhard has confirmed the new patch fixes this issue.

Good - so latest kernels should be fine (on Bernhard's box) and no
change is needed, right?

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

mcgrof at gmail

Nov 9, 2009, 7:49 AM

Post #9 of 11 (45 views)
Permalink

Re: Reverting 5d423 fixes loading of ath9k on Acer Extensa 7630EZ [In reply to]

On Sun, Nov 8, 2009 at 11:15 PM, Ingo Molnar <mingo[at]elte.hu> wrote:
>
> * Luis R. Rodriguez <mcgrof[at]gmail.com> wrote:
>
>> On Wed, Nov 4, 2009 at 11:14 PM, Ingo Molnar <mingo[at]elte.hu> wrote:
>> >
>> > * Linus Torvalds <torvalds[at]linux-foundation.org> wrote:
>> >
>> >> On Wed, 4 Nov 2009, Luis R. Rodriguez wrote:
>> >> >
>> >> > Can you please consider reviewing this issue and help determine if
>> >> > this indeed needs to be reverted for 2.6.32 and the next 2.6.31.y.
>> >> >
>> >> > I am curious if other devices would work by reverting this as well.
>> >> > [ ... ] For details please feel free to check:
>> >> >
>> >> > http://bugzilla.kernel.org/show_bug.cgi?id=14402
>> >>
>> >> That same commit was the cause for
>> >>
>> >> ?? ?? ?? http://bugzilla.kernel.org/show_bug.cgi?id=13940
>> >>
>> >> and we just increased the rounding to make it go away (see commit
>> >> 15b812f1). But that was a hack.
>> >>
>> >> And if that didn't help the ath9k case, then we should just revert
>> >> entirely.
>> >
>> > Agreed - below is the combo 15b812f1 + 5d423ccd revert. (Would be nice
>> > to get the boot log of the latest post-15b812f1 kernel that Yinghai
>> > asked for before we revert, in the hope of better understanding the
>> > problem.)
>>
>> Bernhard has confirmed the new patch fixes this issue.
>
> Good - so latest kernels should be fine (on Bernhard's box) and no
> change is needed, right?

Yea, but it does mean we need a fix propagated down to older kernels.

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

mcgrof at gmail

Nov 10, 2009, 12:30 PM

Post #10 of 11 (41 views)
Permalink

Re: Reverting 5d423 fixes loading of ath9k on Acer Extensa 7630EZ [In reply to]

On Mon, Nov 9, 2009 at 7:49 AM, Luis R. Rodriguez <mcgrof[at]gmail.com> wrote:
> On Sun, Nov 8, 2009 at 11:15 PM, Ingo Molnar <mingo[at]elte.hu> wrote:
>>
>> * Luis R. Rodriguez <mcgrof[at]gmail.com> wrote:
>>
>>> On Wed, Nov 4, 2009 at 11:14 PM, Ingo Molnar <mingo[at]elte.hu> wrote:
>>> >
>>> > * Linus Torvalds <torvalds[at]linux-foundation.org> wrote:
>>> >
>>> >> On Wed, 4 Nov 2009, Luis R. Rodriguez wrote:
>>> >> >
>>> >> > Can you please consider reviewing this issue and help determine if
>>> >> > this indeed needs to be reverted for 2.6.32 and the next 2.6.31.y.
>>> >> >
>>> >> > I am curious if other devices would work by reverting this as well.
>>> >> > [ ... ] For details please feel free to check:
>>> >> >
>>> >> > http://bugzilla.kernel.org/show_bug.cgi?id=14402
>>> >>
>>> >> That same commit was the cause for
>>> >>
>>> >> ?? ?? ?? http://bugzilla.kernel.org/show_bug.cgi?id=13940
>>> >>
>>> >> and we just increased the rounding to make it go away (see commit
>>> >> 15b812f1). But that was a hack.
>>> >>
>>> >> And if that didn't help the ath9k case, then we should just revert
>>> >> entirely.
>>> >
>>> > Agreed - below is the combo 15b812f1 + 5d423ccd revert. (Would be nice
>>> > to get the boot log of the latest post-15b812f1 kernel that Yinghai
>>> > asked for before we revert, in the hope of better understanding the
>>> > problem.)
>>>
>>> Bernhard has confirmed the new patch fixes this issue.
>>
>> Good - so latest kernels should be fine (on Bernhard's box) and no
>> change is needed, right?
>
> Yea, but it does mean we need a fix propagated down to older kernels.

Greg, please consider applying 15b812f1d0 to stable 2.6.31 as it fixes
an issue introduced on 2.6.31 and only fixed now for 2.6.32. This
should fix an issue when loading either sky2 or ath9k on some specific
platforms.

The details for the ath9k debugging are available on this bug:

http://bugzilla.kernel.org/show_bug.cgi?id=14402

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

mcgrof at gmail

Nov 10, 2009, 12:50 PM

Post #11 of 11 (41 views)
Permalink

Re: Reverting 5d423 fixes loading of ath9k on Acer Extensa 7630EZ [In reply to]

On Tue, Nov 10, 2009 at 12:30 PM, Luis R. Rodriguez <mcgrof[at]gmail.com> wrote:
> On Mon, Nov 9, 2009 at 7:49 AM, Luis R. Rodriguez <mcgrof[at]gmail.com> wrote:
>> On Sun, Nov 8, 2009 at 11:15 PM, Ingo Molnar <mingo[at]elte.hu> wrote:
>>>
>>> * Luis R. Rodriguez <mcgrof[at]gmail.com> wrote:
>>>
>>>> On Wed, Nov 4, 2009 at 11:14 PM, Ingo Molnar <mingo[at]elte.hu> wrote:
>>>> >
>>>> > * Linus Torvalds <torvalds[at]linux-foundation.org> wrote:
>>>> >
>>>> >> On Wed, 4 Nov 2009, Luis R. Rodriguez wrote:
>>>> >> >
>>>> >> > Can you please consider reviewing this issue and help determine if
>>>> >> > this indeed needs to be reverted for 2.6.32 and the next 2.6.31.y.
>>>> >> >
>>>> >> > I am curious if other devices would work by reverting this as well.
>>>> >> > [ ... ] For details please feel free to check:
>>>> >> >
>>>> >> > http://bugzilla.kernel.org/show_bug.cgi?id=14402
>>>> >>
>>>> >> That same commit was the cause for
>>>> >>
>>>> >> ?? ?? ?? http://bugzilla.kernel.org/show_bug.cgi?id=13940
>>>> >>
>>>> >> and we just increased the rounding to make it go away (see commit
>>>> >> 15b812f1). But that was a hack.
>>>> >>
>>>> >> And if that didn't help the ath9k case, then we should just revert
>>>> >> entirely.
>>>> >
>>>> > Agreed - below is the combo 15b812f1 + 5d423ccd revert. (Would be nice
>>>> > to get the boot log of the latest post-15b812f1 kernel that Yinghai
>>>> > asked for before we revert, in the hope of better understanding the
>>>> > problem.)
>>>>
>>>> Bernhard has confirmed the new patch fixes this issue.
>>>
>>> Good - so latest kernels should be fine (on Bernhard's box) and no
>>> change is needed, right?
>>
>> Yea, but it does mean we need a fix propagated down to older kernels.
>
> Greg, please consider applying 15b812f1d0 to stable 2.6.31 as it fixes
> an issue introduced on 2.6.31 and only fixed now for 2.6.32. This
> should fix an issue when loading either sky2 or ath9k on some specific
> platforms.
>
> The details for the ath9k debugging are available on this bug:
>
> http://bugzilla.kernel.org/show_bug.cgi?id=14402

Just noticed this did make it to 2.6.31.6, sorry for the noise and thanks.

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

Index | Next | Previous | View Threaded

Interested in having your list archived? Contact lists@gossamer-threads.com