From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Subject: Re: Is the libiscsi engine patch applied correctly? References: <20190425155203.GA11294@redhat.com> <035ceaed-70a7-da29-8677-f1669f7f09db@kernel.dk> <20190425202214.GL3926@redhat.com> From: Jens Axboe Message-ID: Date: Thu, 25 Apr 2019 14:25:12 -0600 MIME-Version: 1.0 In-Reply-To: <20190425202214.GL3926@redhat.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit To: "Richard W.M. Jones" Cc: fio@vger.kernel.org List-ID: On 4/25/19 2:22 PM, Richard W.M. Jones wrote: > On Thu, Apr 25, 2019 at 10:10:40AM -0600, Jens Axboe wrote: >> On 4/25/19 9:52 AM, Richard W.M. Jones wrote: >>> It seems like these two lines might be in the wrong order? (and the >>> same thing later in the file too) >>> >>> https://github.com/axboe/fio/blob/afc22c98609ee80a99fbed24231181bdab2bc659/optgroup.h#L65-L66 >> >> They are, that's a bug. Thanks for letting me know, I've fixed it up. > > Thanks. I think also these lines later in the same file -- although > not technically a bug -- should also be swapped around: > > FIO_OPT_G_INVALID = (1ULL << __FIO_OPT_G_NR), > FIO_OPT_G_ISCSI = (1ULL << __FIO_OPT_G_ISCSI), Order of those don't really matter. But for pure aesthetics, yeah they should probably be swapped. -- Jens Axboe