From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eliezer Tamir Subject: Re: [PATCH v4 net-next] net: poll/select low latency socket support Date: Fri, 28 Jun 2013 09:00:41 +0300 Message-ID: <51CD2689.2020305@linux.intel.com> References: <20130624072751.26134.78163.stgit@ladj378.jer.intel.com> <20130624072803.26134.41593.stgit@ladj378.jer.intel.com> <51CCD812.5090408@codyps.com> <51CCD906.6030902@codyps.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Cc: David Miller , linux-kernel@vger.kernel.org, netdev@vger.kernel.org, Jesse Brandeburg , Don Skidmore , e1000-devel@lists.sourceforge.net, Willem de Bruijn , Eric Dumazet , Ben Hutchings , Andi Kleen , HPA , Eilon Greenstien , Or Gerlitz , Amir Vadai , Alex Rosenbaum , Eliezer Tamir To: Cody P Schafer Return-path: In-Reply-To: <51CCD906.6030902@codyps.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On 28/06/2013 03:29, Cody P Schafer wrote: > On 06/27/2013 05:25 PM, Cody P Schafer wrote: >> On 06/24/2013 12:28 AM, Eliezer Tamir wrote: >>> select/poll busy-poll support. >>> ... >> >> I'm seeing warnings about using smp_processor_id() while preemptable >> (log included below) due to this patch. I expect the use of >> ll_end_time() -> sched_clock() here is triggering this. >> >> Apologies if this has already been noted. > > To be clear, given how sched_time() is used here the BUG appears > invalid, but we need a way to mark this as OK for the smp_processor_id() > checks so we don't get BUG spam. > Thanks for reporting this. -Eliezer