Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932801AbdCJOzH (ORCPT ); Fri, 10 Mar 2017 09:55:07 -0500 Received: from mx1.redhat.com ([209.132.183.28]:39086 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751893AbdCJOzF (ORCPT ); Fri, 10 Mar 2017 09:55:05 -0500 Date: Fri, 10 Mar 2017 09:55:01 -0500 (EST) From: Mikulas Patocka X-X-Sender: mpatocka@file01.intranet.prod.int.rdu2.redhat.com To: Mike Snitzer cc: Lars Ellenberg , NeilBrown , Jens Axboe , Jack Wang , LKML , Kent Overstreet , Pavel Machek , linux-raid@vger.kernel.org, device-mapper development , linux-block@vger.kernel.org Subject: Re: [PATCH v2] blk: improve order of bio handling in generic_make_request() In-Reply-To: <20170310143822.GA23879@redhat.com> Message-ID: References: <87h93blz6g.fsf@notabene.neil.brown.name> <71562c2c-97f4-9a0a-32ec-30e0702ca575@profitbricks.com> <87lgsjj9w8.fsf@notabene.neil.brown.name> <87r328j00i.fsf@notabene.neil.brown.name> <87d1dphhuy.fsf@notabene.neil.brown.name> <58be6551-4aa7-72ee-1616-a1545606d029@kernel.dk> <87varhg14d.fsf@notabene.neil.brown.name> <20170310143822.GA23879@redhat.com> User-Agent: Alpine 2.02 (LRH 1266 2009-07-14) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.25]); Fri, 10 Mar 2017 14:55:05 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1718 Lines: 46 On Fri, 10 Mar 2017, Mike Snitzer wrote: > On Fri, Mar 10 2017 at 7:34am -0500, > Lars Ellenberg wrote: > > > > --- a/block/blk-core.c > > > +++ b/block/blk-core.c > > > @@ -1975,7 +1975,14 @@ generic_make_request_checks(struct bio *bio) > > > */ > > > blk_qc_t generic_make_request(struct bio *bio) > > > { > > > - struct bio_list bio_list_on_stack; > > > + /* > > > + * bio_list_on_stack[0] contains bios submitted by the current > > > + * make_request_fn. > > > + * bio_list_on_stack[1] contains bios that were submitted before > > > + * the current make_request_fn, but that haven't been processed > > > + * yet. > > > + */ > > > + struct bio_list bio_list_on_stack[2]; > > > blk_qc_t ret = BLK_QC_T_NONE; > > > > May I suggest that, if you intend to assign something that is not a > > plain &(struct bio_list), but a &(struct bio_list[2]), > > you change the task member so it is renamed (current->bio_list vs > > current->bio_lists, plural, is what I did last year). > > Or you will break external modules, silently, and horribly (or, > > rather, they won't notice, but break the kernel). > > Examples of such modules would be DRBD, ZFS, quite possibly others. > > drbd is upstream -- so what is the problem? (if you are having to > distribute drbd independent of the upstream drbd then why is drbd > upstream?) > > As for ZFS, worrying about ZFS kABI breakage is the last thing we should > be doing. It's better to make external modules not compile than to silently introduce bugs in them. So yes, I would rename that. Mikulas > So Nack from me on this defensive make-work for external modules. >