Received: by 2002:a05:6a10:f347:0:0:0:0 with SMTP id d7csp2422790pxu; Mon, 14 Dec 2020 01:47:52 -0800 (PST) X-Google-Smtp-Source: ABdhPJzk2uXikeNKY3RuPj4HU0GTITwNA809SGsPIWtSf5WSbRB6fBMYrPhrIdZGCKkSr8MFb7Z7 X-Received: by 2002:a17:906:7118:: with SMTP id x24mr1740078ejj.333.1607939272142; Mon, 14 Dec 2020 01:47:52 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1607939272; cv=none; d=google.com; s=arc-20160816; b=uUjo8jmwoqDc98pcaVLV8lu/gDCXaxtPhnJeSrE+tTLKOlX6iVzOXFl7W6Rc2DIfFo nSL6zWQ75M34zWPVIGh4XsfoWWFciZ5GoyW2zuGNfUyts3swJ0HUUnqXN67lwy8ugpGM DwIY/heYFwKhU3Wa/i/RUlDMafRfHO5wx3Zejd8KUV/teHUQJQBpHSj9+9WCtJqW82Ok VKbPguedCvCU2fYt/5DxuSNPoALpp17WH7XXdgjH5BOkhHqgdviXYXYi1Nj56Rfcmu7j rqrOUMP4vFhc+Jo8zhSEw8cFRE5WxE4Oz0o1XEOFluWJ+XUjeEJ20vsuTCFaN5AJMoEe Of6A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:mime-version:message-id:subject:cc:to:from:date :dkim-signature; bh=tBhsL6REyRvOd/e8dX1kDFWpy/QxJWV1iYmuLwHeWEI=; b=cznW1lrOKu1P9+Yl+AdIAyNUcJiUPv3fjlF8IL7eif0NKakIY/F0d+FShX4IfQK8TL jOk9xbpEO591x+tM9+CLhRU7U8zSrfOAhrBpQE/ddh3t+T4YGgintzDOgufxeB5tV1K1 3W8MH0iv4yAOBZjwmFzzo05ZqeJ+MD9WwJTbpJiBjQQ58+OLekYrBo8GfwtdTqPTevUR AoaCutKFAbfOUh3xPKHZVtkfLgfR7TnMoPFWbz7KSvPk8q5nWwMdbirC20ILat/bbhgP 1eU8OA4cxXmh5Bu3GST7l3ANcJX5A+0HdGKMPkg4ydzwH8ip9W8TjzEV4YeKS7HEhE61 aVMA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@canb.auug.org.au header.s=201702 header.b=V1vNNohq; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id a18si9422948ejb.180.2020.12.14.01.47.29; Mon, 14 Dec 2020 01:47:52 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@canb.auug.org.au header.s=201702 header.b=V1vNNohq; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2438559AbgLNESr (ORCPT + 99 others); Sun, 13 Dec 2020 23:18:47 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35866 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725911AbgLNESr (ORCPT ); Sun, 13 Dec 2020 23:18:47 -0500 Received: from ozlabs.org (bilbo.ozlabs.org [IPv6:2401:3900:2:1::2]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D9555C0613CF; Sun, 13 Dec 2020 20:18:06 -0800 (PST) Received: from authenticated.ozlabs.org (localhost [127.0.0.1]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-256) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by mail.ozlabs.org (Postfix) with ESMTPSA id 4CvSmX4mstz9sT6; Mon, 14 Dec 2020 15:18:04 +1100 (AEDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=canb.auug.org.au; s=201702; t=1607919484; bh=Ap4veYE6g8S6YKlenbtrX+KmmgQamRiHcRuH1VqFN+s=; h=Date:From:To:Cc:Subject:From; b=V1vNNohqhMdlrFbS4ZZYvngSMFee7gyh97cqz97pWhDiVPHQ3W+37m+OicYCvzBNs 35VbiBgIa3+H60yTvjMsoCIiVHvSTKHJAcmG9uJ8PpHA8bwQCYXW+IjLbB4AbdsjLm QknQECl8GNvV+myaxH/3ynDOt84b/mqtmGNx2mqCXF0Za7CEKE8hoKOxW6rCr8GACg ClQLit/z1Q9nsseSvLo0/nqHxrWTW+0zZ+bqIDKARFShx3dWmSqppmiFQhyuesnW64 URNx8PH32Zdhk0GhODELBUFxdWyUjxXWG0pcGTo5c3vZxgJoorV0SU3F1OQJyhFaWx e0pnOjUbqO0Sw== Date: Mon, 14 Dec 2020 15:18:03 +1100 From: Stephen Rothwell To: Jens Axboe Cc: Christoph Hellwig , Song Liu , Linux Kernel Mailing List , Linux Next Mailing List Subject: linux-next: build failure after merge of the block tree Message-ID: <20201214151803.411b3b41@canb.auug.org.au> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="Sig_/rgN3xbXPFOB5eCTLP4HJp/x"; protocol="application/pgp-signature"; micalg=pgp-sha256 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --Sig_/rgN3xbXPFOB5eCTLP4HJp/x Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable Hi all, After merging the block tree, today's linux-next build (powerpc ppc64_defconfig) failed like this: drivers/md/raid0.c: In function 'raid0_handle_discard': drivers/md/raid0.c:511:26: error: passing argument 1 of 'trace_block_bio_re= map' from incompatible pointer type [-Werror=3Dincompatible-pointer-types] 511 | trace_block_bio_remap(bdev_get_queue(rdev->bdev), | ^~~~~~~~~~~~~~~~~~~~~~~~~~ | | | struct request_queue * In file included from include/trace/events/block.h:11, from drivers/md/raid0.c:17: include/trace/events/block.h:458:23: note: expected 'struct bio *' but argu= ment is of type 'struct request_queue *' 458 | TP_PROTO(struct bio *bio, dev_t dev, sector_t from), | ~~~~~~~~~~~~^~~ include/linux/tracepoint.h:238:34: note: in definition of macro '__DECLARE_= TRACE' 238 | static inline void trace_##name(proto) \ | ^~~~~ include/linux/tracepoint.h:411:24: note: in expansion of macro 'PARAMS' 411 | __DECLARE_TRACE(name, PARAMS(proto), PARAMS(args), \ | ^~~~~~ include/linux/tracepoint.h:547:2: note: in expansion of macro 'DECLARE_TRAC= E' 547 | DECLARE_TRACE(name, PARAMS(proto), PARAMS(args)) | ^~~~~~~~~~~~~ include/linux/tracepoint.h:547:22: note: in expansion of macro 'PARAMS' 547 | DECLARE_TRACE(name, PARAMS(proto), PARAMS(args)) | ^~~~~~ include/trace/events/block.h:456:1: note: in expansion of macro 'TRACE_EVEN= T' 456 | TRACE_EVENT(block_bio_remap, | ^~~~~~~~~~~ include/trace/events/block.h:458:2: note: in expansion of macro 'TP_PROTO' 458 | TP_PROTO(struct bio *bio, dev_t dev, sector_t from), | ^~~~~~~~ drivers/md/raid0.c:512:5: warning: passing argument 2 of 'trace_block_bio_r= emap' makes integer from pointer without a cast [-Wint-conversion] 512 | discard_bio, disk_devt(mddev->gendisk), | ^~~~~~~~~~~ | | | struct bio * In file included from include/trace/events/block.h:11, from drivers/md/raid0.c:17: include/trace/events/block.h:458:34: note: expected 'dev_t' {aka 'unsigned = int'} but argument is of type 'struct bio *' 458 | TP_PROTO(struct bio *bio, dev_t dev, sector_t from), | ~~~~~~^~~ include/linux/tracepoint.h:238:34: note: in definition of macro '__DECLARE_= TRACE' 238 | static inline void trace_##name(proto) \ | ^~~~~ include/linux/tracepoint.h:411:24: note: in expansion of macro 'PARAMS' 411 | __DECLARE_TRACE(name, PARAMS(proto), PARAMS(args), \ | ^~~~~~ include/linux/tracepoint.h:547:2: note: in expansion of macro 'DECLARE_TRAC= E' 547 | DECLARE_TRACE(name, PARAMS(proto), PARAMS(args)) | ^~~~~~~~~~~~~ include/linux/tracepoint.h:547:22: note: in expansion of macro 'PARAMS' 547 | DECLARE_TRACE(name, PARAMS(proto), PARAMS(args)) | ^~~~~~ include/trace/events/block.h:456:1: note: in expansion of macro 'TRACE_EVEN= T' 456 | TRACE_EVENT(block_bio_remap, | ^~~~~~~~~~~ include/trace/events/block.h:458:2: note: in expansion of macro 'TP_PROTO' 458 | TP_PROTO(struct bio *bio, dev_t dev, sector_t from), | ^~~~~~~~ drivers/md/raid0.c:511:4: error: too many arguments to function 'trace_bloc= k_bio_remap' 511 | trace_block_bio_remap(bdev_get_queue(rdev->bdev), | ^~~~~~~~~~~~~~~~~~~~~ In file included from include/trace/events/block.h:11, from drivers/md/raid0.c:17: include/linux/tracepoint.h:238:21: note: declared here 238 | static inline void trace_##name(proto) \ | ^~~~~~ include/linux/tracepoint.h:411:2: note: in expansion of macro '__DECLARE_TR= ACE' 411 | __DECLARE_TRACE(name, PARAMS(proto), PARAMS(args), \ | ^~~~~~~~~~~~~~~ include/linux/tracepoint.h:547:2: note: in expansion of macro 'DECLARE_TRAC= E' 547 | DECLARE_TRACE(name, PARAMS(proto), PARAMS(args)) | ^~~~~~~~~~~~~ include/trace/events/block.h:456:1: note: in expansion of macro 'TRACE_EVEN= T' 456 | TRACE_EVENT(block_bio_remap, | ^~~~~~~~~~~ Caused by commit 1c02fca620f7 ("block: remove the request_queue argument to the block_bio_= remap tracepoint") interacting with commit 57a0f3a81ef2 ("Revert "md: add md_submit_discard_bio() for submitting dis= card bio"") from Linus' tree. I have applied the following merge fix: From: Stephen Rothwell Date: Mon, 14 Dec 2020 15:14:26 +1100 Subject: [PATCH] fixup for "block: remove the request_queue argument to the= block_bio_remap tracepoint" conflicting with "Revert "md: add md_submit_discard_bio() for submitting discard bio"" Signed-off-by: Stephen Rothwell --- drivers/md/raid0.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/md/raid0.c b/drivers/md/raid0.c index 8f5b0143d8c6..67f157f2525d 100644 --- a/drivers/md/raid0.c +++ b/drivers/md/raid0.c @@ -508,8 +508,8 @@ static void raid0_handle_discard(struct mddev *mddev, s= truct bio *bio) bio_chain(discard_bio, bio); bio_clone_blkg_association(discard_bio, bio); if (mddev->gendisk) - trace_block_bio_remap(bdev_get_queue(rdev->bdev), - discard_bio, disk_devt(mddev->gendisk), + trace_block_bio_remap(discard_bio, + disk_devt(mddev->gendisk), bio->bi_iter.bi_sector); submit_bio_noacct(discard_bio); } --=20 2.29.2 --=20 Cheers, Stephen Rothwell --Sig_/rgN3xbXPFOB5eCTLP4HJp/x Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- iQEzBAEBCAAdFiEENIC96giZ81tWdLgKAVBC80lX0GwFAl/W53sACgkQAVBC80lX 0GwR3wf/VaRztQkjmysC3E5kSROkNLTnSAoLIAi6it6wcBMXe0I+SK5E+DJK/pOt QMVrL2Xx235qjTU3Zuqjr470XVtvrTX8PfBZnLOeiIqZ0IpyZ8xdbWyyQLWwc2O/ t621f+alBHBW0Q9F+YwuL6CQY0PV3S9Pw+9OJ+Q12VQ9niyRQXecrNfr9yJnJJxj Msp0KdhKuNRqzGZmtjn+gFUkmIDyAVVCboKgr5CwnwQsdqzyfScuwnJiKWiCGtqm ewdk2HwlqgA6PuKoeTGTbJ66x7bSQ7sPt6uRpeQmaXnaWSH5N9XUX4S1v91OtSvs ZUdg+A6TMqMZXmFncMjaW88mPf7f9A== =QpF7 -----END PGP SIGNATURE----- --Sig_/rgN3xbXPFOB5eCTLP4HJp/x--