From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pa0-f49.google.com (mail-pa0-f49.google.com [209.85.220.49]) by kanga.kvack.org (Postfix) with ESMTP id 7E4E76B0032 for ; Mon, 16 Feb 2015 14:29:34 -0500 (EST) Received: by paceu11 with SMTP id eu11so507894pac.10 for ; Mon, 16 Feb 2015 11:29:34 -0800 (PST) Received: from mailout2.w1.samsung.com (mailout2.w1.samsung.com. [210.118.77.12]) by mx.google.com with ESMTPS id n9si10148716pdo.38.2015.02.16.11.29.33 for (version=TLSv1 cipher=RC4-MD5 bits=128/128); Mon, 16 Feb 2015 11:29:33 -0800 (PST) Received: from eucpsbgm1.samsung.com (unknown [203.254.199.244]) by mailout2.w1.samsung.com (Oracle Communications Messaging Server 7u4-24.01(7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTP id <0NJV00KYWQBVOVA0@mailout2.w1.samsung.com> for linux-mm@kvack.org; Mon, 16 Feb 2015 19:33:31 +0000 (GMT) Message-id: <54E24517.6040108@partner.samsung.com> Date: Mon, 16 Feb 2015 22:29:27 +0300 From: Stefan Strogin MIME-version: 1.0 Subject: Re: [PATCH 0/4] mm: cma: add some debug information for CMA References: <20150213030308.GG6592@js1304-P5Q-DELUXE> <54DEFBF4.40206@lge.com> In-reply-to: <54DEFBF4.40206@lge.com> Content-type: text/plain; charset=utf-8 Content-transfer-encoding: 7bit Sender: owner-linux-mm@kvack.org List-ID: To: Gioh Kim , Joonsoo Kim Cc: linux-mm@kvack.org, linux-kernel@vger.kernel.org, Andrew Morton , Marek Szyprowski , Michal Nazarewicz , aneesh.kumar@linux.vnet.ibm.com, Laurent Pinchart , Dmitry Safonov , Pintu Kumar , Weijie Yang , Laura Abbott , SeongJae Park , Hui Zhu , Minchan Kim , Dyasly Sergey , Vyacheslav Tyrtov , gregory.0xf0@gmail.com, sasha.levin@oracle.com, pavel@ucw.cz, stefan.strogin@gmail.com Hello Gioh, Thank you for your answer. On 14/02/15 10:40, Gioh Kim wrote: >> >> If this tracer is justifiable, I think that making it conditional is >> better than just enabling always on CONFIG_CMA_DEBUGFS. Some users >> don't want to this feature although they enable CONFIG_CMA_DEBUGFS. >> >> Thanks. >> > > Hello, > > Thanks for your work. It must be helpful to me. > > What about add another option to activate stack-trace? > In my platform I know all devices using cma area, so I usually don't > need stack-trace. > So Joonsoo suggests to add an option for buffer list, and you suggest to add another in addition to the first one (and also add CONFIG_STACKTRACE to dependences) right? -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755727AbbBPT3c (ORCPT ); Mon, 16 Feb 2015 14:29:32 -0500 Received: from mailout2.w1.samsung.com ([210.118.77.12]:13429 "EHLO mailout2.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752990AbbBPT3b (ORCPT ); Mon, 16 Feb 2015 14:29:31 -0500 X-AuditID: cbfec7f4-b7f126d000001e9a-bc-54e244885436 Message-id: <54E24517.6040108@partner.samsung.com> Date: Mon, 16 Feb 2015 22:29:27 +0300 From: Stefan Strogin User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0 MIME-version: 1.0 To: Gioh Kim , Joonsoo Kim Cc: linux-mm@kvack.org, linux-kernel@vger.kernel.org, Andrew Morton , Marek Szyprowski , Michal Nazarewicz , aneesh.kumar@linux.vnet.ibm.com, Laurent Pinchart , Dmitry Safonov , Pintu Kumar , Weijie Yang , Laura Abbott , SeongJae Park , Hui Zhu , Minchan Kim , Dyasly Sergey , Vyacheslav Tyrtov , gregory.0xf0@gmail.com, sasha.levin@oracle.com, pavel@ucw.cz, stefan.strogin@gmail.com Subject: Re: [PATCH 0/4] mm: cma: add some debug information for CMA References: <20150213030308.GG6592@js1304-P5Q-DELUXE> <54DEFBF4.40206@lge.com> In-reply-to: <54DEFBF4.40206@lge.com> Content-type: text/plain; charset=utf-8 Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFtrLIsWRmVeSWpSXmKPExsVy+t/xa7odLo9CDL7M4beYs34Nm8Xj1/NY LD6tfMpm8azpC6PFtOkbWC1WdjezWWzvnMFu0TlxCbvF5V1z2CzurfnParH2yF12iwXHW1gt ln19z25x99RRNou+74fZLSbPlrL4dnIOs8XiI7eZLa7t6GeyuPzmP7vF1Bk/2C1ObjjLarGh mctB3ONyXy+Tx85Zd9k9ZnfMZPXYtKqTzWPTp0nsHl1vrzB5nJjxm8XjwaHNLB7r/rxi8vj4 9BaLx8F3e5g8+rasYvRYsfo7u8fnTXIeXRt/sQbwR3HZpKTmZJalFunbJXBl7Ly9mLHgMmtF 35aD7A2M21m6GDk5JARMJP6/bWOEsMUkLtxbz9bFyMUhJLCUUeJL+1E2kISQwEdGiaOX+UBs XgEjiWVfP4E1sAioShz4cRXMZgMadOzCdDBbVCBCYv6x18wQ9YISPybfA1smIuAq8erXTEaQ BcwCZ1glPlzYxgSSEBZwkXj2fiIzxOZWRok799vBJnECbdi4oIO9i5EDqENdYsqUXJAws4C8 xOY1b5knMArMQrJjFkLVLCRVCxiZVzGKppYmFxQnpeca6hUn5haX5qXrJefnbmKERPmXHYyL j1kdYhTgYFTi4b1g8ihEiDWxrLgy9xCjBAezkghviDRQiDclsbIqtSg/vqg0J7X4ECMTB6dU A6Ow3yxh+wk+h5NPbXh2cFFm78ZSlcgT/vtO+ArelA+4rl/7eJ7V12xJgTszzRpqnLLatt2t vit+/rHw1cJNe1PX2N34Kr4lLfTneZEj0wuPz36Q6LuHmSk/88yO7uS7BROLF+ydsf3Q63Vn dpR8W8n312VV6NaWPY2qThxSclrOze/3zmBwXX5YiaU4I9FQi7moOBEAgxQA5NACAAA= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello Gioh, Thank you for your answer. On 14/02/15 10:40, Gioh Kim wrote: >> >> If this tracer is justifiable, I think that making it conditional is >> better than just enabling always on CONFIG_CMA_DEBUGFS. Some users >> don't want to this feature although they enable CONFIG_CMA_DEBUGFS. >> >> Thanks. >> > > Hello, > > Thanks for your work. It must be helpful to me. > > What about add another option to activate stack-trace? > In my platform I know all devices using cma area, so I usually don't > need stack-trace. > So Joonsoo suggests to add an option for buffer list, and you suggest to add another in addition to the first one (and also add CONFIG_STACKTRACE to dependences) right?