Re: [PATCH] percpu: fix DEBUG_PREEMPT per_cpu checking

[Andrew Morton]

On Sat, 23 Feb 2008 19:40:17 +0000 (GMT) Hugh Dickins <hugh@xxxxxxxxxxx> wrote:

> 2.6.25-rc1 percpu changes broke CONFIG_DEBUG_PREEMPT's per_cpu checking
> on several architectures.  On s390, sparc64 and x86 it's been weakened to
> not checking at all; whereas on powerpc64 it's become too strict, issuing
> warnings from __raw_get_cpu_var in io_schedule and init_timer for example.
> 
> Fix this by weakening powerpc's __my_cpu_offset to use the non-checking
> local_paca instead of get_paca (which itself contains such a check);
> and strengthening the generic my_cpu_offset to go the old slow way via
> smp_processor_id when CONFIG_DEBUG_PREEMPT (debug_smp_processor_id is
> where all the knowledge of what's correct when lives).
> 
> Signed-off-by: Hugh Dickins <hugh@xxxxxxxxxxx>
> Reviewed-by: Mike Travis <travis@xxxxxxx>
> ---
> I posted this a couple of weeks ago, but it seems we all expected
> someone else to forward it to you.

I merged it, but for non-critical fixes I'll tend to sit on them for a week
or so before sending them in to Linus.

This patch has now turned up in git-sched which is a considerable
inconvenience.

Ingo, please don't just merge everything which crosses your desk.  I'm now
in the position that I either drop the patch and completely lose track of
it or I need to apply then revert it to make your tree apply.

Hopefully Linus will apply the patch from Hugh's email, but that isn't the
general solution to this increasingly worse problem.

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