From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751702AbdK1JKS (ORCPT ); Tue, 28 Nov 2017 04:10:18 -0500 Received: from mx2.suse.de ([195.135.220.15]:38299 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750975AbdK1JKJ (ORCPT ); Tue, 28 Nov 2017 04:10:09 -0500 Date: Tue, 28 Nov 2017 10:10:05 +0100 Message-ID: From: Takashi Iwai To: SF Markus Elfring Cc: alsa-devel@alsa-project.org, Arvind Yadav , Jaroslav Kysela , Takashi Sakamoto , kernel-janitors@vger.kernel.org, LKML Subject: Re: ALSA: nm256: Fine-tuning for three function implementations In-Reply-To: <2cbef557-5f89-c630-e108-14ef2ce6b41a@users.sourceforge.net> References: <24f8c777-1eb4-e7e7-9371-79f32700c9dc@users.sourceforge.net> <2cbef557-5f89-c630-e108-14ef2ce6b41a@users.sourceforge.net> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.8 Emacs/25.3 (x86_64-suse-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 28 Nov 2017 09:19:48 +0100, SF Markus Elfring wrote: > > >>>> There is a general source code transformation pattern involved. > >>>> So I find that it is systematic. > >>>> > >>>> But I did not dare to develop a script variant for the semantic patch > >>>> language (Coccinelle software) which can handle all special use cases > >>>> as a few of them are already demonstrated in this tiny patch series. > >>> > >>> Then you're doing everything by hands, > >> > >> I am navigating through possible changes around the pattern > >> “Use common error handling code” mostly manually so far. > >> > >> > >>> and can be wrong > >> > >> Such a possibility remains as usual. > > > > "As usual" doesn't suffice. > > There can be additional means be used to reduce the probability > of undesired side effects. Irrelevant, it doesn't fix a bug, nor dramatic improvement. > > It must be "almost perfect" for such a code refactoring. > > Can you get the impression that the shown transformation patterns were correctly > applied for the source file “sound/pci/nm256/nm256.c”? Impression doesn't matter. The question is whether it's 100% correct or not in such a case. > > The damage by a overseen mistake is much higher than the merit by such a patch. > > Are there any more software developers and code reviewers available > who would like to point another programming mistake out for this Linux module? If you have find such, then it's fine, you can get your patches reviewed and more assured. But in the current situation, no one else is interested in it, and that's going to nowhere. > > If the patch is about fixing a bug, it's a different story. > > How do “deviations” from the coding style and the evolution of object code size > fit to this view here? Again, it's no fix for a bug. > > Or it's about a really trivial change (e.g. your sizeof() conversion > > patches), I can check and apply easily. > > My update selection can contain also trivial adjustments. The *really* trivial ones were applied. The rest are not. > > But for other changes with more lines, it makes little sense. > > Do you need any more information to see and eventually accept the sense again? No. This kind of code refactoring has no more information. It's a "trivial" change, after all. > > Again, the risk of breakage increases while the merit is negligible. > > We disagree about corresponding benefits at the moment. > Would any other contributors comment the situation a bit more? I hear openly. > >>> -- that's the heart of the problem. > >> > >> There might be related opportunities for further improvements. > >> Do you trust adjustments from an evolving tool more than > >> my concrete contributions? > > > > Yes, loudly. > > I noticed that the development status of tools which you might find nice > at the moment can be also questionable. It depends on the result. > > I stop at this point, as the rest is simply a repeat from the previous mail. > > Are you using a continuous integration system? Not really in my side. But there are others doing that. Takashi