MIME-Version: 1.0
In-Reply-To: <1329285786.2555.38.camel@edumazet-laptop>
References: <1329280586-6878-1-git-send-email-tom.leiming@gmail.com>
	<1329281947.2555.26.camel@edumazet-laptop>
	<CACVXFVN7ye6QkJLWSTecy24FeHY6SRB6cDLAXyrPF7kBKgvq5g@mail.gmail.com>
	<1329285786.2555.38.camel@edumazet-laptop>
Date: Wed, 15 Feb 2012 16:56:17 +0800
Message-ID: <CACVXFVOyJa8K4rqNatYdQqiPd9nS+6fU0e2A6mU4urejfS9wuQ@mail.gmail.com>
Subject: Re: [PATCH v1] percpu: use raw_local_irq_* in _this_cpu op
From: Ming Lei <tom.leiming@gmail.com>
To: Eric Dumazet <eric.dumazet@gmail.com>
Cc: Christoph Lameter <cl@linux.com>, Tejun Heo <tj@kernel.org>,
        Peter Zijlstra <a.p.zijlstra@chello.nl>, linux-kernel@vger.kernel.org
Content-Type: text/plain; charset=ISO-8859-1
Sender: linux-kernel-owner@vger.kernel.org
Content-Transfer-Encoding: 8bit
Content-Length: 2225
Lines: 55

On Wed, Feb 15, 2012 at 2:03 PM, Eric Dumazet <eric.dumazet@gmail.com> wrote:
> Le mercredi 15 f?vrier 2012 ? 13:08 +0800, Ming Lei a ?crit :
>
>> Just several warnings, no errors. Also the 5 warnings are introduced
>> by previous commit, sounds nothing to do with motivation of this patch.
>> You may prepare a standalone patch to fix all the code style in this file, :-)
>>
>
> Oh well, I dont use checkpatch but spoke about trailing \
>
> Cant you see them on your screen ?
>
> We had :
>
> #define _this_cpu_generic_to_op(pcp, val, op) ? ? ? ? ? ? ? ? ? ? ? ? ? \
> do { ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?\
> ? ? ? ?unsigned long flags; ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?\
> ? ? ? ?local_irq_save(flags); ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?\
> ? ? ? ?*__this_cpu_ptr(&(pcp)) op val; ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? \
> ? ? ? ?local_irq_restore(flags); ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? \
> } while (0)
>
>
> You want :
>
> #define _this_cpu_generic_to_op(pcp, val, op) ? ? ? ? ? ? ? ? ? ? ? ? ? \
> do { ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?\
> ? ? ? ?unsigned long flags; ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?\
> ? ? ? ?raw_local_irq_save(flags); ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?\
> ? ? ? ?*__this_cpu_ptr(&(pcp)) op val; ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? \
> ? ? ? ?raw_local_irq_restore(flags); ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? \
> } while (0)
>
>
> I suggest :
>
> #define _this_cpu_generic_to_op(pcp, val, op) ? ? ? ? ? ? ? ? ? ? ? ? ? \
> do { ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?\
> ? ? ? ?unsigned long flags; ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?\
> ? ? ? ?raw_local_irq_save(flags); ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?\
> ? ? ? ?*__this_cpu_ptr(&(pcp)) op val; ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? \
> ? ? ? ?raw_local_irq_restore(flags); ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? \
> } while (0)
>
>
> Is it clear now ?

Thanks for your pointing it out.
Yes, I have updated the patch and sent it out.


thanks,
-- 
Ming Lei
????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m????????????I?