![]() | |||||||||||||
Messages in this thread |
Jamie Lokier <jamie@shareable.org> wrote: > > Andrew Morton wrote: > > > Doesn't refusing to boot seem to heavy handed for this bug? The buggy > > > CPUs have been around for many years (it is practically the entire AMD > > > line for the last 4 years or so), and nobody in userspace has > > > complained about the 2.4 behaviour so far. (Linux 2.4 behaviour is, > > > of course, to ignore the errata). > > > > That is the case at present. But the 2.6 kernel was hitting this > > erracularity daily. > > We're talking about what to offer userspace now... I think we all > agree that the kernel itself shouldn't be allowed to hit it, one way > or another. Oh yes, if it hits in-kernel you get a dead box. Looking at Andi's patch, it is also a dead box if the fault happens inside down_write(mmap_sem). That should be fixed, methinks. And I think we're also a bit deadlocky if it happens inside down_read(), because double down_read() is illegal because an intervening down_write() from another thread can block the second down_read(). Or maybe not: the rwsem semantics may have changed since I last looked. And if the fault happens inside spinlock on a !CONFIG_PREEMPT kernel we end up doing down_read() with spinlocks held, I think? > > If some smart cookie decides to add prefetches to some STL implementation > > or something, they are likely to start hitting it with the same frequency. > > Especially now that GCC has intrinsics for prefetches, and GCC's > optimiser can generate prefetches automatically (-fprefetch-loop-arrays). Yup. Although prefetch-loop-arrays doesn't sound like something which will deref a dud pointer? > ... > I understand you're advocating a policy that says we can't do anything > about old systems, but from 2.6 onwards apps can depend on not being > hit by that erratum in userspace, is that right? I expect this will be backported to 2.4 asap. - 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/ | ||||||||||||
| Last update: 2005-03-22 13:49 [from the cache] ©2003-2008 | |||||||||||||