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@xxxxxxx/ > 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 >
