Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756086AbYAFUDz (ORCPT ); Sun, 6 Jan 2008 15:03:55 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753081AbYAFUDp (ORCPT ); Sun, 6 Jan 2008 15:03:45 -0500 Received: from py-out-1112.google.com ([64.233.166.179]:41479 "EHLO py-out-1112.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752677AbYAFUDo (ORCPT ); Sun, 6 Jan 2008 15:03:44 -0500 DomainKey-Signature: a=rsa-sha1; c=nofws; d=googlemail.com; s=gamma; h=message-id:date:from:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=imqGe7hCJx5TmpmIbK2sfeje+fA2w5XHHouSetI+EgrDpRfS60ZfLBVmrQkZfKrIYB9U593I33IklbDKyGzqBp8aQPGHuqwNskhfgXdb3sARsoLbSweY2TlobobCAsYC9rVMA3hi+jV6OA4PpU0ldqQFNshNr+wcFvfivu6pXKQ= Message-ID: <64bb37e0801061203l503f29f0hd922a1347f8169ac@mail.gmail.com> Date: Sun, 6 Jan 2008 21:03:42 +0100 From: "Torsten Kaiser" To: "FUJITA Tomonori" Subject: Re: 2.6.24-rc6-mm1 Cc: akpm@linux-foundation.org, jarkao2@gmail.com, herbert@gondor.apana.org.au, linux-kernel@vger.kernel.org, neilb@suse.de, bfields@fieldses.org, netdev@vger.kernel.org, tom@opengridcomputing.com, fujita.tomonori@lab.ntt.co.jp In-Reply-To: <20080106223650E.tomof@acm.org> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <64bb37e0801060241o2c6d8172r73b69291fce76ce1@mail.gmail.com> <20080106202616Z.tomof@acm.org> <64bb37e0801060335k4afb3134u9c6fadb57d525dc5@mail.gmail.com> <20080106223650E.tomof@acm.org> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3011 Lines: 82 On Jan 6, 2008 2:33 PM, FUJITA Tomonori wrote: > On Sun, 6 Jan 2008 12:35:35 +0100 > "Torsten Kaiser" wrote: > > On Jan 6, 2008 12:23 PM, FUJITA Tomonori wrote: > > > On Sun, 6 Jan 2008 11:41:10 +0100 > > > "Torsten Kaiser" wrote: > > > > I will applie your patch and see if this hunk from > > > > find_next_zero_area() makes a difference: > > > > > > > > end = index + nr; > > > > - if (end > size) > > > > + if (end >= size) > > > > return -1; -> that might still have made a difference, but ... > > > > - for (i = index + 1; i < end; i++) { > > > > + for (i = index; i < end; i++) { ... as you say below, the test for the index position is only needed if index is modified after find_next_zero_bit(). > > > > if (test_bit(i, map)) { > > > > > > The patch should not make a difference for X86_64. > > > > Hmm... > > arch/x86/kernel/pci-gart_64.c: > > alloc_iommu() calls iommu_area_alloc() > > lib/iommu-helper.c: > > iommu_area_alloc() calls find_next_zero_area() > > -> so the above code should be called even on X86_64 > > Oops, I meant that the patch fixes the align allocation (non zero > align_mask case). X86_64 doesn't use the align allocation. > > > > And the change in the for loop means that 'index' will now be tested, > > but with the old code it was not. > > With the old code, 'index' is tested by find_next_zero_bit. > > With the new code and non zero align_mask case, 'index' is not tested > by find_next_zero_bit. So test_bit needs to start with 'index'. > > So If I understand the correctly, this patch should not make a > difference for x86_64 though I might miss something. You did not miss anything. After 18 packages my system crashed again. > > And double using something does fit with the errors I'm seeing... > > > > > Can you try the patch to revert my IOMMU changes? > > > > > > http://www.mail-archive.com/linux-scsi@vger.kernel.org/msg12694.html > > > > Testing for this bug is a little bit slow, as I'm compiling ~100 > > packages trying to trigger it. > > If my current testrun with the patch from > > http://www.mail-archive.com/linux-scsi@vger.kernel.org/msg12702.html > > crashes, I will revert the hole IOMMU changes with above patch and try again. > > Thanks for testing, OK, I'm still testing this, but after 95 completed packages I'm rather certain that reverting the IOMMU changes with this patch fixes my problem. I didn't have time to look more into this, so I can't offer any concrete ideas where the bug is. If you send more patches, I'm willing to test them, but it might take some more time during the next week. Thanks for looking into this. Torsten -- 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/