From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751790AbcFWQHQ (ORCPT ); Thu, 23 Jun 2016 12:07:16 -0400 Received: from aserp1040.oracle.com ([141.146.126.69]:23603 "EHLO aserp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750926AbcFWQHF convert rfc822-to-8bit (ORCPT ); Thu, 23 Jun 2016 12:07:05 -0400 User-Agent: K-9 Mail for Android In-Reply-To: <20160623143034.GN6521@e104818-lin.cambridge.arm.com> References: <1466684020-5224-1-git-send-email-manjeet.p@samsung.com> <20160623143034.GN6521@e104818-lin.cambridge.arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Subject: Re: [PATCH] arm64:swiotlb:Enable only when Input size through command line From: Konrad Rzeszutek Wilk Date: Thu, 23 Jun 2016 12:06:10 -0400 To: Catalin Marinas , Manjeet Pawar CC: corbet@lwn.net, will.deacon@arm.com, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, r.thapliyal@samsung.com, akhilesh.k@samsung.com, ajeet.y@samsung.com, pankaj.m@samsung.com Message-ID: <9A15F5E5-775E-4079-961F-67FD54B8F2F4@oracle.com> X-Source-IP: userv0021.oracle.com [156.151.31.71] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On June 23, 2016 10:30:34 AM EDT, Catalin Marinas wrote: >On Thu, Jun 23, 2016 at 05:43:40PM +0530, Manjeet Pawar wrote: >> From: Rohit Thapliyal >> >> swiotlb default size of 64M is too big as >> default value therefore it is made configurable >> through command line through swiotlb_size parameter. >> swiotlb allocation shall be done only when the >> swiotlb size is given through command line. >> Otherwise no swiotlb is allocated. > >I already queued this patch: > >http://lkml.kernel.org/g/1465372426-4077-1-git-send-email-jszhang@marvell.com > >If you have any objections to it, please reply there. I do (sorry about duplicate email, the other got rejected by mailing lists). Why not expand the swiotlb= parameter instead of introducing a new one? Also, why not use the swiotlb by itself? That does the job as well?