Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753420AbbBKUuA (ORCPT ); Wed, 11 Feb 2015 15:50:00 -0500 Received: from userp1040.oracle.com ([156.151.31.81]:16772 "EHLO userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752282AbbBKUt6 (ORCPT ); Wed, 11 Feb 2015 15:49:58 -0500 Date: Wed, 11 Feb 2015 15:48:41 -0500 From: Konrad Rzeszutek Wilk To: "Wang, Xiaoming" Cc: David Vrabel , "linux-mips@linux-mips.org" , "pebolle@tiscali.nl" , "Zhang, Dongxing" , "lauraa@codeaurora.org" , "d.kasatkin@samsung.com" , "heiko.carstens@de.ibm.com" , "linux-kernel@vger.kernel.org" , "ralf@linux-mips.org" , "chris@chris-wilson.co.uk" , "takahiro.akashi@linaro.org" , "linux@horizon.com" , "xen-devel@lists.xenproject.org" , "boris.ostrovsky@oracle.com" , "Liu, Chuansheng" , "akpm@linux-foundation.org" Subject: Re: [Xen-devel] [PATCH] modify the IO_TLB_SEGSIZE to io_tlb_segsize configurable as flexible requirement about SW-IOMMU. Message-ID: <20150211204841.GB30585@l.oracle.com> References: <1423177274-22118-1-git-send-email-xiaoming.wang@intel.com> <20150205193241.GC11646@x230.dumpdata.com> <54D9D363.5060904@citrix.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) X-Source-IP: aserv0022.oracle.com [141.146.126.234] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3688 Lines: 85 On Wed, Feb 11, 2015 at 08:38:29AM +0000, Wang, Xiaoming wrote: > Dear David > > > -----Original Message----- > > From: David Vrabel [mailto:david.vrabel@citrix.com] > > Sent: Tuesday, February 10, 2015 5:46 PM > > To: Wang, Xiaoming; Konrad Rzeszutek Wilk > > Cc: linux-mips@linux-mips.org; pebolle@tiscali.nl; Zhang, Dongxing; > > lauraa@codeaurora.org; d.kasatkin@samsung.com; > > heiko.carstens@de.ibm.com; linux-kernel@vger.kernel.org; ralf@linux- > > mips.org; chris@chris-wilson.co.uk; takahiro.akashi@linaro.org; > > david.vrabel@citrix.com; linux@horizon.com; xen- > > devel@lists.xenproject.org; boris.ostrovsky@oracle.com; Liu, Chuansheng; > > akpm@linux-foundation.org > > Subject: Re: [Xen-devel] [PATCH] modify the IO_TLB_SEGSIZE to > > io_tlb_segsize configurable as flexible requirement about SW-IOMMU. > > > > On 06/02/15 00:10, Wang, Xiaoming wrote: > > > > > > > > >> -----Original Message----- > > >> From: Konrad Rzeszutek Wilk [mailto:konrad.wilk@oracle.com] > > >> Sent: Friday, February 6, 2015 3:33 AM > > >> To: Wang, Xiaoming > > >> Cc: ralf@linux-mips.org; boris.ostrovsky@oracle.com; > > >> david.vrabel@citrix.com; linux-mips@linux-mips.org; linux- > > >> kernel@vger.kernel.org; xen-devel@lists.xenproject.org; akpm@linux- > > >> foundation.org; linux@horizon.com; lauraa@codeaurora.org; > > >> heiko.carstens@de.ibm.com; d.kasatkin@samsung.com; > > >> takahiro.akashi@linaro.org; chris@chris-wilson.co.uk; > > >> pebolle@tiscali.nl; Liu, Chuansheng; Zhang, Dongxing > > >> Subject: Re: [PATCH] modify the IO_TLB_SEGSIZE to io_tlb_segsize > > >> configurable as flexible requirement about SW-IOMMU. > > >> > > >> On Fri, Feb 06, 2015 at 07:01:14AM +0800, xiaomin1 wrote: > > >>> The maximum of SW-IOMMU is limited to 2^11*128 = 256K. > > >>> While in different platform and different requirements this seems > > improper. > > >>> So modify the IO_TLB_SEGSIZE to io_tlb_segsize as configurable is > > >>> make > > >> sense. > > >> > > >> More details please. What is the issue you are hitting? > > >> > > > Example: > > > If 1M bytes are requied. There has an error like. > > > > Instead of allowing the bouncing of such large buffers, could the gadget > > driver be modified to submit the buffers to the hardware in smaller chunks? > > > > David > > Our target is try to make IO_TLB_SEGSIZE configurable. > Neither 256 bytes or 1M bytes seems suitable value, I think. > It's better to use the tactics something like > kmem_cache_create in kmalloc function. > But SW-IOMMU seems more lighter. > So we choose variable rather than function. Would it be possible to understand why the gadget needs such large buffer? That is irrespective of the patchset you are proposing. In regards to the pathchset - I don't see anything fundamentally wrong with the patch. What I am afraid is that this fixes the symptoms instead of the underlaying problem. The problem I think is that with this large 1MB requests you risk of using the SWIOTLB bounce buffer which can result in poor performance. So eventually somebody will have to figure out why the performance is poor and have a hard time figuring what is wrong - as the symptoms have been removed. Hence looking at potentially using an scatter gather mechanism and chop up the requests in smaller sizes might be an better option. But I don't know? Perhaps you are more familiar with the gadget and could tell me why it needs an 1MB size request? > > Xiaoming. -- 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/