From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Bristot de Oliveira Subject: Re: [RFC PATCH 0/2] x86: Fix missing core serialization on migration Date: Tue, 14 Nov 2017 19:24:22 +0100 Message-ID: <95f902ea-3e97-0280-2d52-28cadef7806d@redhat.com> References: <4d47fbb8-8f99-19d3-a9cf-66841aeffac3@scylladb.com> <4431530.14831.1510672632887.JavaMail.zimbra@efficios.com> <20171114160541.GC3165@worktop.lehotels.local> <20171114163159.GD3165@worktop.lehotels.local> <6f9f57fa-8057-cdbe-231b-20920b3b3670@redhat.com> <20171114174049.GF3165@worktop.lehotels.local> <20171114181732.bwahj6woik27trou@hirez.programming.kicks-ass.net> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20171114181732.bwahj6woik27trou-Nxj+rRp3nVydTX5a5knrm8zTDFooKrT+cvkQGrU6aU0@public.gmane.org> Content-Language: en-US Sender: linux-api-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Peter Zijlstra Cc: Andy Lutomirski , Thomas Gleixner , Mathieu Desnoyers , Avi Kivity , Linus Torvalds , linux-kernel , linux-api , "Paul E. McKenney" , Boqun Feng , Andrew Hunter , maged michael , Benjamin Herrenschmidt , Paul Mackerras , Michael Ellerman , Dave Watson , Ingo Molnar , "H. Peter Anvin" , Andrea Parri , "Russell King, ARM Linux" List-Id: linux-api@vger.kernel.org On 11/14/2017 07:17 PM, Peter Zijlstra wrote: > On Tue, Nov 14, 2017 at 07:01:55PM +0100, Daniel Bristot de Oliveira wrote: >> On 11/14/2017 06:40 PM, Peter Zijlstra wrote: >>> On Tue, Nov 14, 2017 at 06:17:13PM +0100, Daniel Bristot de Oliveira wrote: >>> >>>> IIRC, if the dest cpu is idle and the system is with idle=poll, no IPI >>>> is fired as well, but that is not a very common case. >>> >>> You're thinking about wake from idle? That is almost always without IPI, >>> even without idle=poll. >>> >> >> I meant the resched_curr(rq) of an rq on another CPU. If the dest is >> idle && idle=poll, the IPI will not be send. > > I'm saying the IPI will not be send even without idle=poll. MWAIT based > idle will also have TIF_POLLING_NRFLAG set. > Yeah! you are right! I missed that point... sorry :-) -- Daniel