Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756215Ab0FIIyK (ORCPT ); Wed, 9 Jun 2010 04:54:10 -0400 Received: from ogre.sisk.pl ([217.79.144.158]:41160 "EHLO ogre.sisk.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753197Ab0FIIyF (ORCPT ); Wed, 9 Jun 2010 04:54:05 -0400 From: "Rafael J. Wysocki" To: Jens Axboe Subject: Re: 2.6.35-rc2-git2: Reported regressions from 2.6.34 Date: Wed, 9 Jun 2010 10:55:38 +0200 User-Agent: KMail/1.12.4 (Linux/2.6.35-rc1-rjw; KDE/4.3.5; x86_64; ; ) Cc: Linus Torvalds , Carl Worth , Eric Anholt , Venkatesh Pallipadi , Dave Airlie , Jesse Barnes , Ingo Molnar , David =?iso-8859-15?q?H=E4rdeman?= , Mauro Carvalho Chehab , Eric Dumazet , Linux Kernel Mailing List , Maciej Rutecki , Andrew Morton , Kernel Testers List , Network Development , Linux ACPI , Linux PM List , Linux SCSI List , Linux Wireless List , DRI References: <4C0F4872.7090909@fusionio.com> In-Reply-To: <4C0F4872.7090909@fusionio.com> MIME-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-15" Content-Transfer-Encoding: 7bit Message-Id: <201006091055.38462.rjw@sisk.pl> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2670 Lines: 63 On Wednesday 09 June 2010, Jens Axboe wrote: > On 2010-06-09 03:53, Linus Torvalds wrote: > >> Bug-Entry : http://bugzilla.kernel.org/show_bug.cgi?id=16129 > >> Subject : BUG: using smp_processor_id() in preemptible [00000000] code: jbd2/sda2 > >> Submitter : Jan Kreuzer > >> Date : 2010-06-05 06:15 (4 days old) > > > > This seems to have been introduced by > > > > commit 7cbaef9c83e58bbd4bdd534b09052b6c5ec457d5 > > Author: Ingo Molnar > > Date: Sat Nov 8 17:05:38 2008 +0100 > > > > sched: optimize sched_clock() a bit > > > > sched_clock() uses cycles_2_ns() needlessly - which is an irq-disabling > > variant of __cycles_2_ns(). > > > > Most of the time sched_clock() is called with irqs disabled already. > > The few places that call it with irqs enabled need to be updated. > > > > Signed-off-by: Ingo Molnar > > > > and this seems to be one of those calling cases that need to be updated. > > > > Ingo? The call trace is: > > > > BUG: using smp_processor_id() in preemptible [00000000] code: jbd2/sda2-8/337 > > caller is native_sched_clock+0x3c/0x68 > > Pid: 337, comm: jbd2/sda2-8 Not tainted 2.6.35-rc1jan+ #4 > > Call Trace: > > [] debug_smp_processor_id+0xc9/0xe4 > > [] native_sched_clock+0x3c/0x68 > > [] sched_clock+0x9/0xd > > [] blk_rq_init+0x97/0xa3 > > [] get_request+0x1c4/0x2d0 > > [] get_request_wait+0x29/0x1a6 > > [] __make_request+0x338/0x45b > > [] generic_make_request+0x2bb/0x330 > > [] submit_bio+0xd2/0xef > > [] submit_bh+0xf4/0x116 > > [] block_write_full_page_endio+0x89/0x96 > > [] block_write_full_page+0x15/0x17 > > [] ext4_writepage+0x356/0x36b > > [] __writepage+0x1a/0x39 > > [] write_cache_pages+0x20d/0x346 > > [] generic_writepages+0x27/0x29 > > [] journal_submit_data_buffers+0x110/0x17d > > [] jbd2_journal_commit_transaction+0x4cb/0x156d > > [] kjournald2+0x147/0x37a > > > > (from the bugzilla thing) > > This should be fixed by commit 28f4197e which was merged on friday. Thanks, closed. Rafael -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/