Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751253Ab0DLMLR (ORCPT ); Mon, 12 Apr 2010 08:11:17 -0400 Received: from mail-ew0-f220.google.com ([209.85.219.220]:55066 "EHLO mail-ew0-f220.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750893Ab0DLMLQ (ORCPT ); Mon, 12 Apr 2010 08:11:16 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; b=BAagHEGYFxoFuzDenChCyv6SSeYBaHN8FLOq9GVsz7q12DSQ8z/80J61bQDuv/uhaC b2xYsxk/kAuUJaiDu+qUQtzcRkSKheMuVdTT16MRDPbdO03zQyclgWcVjOuYAR5s847D gBIps4zIzjokVR60AvBKTmhPboKDV6IQPBLLM= MIME-Version: 1.0 In-Reply-To: <20100412115300.GM18855@one.firstfloor.org> References: <20100407090623.GN30807@buzzloop.caiaq.de> <87sk719hbm.fsf@basil.nowhere.org> <20100412111439.GU30801@buzzloop.caiaq.de> <20100412115300.GM18855@one.firstfloor.org> Date: Mon, 12 Apr 2010 13:11:13 +0100 Message-ID: Subject: Re: USB transfer_buffer allocations on 64bit systems From: Pedro Ribeiro To: Andi Kleen Cc: Daniel Mack , Alan Stern , linux-kernel@vger.kernel.org, akpm@linux-foundation.org, Greg KH , alsa-devel@alsa-project.org, linux-usb@vger.kernel.org Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1693 Lines: 41 On 12 April 2010 12:53, Andi Kleen wrote: >> Which is exactly what usb_buffer_alloc() does already. So at least for >> x86 you say this is the right thing to do? However, we don't necessarily >> need coherent memory on other platforms, which is why I hessitate to >> enforce that type of memory for all transfer_buffer allocations. > > Yes today it's faster at least. > > Probably we should have better interfaces for this, but we don't. > >> >> > Or just use GFP_KERNEL and pci_map_* later. >> >> The USB core does this already, but at least on Pedro's machine, this >> seems unsufficient. Unfortunately, I can't reproduce the issue yet, even >> with more than 4GB of RAM installed. > > Then something must be broken in Pedro's system and likely other drivers > will also not work. I don't think it should be worked around in the USB layer. > > -Andi I'm not putting into question whether something is broken in my system, but if it is, it must be the ICH9 platform, because I was able to reproduce it in another laptop. My laptop is a Lenovo T400 and I was able to reproduce it in a Acer Aspire 59xx (I don't remember the exact model, but it is one of the new ones with 15.6 inch - i think they all use the same base). And the common thing between them is the ICH9 platform. The only which solved this problem was the first patch sent to me by Daniel Mack. I've been using it for days straight and it works fine. Pedro -- 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/