Messages in this thread | | | Date | Wed, 26 Sep 2012 15:02:57 +0200 | From | Borislav Petkov <> | Subject | Re: [PATCH] x86: remove the useless branch in c_start() |
| |
On Wed, Sep 26, 2012 at 11:43:52AM +0800, Michael Wang wrote: > On 09/19/2012 01:42 PM, Michael Wang wrote: > > Since 'cpu == -1' in cpumask_next() is legal, no need to handle '*pos == 0' > > specially. > > > > About the comments: > > /* just in case, cpu 0 is not the first */ > > A test with a cpumask in which cpu 0 is not the first has been done, and it > > works well. > > Could I get some comments on this patch?
Looks ok to me:
Acked-by: Borislav Petkov <borislav.petkov@amd.com>
Thanks.
-- Regards/Gruss, Boris.
Advanced Micro Devices GmbH Einsteinring 24, 85609 Dornach GM: Alberto Bozzo Reg: Dornach, Landkreis Muenchen HRB Nr. 43632 WEEE Registernr: 129 19551
| |