From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-0.8 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0AE2FCA9EB9 for ; Sat, 26 Oct 2019 06:46:17 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id D115121655 for ; Sat, 26 Oct 2019 06:46:16 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726137AbfJZGqQ (ORCPT ); Sat, 26 Oct 2019 02:46:16 -0400 Received: from kvm5.telegraphics.com.au ([98.124.60.144]:58602 "EHLO kvm5.telegraphics.com.au" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726010AbfJZGqQ (ORCPT ); Sat, 26 Oct 2019 02:46:16 -0400 Received: from localhost (localhost.localdomain [127.0.0.1]) by kvm5.telegraphics.com.au (Postfix) with ESMTP id E98AF2A49A; Sat, 26 Oct 2019 02:46:09 -0400 (EDT) Date: Sat, 26 Oct 2019 17:46:08 +1100 (AEDT) From: Finn Thain To: =?ISO-8859-15?Q?Michal_Such=E1nek?= cc: Christoph Hellwig , linux-scsi@vger.kernel.org, Jonathan Corbet , Jens Axboe , "James E.J. Bottomley" , "Martin K. Petersen" , Alexander Viro , Mauro Carvalho Chehab , Eric Biggers , "J. Bruce Fields" , Benjamin Coddington , Hannes Reinecke , Omar Sandoval , Ming Lei , Damien Le Moal , Bart Van Assche , Tejun Heo , linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org Subject: Re: [PATCH v2 2/8] cdrom: factor out common open_for_* code In-Reply-To: <20191025104230.GN938@kitsune.suse.cz> Message-ID: References: <20191024021958.GA11485@infradead.org> <20191024085014.GF938@kitsune.suse.cz> <20191025023908.GB14108@infradead.org> <20191025104230.GN938@kitsune.suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-scsi-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-scsi@vger.kernel.org On Fri, 25 Oct 2019, Michal Such?nek wrote: > On Thu, Oct 24, 2019 at 07:39:08PM -0700, Christoph Hellwig wrote: > > On Thu, Oct 24, 2019 at 10:50:14AM +0200, Michal Such?nek wrote: > > > Then I will get complaints I do unrelated changes and it's hard to > > > review. The code gets removed later anyway. > > > > If you refactor you you pretty much have a card blanche for the > > refactored code and the direct surroundings. > > This is different from what other reviewers say: > > https://lore.kernel.org/lkml/1517245320.2687.14.camel@wdc.com/ > I don't see any inconsistency there. Both reviews are valuable. In general, different reviewers may give contradictory advice. Reviewers probably even contradict themselves eventually. Yet it rarely happens that the same patch gets contradictory reviews. If it did, you might well complain. > Either way, this code is removed in a later patch so this discussion is > moot. > > It makes sense to have a bisection point here in case something > goes wrong but it is pointless to argue about the code structure > inherited from the previous revision. A patch may refactor some code only to have the next patch remove that code. This doesn't generally mean that the former patch is redundant. The latter patch may end up committed and subsequently reverted. The latter patch may become easier to review because of the former. The former patch may be eligible for -stable. The former patch may be the result of an automatic process. And so on. I don't know what Christoph had in mind here but he's usually right, so it's worth asking. -- > > Thanks > > Michal >