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: 2837946 Return-Path: X-Original-To: patchwork-dri-devel@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork2.web.kernel.org (Postfix) with ESMTP id 18F23BF535 for ; Fri, 2 Aug 2013 14:17:00 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id BB3AB20251 for ; Fri, 2 Aug 2013 14:16:58 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) by mail.kernel.org (Postfix) with ESMTP id 71FC720171 for ; Fri, 2 Aug 2013 14:16:57 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 1ABC5E81AB for ; Fri, 2 Aug 2013 07:16:57 -0700 (PDT) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from mail-la0-f54.google.com (mail-la0-f54.google.com [209.85.215.54]) by gabe.freedesktop.org (Postfix) with ESMTP id F36BCE65D6 for ; Fri, 2 Aug 2013 03:11:05 -0700 (PDT) Received: by mail-la0-f54.google.com with SMTP id ea20so309924lab.27 for ; Fri, 02 Aug 2013 03:11:04 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc:content-type:x-gm-message-state; bh=TFMVCSnUKupc3TU0yLRlPGVkU4tr4sVluYpt+ix55hE=; b=P4Sxv7IoQa/RpQc29G3jznnntaiNqHBvhx3GBMN9VewJWo/akL2ye/DYUpioQb4vbz UWa+puat5Dfks7LY8DmNW6/wdSw/2GZ/UFeI85O7D0QM8RucCVI/0/m6jZmIdLcfUQA0 fgMsUPhtw1GFVKd+jGxqgqNoVsx/oSkq7vV1VeI8Xh1aBkS/Q86h+JMsBigwVu8IEn9r iqmlb+oYUFlo3ZYSZyy/w4WU6NHL+gu9anxX2+VO/3G8al45RoRxhGjKabLef3mwM4gh y/WbqkAiFG6ooUTMXFg8NHvXAwjuWLZPUA0ERsHiYeLNkJ3xRi4uNW3vLBNkjVTYHPQN J3OA== 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 X-Gm-Message-State: ALoCoQnmNnrsrN12484hTUoUQNgQnP2Slge7xSUK4WcH+5Ug02iGhQ7EClrop9ZJa3LK3JkDtgCC X-Mailman-Approved-At: Fri, 02 Aug 2013 07:11:22 -0700 Cc: "kgene.kim" , Patch Tracking , DRI mailing list , sunil joshi , "linux-samsung-soc@vger.kernel.org" , Sylwester Nawrocki , m.szyprowski@samsung.com, linaro-kernel@lists.linaro.org, "arun.kk" , linux-media@vger.kernel.org X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.13 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces+patchwork-dri-devel=patchwork.kernel.org@lists.freedesktop.org Errors-To: dri-devel-bounces+patchwork-dri-devel=patchwork.kernel.org@lists.freedesktop.org X-Spam-Status: No, score=-5.7 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_MED, RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP 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,