Hi,<br><br><div class="gmail_quote">On Mon, Jan 16, 2012 at 6:51 PM, Maxim Dounin <span dir="ltr"><<a href="mailto:mdounin@mdounin.ru">mdounin@mdounin.ru</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Hello!<br>
<br>
On Mon, Jan 16, 2012 at 01:52:09PM +0800, Joshua Zhu wrote:<br>
<br>
[...]<br>
<div class="im"><br>
> > diff -r e1296af53cc0 src/os/unix/ngx_process_cycle.c<br>
> > --- a/src/os/unix/ngx_process_cycle.c   Mon Dec 26 00:00:00 2011 +0400<br>
> > +++ b/src/os/unix/ngx_process_cycle.c   Fri Jan 13 11:58:01 2012 -0500<br>
> > @@ -914,7 +914,10 @@<br>
> >         ngx_log_error(NGX_LOG_NOTICE, cycle->log, 0,<br>
> >                       "sched_setaffinity(0x%08Xl)", cpu_affinity);<br>
> ><br>
> > -        if (sched_setaffinity(0, 32, (cpu_set_t *) &cpu_affinity) == -1) {<br>
> > +        if (sched_setaffinity(0, sizeof(cpu_affinity),<br>
> > +                              (cpu_set_t *) &cpu_affinity)<br>
> > +            == -1)<br>
> > +        {<br>
> >             ngx_log_error(NGX_LOG_ALERT, cycle->log, ngx_errno,<br>
> >                           "sched_setaffinity(0x%08Xl) failed",<br>
> > cpu_affinity);<br>
> >         }<br>
> ><br>
> > Could you please test if it works for you?<br>
> ><br>
><br>
> Yes, it works on one of my Linux box which has two cores. But I have not<br>
> tested it on more other systems yet.<br>
<br>
</div>Ok, I'll commit it then, as this looks like an obvious fix.<br>
<div class="im"><br>
[...]<br>
<br>
</div></blockquote><br></div>OK. This fix looks good to me now. Just commit this piece of code is enough.<br clear="all"><br>Regards,<br><br>-- <br>Joshua Zhu<br>Senior Software Engineer<br>Server Platforms Team at Taobao<br>