Summary: | amd64 gentoo-dev-sources fails during compile | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | rj rooke <rjrooke> |
Component: | [OLD] Core system | Assignee: | x86-kernel (DEPRECATED) <x86-kernel> |
Status: | RESOLVED UPSTREAM | ||
Severity: | normal | CC: | amd64, himbeere, ianh, johnm, willie, x86-kernel |
Priority: | High | ||
Version: | unspecified | ||
Hardware: | AMD64 | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: | Patch fixing iommu_aperture_* compilation problem |
Description
rj rooke
2004-04-08 21:59:06 UTC
did make clean and recompiled - here are some possbily related errors from earlier in compile: CC arch/x86_64/kernel/io_apic.o arch/x86_64/kernel/io_apic.c: In function `check_ioapic': arch/x86_64/kernel/io_apic.c:272: warning: ISO C90 forbids mixed declarations and code CC arch/x86_64/mm/ioremap.o arch/x86_64/mm/ioremap.c: In function `__ioremap': arch/x86_64/mm/ioremap.c:141: warning: ISO C90 forbids mixed declarations and code CC arch/x86_64/mm/extable.o disable pre-empt and see if that helps. this is where i mention that pre-empt is a bad idea on amd64, yadda yadda, unstable this that the other, headaches in #gentoo-amd64... maybe some other stuff. just say no to pre-empt. etc. Created attachment 29023 [details, diff]
Patch fixing iommu_aperture_* compilation problem
The variables iommu_aperture_allowed and iommu_aperture_disabled are defined in
aperture.c. This is dependent on CONFIG_GART_IOMMU. Therefore the references
to these variables in io_apic.c should also conditional.
you know, it'd probably help if one of you mentioned a version number. *cough* only the patch mentions a version number... so the problem is with gentoo-dev-sources-2.6.5? I'll poke johnm with a stick and get him to look at this, he's "the dude". Agreed. The version is.... gentoo-dev-sources-2.6.5, as you said. I'll get used to the Gentoo nomenclature eventually! since there's already a freeze for the 2004.1 release, this fix wont make it into gentoo-dev-sources-2.6.5 (which is being used to make the livecd), but this isnt much of a problem as genkernel enables IOMMU by default. The temporary workaround is to enable IOMMU I guess, with any fixes for this appearing in gentoo-dev-sources 2.6.5-r1 (and not appearing on livecds, whose kernels will be made with genkernel anyways). I'll poke johnm with a stick again later just in case he forgets. ;) oh yeah. as you've probably already noticed, the IOMMU disables itself if you dont have more than 4 gigs of memory.. so it doesn't really hurt to enable it, and it's required anyways if you have more than 4 gigs of ram. at least the workaround of enabling it wont be much of a pain (i leave it enabled, so i never even noticed this bug). Adding gart_iommu fixes the problem for me in gentoo-dev-sources 2.6.5 - preemptible kernel seems to be ok as well. First time entering a bug - do I do anything else, or does somebody else close it? thx since the bug is assigned to the amd64 herd, one of us will close the bug when the fix makes it into portage. keep filing bugs for anything you see and think we can fix, just try to provide more information next time. ;) and as a side-note, pre-empt is still seriously not recommended on amd64... even the help for pre-empt mentions that it isnt very well tested on x86-64, and I've dealt with a handfull of bugs myself that were directly attributable to pre-empt. *** Bug 47627 has been marked as a duplicate of this bug. *** *pokes johnm with a stick* it seems that the fix of enabling iommu for now isn't an acceptable one for everyone who experiences this bug. see bug #47627 I'm pretty sure this fix is already in the upstream package. Wait for the next -rc kernel to come out and see if it is fixed there. *** Bug 48253 has been marked as a duplicate of this bug. *** ping kernel guys - any update on this making it into a g-d-s 2.6.5-r2 since r1 doesnt seem to have it? Reassigning to x86-kernel herd... alrighty. well, looks like nothing is going to happen here, and we already have a 2.6.6 release. |