From patchwork Fri Aug 2 10:10:44 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Vikas C Sajjan X-Patchwork-Id: 18736 Return-Path: X-Original-To: linaro@patches.linaro.org Delivered-To: linaro@patches.linaro.org Received: from mail-qe0-f69.google.com (mail-qe0-f69.google.com [209.85.128.69]) by ip-10-151-82-157.ec2.internal (Postfix) with ESMTPS id 31F0D25E07 for ; Fri, 2 Aug 2013 10:11:07 +0000 (UTC) Received: by mail-qe0-f69.google.com with SMTP id 1sf564472qec.0 for ; Fri, 02 Aug 2013 03:11:07 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=x-beenthere:x-forwarded-to:x-forwarded-for:delivered-to :mime-version:in-reply-to:references:from:date:message-id:subject:to :cc:x-gm-message-state:x-removed-original-auth:x-original-sender :x-original-authentication-results:precedence:mailing-list:list-id :x-google-group-id:list-post:list-help:list-archive:list-unsubscribe :content-type; bh=TFMVCSnUKupc3TU0yLRlPGVkU4tr4sVluYpt+ix55hE=; b=TnQNdLYXoTz0LKGZVLKk8vJpAtS2yErksEEel3lEKmKhPeRUD9PRPRIcVQ+m1/kaXr tmmMOwwSqNG3UQB6+mMrnvQptDPO7CTEBEUZGkz80EvbOcgdk6YCfxBYfekqS8iKoEJf 9+C2NemvWwY4TAbMncklcbecrJwqnI8RxhHhEjljLtyEGNChtc6DU/ui1O5G25/3318I nXALZXs3yM73ChRoBa2FYBa1fqlGA9VcBq9oxh9Ub53zZnzBpOzEQuGZB6vbU4Y9Qj6u tEWCGW6ZT2dSpHzSA2pLFYZ96dno/M6LbXoxj3X2rXDhEy7y16ORvA0mkUY0WWdax3LL 93zw== X-Received: by 10.236.46.105 with SMTP id q69mr2485592yhb.22.1375438266737; Fri, 02 Aug 2013 03:11:06 -0700 (PDT) X-BeenThere: patchwork-forward@linaro.org Received: by 10.49.82.101 with SMTP id h5ls1543478qey.91.gmail; Fri, 02 Aug 2013 03:11:06 -0700 (PDT) X-Received: by 10.220.249.7 with SMTP id mi7mr1791565vcb.72.1375438266610; Fri, 02 Aug 2013 03:11:06 -0700 (PDT) Received: from mail-ve0-f181.google.com (mail-ve0-f181.google.com [209.85.128.181]) by mx.google.com with ESMTPS id zx19si1688614veb.139.2013.08.02.03.11.06 for (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Fri, 02 Aug 2013 03:11:06 -0700 (PDT) Received-SPF: neutral (google.com: 209.85.128.181 is neither permitted nor denied by best guess record for domain of patch+caf_=patchwork-forward=linaro.org@linaro.org) client-ip=209.85.128.181; Received: by mail-ve0-f181.google.com with SMTP id jz10so437820veb.12 for ; Fri, 02 Aug 2013 03:11:06 -0700 (PDT) X-Received: by 10.52.107.201 with SMTP id he9mr1430415vdb.106.1375438266439; Fri, 02 Aug 2013 03:11:06 -0700 (PDT) X-Forwarded-To: patchwork-forward@linaro.org X-Forwarded-For: patch@linaro.org patchwork-forward@linaro.org Delivered-To: patches@linaro.org Received: by 10.221.11.8 with SMTP id pc8csp348169vcb; Fri, 2 Aug 2013 03:11:05 -0700 (PDT) X-Received: by 10.152.6.169 with SMTP id c9mr2684467laa.47.1375438264952; Fri, 02 Aug 2013 03:11:04 -0700 (PDT) Received: from mail-lb0-f182.google.com (mail-lb0-f182.google.com [209.85.217.182]) by mx.google.com with ESMTPS id r4si3450056lah.23.2013.08.02.03.11.04 for (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Fri, 02 Aug 2013 03:11:04 -0700 (PDT) Received-SPF: neutral (google.com: 209.85.217.182 is neither permitted nor denied by best guess record for domain of vikas.sajjan@linaro.org) client-ip=209.85.217.182; Received: by mail-lb0-f182.google.com with SMTP id v20so317065lbc.27 for ; Fri, 02 Aug 2013 03:11:04 -0700 (PDT) X-Received: by 10.152.28.37 with SMTP id y5mr2705289lag.0.1375438264230; Fri, 02 Aug 2013 03:11:04 -0700 (PDT) MIME-Version: 1.0 Received: by 10.112.21.33 with HTTP; Fri, 2 Aug 2013 03:10:44 -0700 (PDT) In-Reply-To: References: <1375355972-25276-1-git-send-email-vikas.sajjan@linaro.org> <5151790.EBRlE0cTxf@flatron> From: Vikas Sajjan Date: Fri, 2 Aug 2013 15:40:44 +0530 Message-ID: Subject: Re: [PATCH] drm/exynos: Add check for IOMMU while passing physically continous memory flag To: Inki Dae Cc: Rob Clark , Tomasz Figa , "linux-samsung-soc@vger.kernel.org" , DRI mailing list , linux-media@vger.kernel.org, "kgene.kim" , "arun.kk" , Patch Tracking , linaro-kernel@lists.linaro.org, sunil joshi , Sylwester Nawrocki , m.szyprowski@samsung.com X-Gm-Message-State: ALoCoQn6FJue9vImK2GJWyO9251ZNBYPiMaB4LlraDrs2gTHCUf9dnnSUVArfl5m6c3ZEmIPBgjM X-Removed-Original-Auth: Dkim didn't pass. X-Original-Sender: vikas.sajjan@linaro.org X-Original-Authentication-Results: mx.google.com; spf=neutral (google.com: 209.85.128.181 is neither permitted nor denied by best guess record for domain of patch+caf_=patchwork-forward=linaro.org@linaro.org) smtp.mail=patch+caf_=patchwork-forward=linaro.org@linaro.org Precedence: list Mailing-list: list patchwork-forward@linaro.org; contact patchwork-forward+owners@linaro.org List-ID: X-Google-Group-Id: 836684582541 List-Post: , List-Help: , List-Archive: List-Unsubscribe: , Hi Inki Dae, On 2 August 2013 12:58, Inki Dae wrote: > > > 2013/8/2 Vikas Sajjan >> >> Hi Rob, >> >> On 2 August 2013 06:03, Rob Clark wrote: >> > On Thu, Aug 1, 2013 at 7:20 PM, Tomasz Figa >> > wrote: >> >> Hi Vikas, >> >> >> >> On Thursday 01 of August 2013 16:49:32 Vikas Sajjan wrote: >> >>> While trying to get boot-logo up on exynos5420 SMDK which has eDP >> >>> panel >> >>> connected with resolution 2560x1600, following error occured even with >> >>> IOMMU enabled: >> >>> [0.880000] [drm:lowlevel_buffer_allocate] *ERROR* failed to allocate >> >>> buffer. [0.890000] [drm] Initialized exynos 1.0.0 20110530 on minor 0 >> >>> >> >>> This patch fixes the issue by adding a check for IOMMU. >> >>> >> >>> Signed-off-by: Vikas Sajjan >> >>> Signed-off-by: Arun Kumar >> >>> --- >> >>> drivers/gpu/drm/exynos/exynos_drm_fbdev.c | 9 ++++++++- >> >>> 1 file changed, 8 insertions(+), 1 deletion(-) >> >>> >> >>> diff --git a/drivers/gpu/drm/exynos/exynos_drm_fbdev.c >> >>> b/drivers/gpu/drm/exynos/exynos_drm_fbdev.c index 8e60bd6..2a86666 >> >>> 100644 >> >>> --- a/drivers/gpu/drm/exynos/exynos_drm_fbdev.c >> >>> +++ b/drivers/gpu/drm/exynos/exynos_drm_fbdev.c >> >>> @@ -16,6 +16,7 @@ >> >>> #include >> >>> #include >> >>> #include >> >>> +#include >> >>> >> >>> #include "exynos_drm_drv.h" >> >>> #include "exynos_drm_fb.h" >> >>> @@ -143,6 +144,7 @@ static int exynos_drm_fbdev_create(struct >> >>> drm_fb_helper *helper, struct platform_device *pdev = >> >>> dev->platformdev; >> >>> unsigned long size; >> >>> int ret; >> >>> + unsigned int flag; >> >>> >> >>> DRM_DEBUG_KMS("surface width(%d), height(%d) and bpp(%d\n", >> >>> sizes->surface_width, sizes->surface_height, >> >>> @@ -166,7 +168,12 @@ static int exynos_drm_fbdev_create(struct >> >>> drm_fb_helper *helper, size = mode_cmd.pitches[0] * mode_cmd.height; >> >>> >> >>> /* 0 means to allocate physically continuous memory */ >> >>> - exynos_gem_obj = exynos_drm_gem_create(dev, 0, size); >> >>> + if (!is_drm_iommu_supported(dev)) >> >>> + flag = 0; >> >>> + else >> >>> + flag = EXYNOS_BO_NONCONTIG; >> >> >> >> While noncontig memory might be used for devices that support IOMMU, >> >> there >> >> should be no problem with using contig memory for them, so this seems >> >> more >> >> like masking the original problem rather than tracking it down. >> > >> > it is probably a good idea to not require contig memory when it is not >> > needed for performance or functionality (and if it is only >> > performance, then fallback gracefully to non-contig).. but yeah, would >> > be good to know if this is masking another issue all the same >> > >> >> Whats happening with CONTIG flag and with IOMMU, is >> >> __iommu_alloc_buffer() ---> dma_alloc_from_contiguous() and in this >> function it fails at >> this condition check if (pageno >= cma->count) >> >> So I tried increasing the CONFIG_CMA_SIZE_MBYTES to 24, this check >> succeeds and it works well without my patch. >> >> But what about the case where CONFIG_CMA is disabled , yet i want >> bigger memory for a device. >> I think using IOMMU we can achieve this. >> >> correct me, if i am wrong. >> > > I'm on summer vacation so I'm afraid that I cannot test and look into it but > I guess you guy didn't declare CMA region for Exynos drm. And in this case, > the size of CMA declared region is 16MB as default. That is why works well > after increasing default size, CONFIG_CMA_SIZE_MBYTES, to 24MB. And I > mentioned long time ago, we are required to use physically contiguous memory > in case that bootloader uses physically contiguous memory for its own > framebuffer, and kernel wants to share the bootloader's framebuffer region > to resolve flickering issue while booted; that is required for product. And > one question, is there any reason that you guy should use non-contiguous > memory for framebuffer with iommu? > yeah, we could not allocate CMA region for FIMD, because the function dma_declare_contiguous() needs "dev" as the first argument and we have access to "dev" node only if it is NON-DT way of probing like the way it is done in arch/arm/mach-davinci/devices-da8xx.c But now, since the probing is through DT way, there is NO way ( Let me know if something is newly added ) to call dma_declare_contiguous() and reserve CMA region . we don't have any specific requirement for NON_CONTIG or CONTIG memory, but only requirement was to allocate a bigger memory like ( 2560 * 1600 * 4 ) for FB. But as Rob suggested, we should have fall-back case if CONTIG memory allocation fails, as below > Inki Dae > >> >> >> > BR, >> > -R >> > >> >> Could you check why the allocation fails when requesting contiguous >> >> memory? >> >> >> >> Best regards, >> >> Tomasz >> >> >> >> -- >> >> To unsubscribe from this list: send the line "unsubscribe linux-media" >> >> in >> >> the body of a message to majordomo@vger.kernel.org >> >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> >> >> >> -- >> Thanks and Regards >> Vikas Sajjan >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-media" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html > > diff --git a/drivers/gpu/drm/exynos/exynos_drm_fbdev.c b/drivers/gpu/drm/exynos/exynos_drm_fbdev.c index df43fa9..15de626 100644 --- a/drivers/gpu/drm/exynos/exynos_drm_fbdev.c +++ b/drivers/gpu/drm/exynos/exynos_drm_fbdev.c @@ -144,7 +144,6 @@ static int exynos_drm_fbdev_create(struct drm_fb_helper *helper, struct platform_device *pdev = dev->platformdev; unsigned long size; int ret; DRM_DEBUG_KMS("surface width(%d), height(%d) and bpp(%d\n", sizes->surface_width, sizes->surface_height, @@ -167,16 +166,14 @@ static int exynos_drm_fbdev_create(struct drm_fb_helper *helper, size = mode_cmd.pitches[0] * mode_cmd.height; - /* 0 means to allocate physically continuous memory */ - exynos_gem_obj = exynos_drm_gem_create(dev, 0, size); + exynos_gem_obj = exynos_drm_gem_create(dev, EXYNOS_BO_CONTIG, size); if (IS_ERR(exynos_gem_obj)) { - ret = PTR_ERR(exynos_gem_obj); - goto err_release_framebuffer; + if(is_drm_iommu_supported(dev)) + exynos_gem_obj = exynos_drm_gem_create(dev, EXYNOS_BO_NONCONTIG, size); + if (IS_ERR(exynos_gem_obj)) { + ret = PTR_ERR(exynos_gem_obj); + goto err_release_framebuffer; + } + dev_warn("\n exynos_gem_obj for FB is allocated with non physically continuous + memory \n"); } > Thanks,