Messages in this thread | | | Date | Thu, 27 Sep 2012 10:29:32 +0800 | From | Michael Wang <> | Subject | Re: [PATCH] x86: remove the useless branch in c_start() |
| |
On 09/26/2012 09:02 PM, Borislav Petkov wrote: > 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.
Thanks for your review.
Regards, Michael Wang >
| |