Messages in this thread | | | Date | Mon, 29 Oct 2007 15:45:09 -0700 | From | Hiroshi Shimamoto <> | Subject | Re: [PATCH 1/3] x86: add lapic_shutdown for x86_64 |
| |
Eric W. Biederman wrote: > Hiroshi Shimamoto <h-shimamoto@ct.jp.nec.com> writes: > >>> Do we really have to introduce this function for 64bit? I remember some >>> issues were faced on i386 w.r.t kernel enabling the LAPIC against the >>> wishes of BIOS hence kernel was disabling it while shutting down. No >>> such problems were reported for x86_64 hence this function existed only >>> for i386. >> Thanks for the comment. I didn't know the issues, so I'd simply added >> this function for unification. >> >>> If that is the case, probably we don't have to introduce lapic_shutdown() >>> for x86_64. Instead call lapic_shutdown() for X86_32, and disble_local_APIC() >>> otherwise? >> I will do that. I was thinking which is good when posting these patches. > > I'm a little concerned here. This sounds like forced unification. > If we can't clean up the infrastructure so things are obviously better > and cleanly factored for both architectures we should not unify the files. > > As a general principle I would rather have two crudy files side by > side the one super crudy file. > > So for unification I suggest finally fixing this right and taking the > apics completely out of the kexec on panic path.
Thanks for the suggestion. But it's hard for me to imagine. I'll try to consider about it.
Thanks Hiroshi Shimamoto - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |