From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759140AbcIWNGi (ORCPT ); Fri, 23 Sep 2016 09:06:38 -0400 Received: from pegasos-out.vodafone.de ([80.84.1.38]:42270 "EHLO pegasos-out.vodafone.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754016AbcIWNGe (ORCPT ); Fri, 23 Sep 2016 09:06:34 -0400 X-Spam-Flag: NO X-Spam-Score: -0.045 Authentication-Results: rohrpostix1.prod.vfnet.de (amavisd-new); dkim=pass header.i=@vodafone.de X-DKIM: OpenDKIM Filter v2.6.8 pegasos-out.vodafone.de F1107261733 Subject: Re: GPU-DRM-TTM: Fine-tuning for several function implementations To: SF Markus Elfring References: <566ABCD9.1060404@users.sourceforge.net> <4d34446f-05ad-c3ce-5d33-8fb4f25af25c@users.sourceforge.net> <05418fb1-ad66-aba3-bd8c-f6b684a83279@users.sourceforge.net> <160f8c46-3601-aca9-cccc-054531cf16a8@users.sourceforge.net> Cc: dri-devel@lists.freedesktop.org, Daniel Vetter , David Airlie , Emil Velikov , Julia Lawall , kernel-janitors@vger.kernel.org, LKML From: =?UTF-8?Q?Christian_K=c3=b6nig?= Message-ID: <21112c34-13cd-af03-2672-58f46bb2fb96@vodafone.de> Date: Fri, 23 Sep 2016 15:06:25 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: <160f8c46-3601-aca9-cccc-054531cf16a8@users.sourceforge.net> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Am 23.09.2016 um 13:49 schrieb SF Markus Elfring: >> Calling the label "unlock" instead of "out" is arguable a little better, > Thanks that you can follow a renaming for this direction in principle. > > >> but nothing I would call a major improvement either. > This was not my intention for such an use case. > > I am proposing some small software updates according to such a design pattern. > > >> So that is a clear NAK to all those patches. > Do you reject also update steps like the following then? > > * drm/ttm: Use kmalloc_array() in two (or four?) functions" > > * drm/ttm: Less function calls in ttm_dma_pool_init() after error detection The reason behind the advise to use kmalloc_array() is to avoid overruns when one of the parameters come from an IOCTL and so are controllable by user space. Those overruns where the source of numerous security problems, but in this case the parameters don't come from an IOCTL and aren't user space controllable. So this change actually doesn't make to much sense either, but I'm leaning towards accepting them for coding style consistency. Regards, Christian. > * Would you like to improve the usage of the variables "n" and "t" > in the function "ttm_dma_pool_init" any further as Joe Perches suggested it? > > >>> 2. How do you think about to add a single space character before any label? >> Bad as well. Why would anybody want to do this? > Do you find another software evolution interesting according to a recent commit? > > "docs: Remove space-before-label guidance from CodingStyle" > https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/commit/Documentation/CodingStyle?id=79c70c304b0b443429b2a0019518532c5162817a > > > Regards, > Markus