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=-2.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_PASS,URIBL_BLOCKED,USER_AGENT_MUTT autolearn=ham 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 9EB10C43142 for ; Wed, 27 Jun 2018 16:00:44 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 4D44425A92 for ; Wed, 27 Jun 2018 16:00:44 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 4D44425A92 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=intel.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754933AbeF0QAm (ORCPT ); Wed, 27 Jun 2018 12:00:42 -0400 Received: from mga01.intel.com ([192.55.52.88]:28283 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752937AbeF0QAk (ORCPT ); Wed, 27 Jun 2018 12:00:40 -0400 X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN X-Amp-File-Uploaded: False Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 27 Jun 2018 09:00:39 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.51,279,1526367600"; d="scan'208";a="50985185" Received: from bee.sh.intel.com (HELO bee) ([10.239.97.14]) by fmsmga008.fm.intel.com with ESMTP; 27 Jun 2018 09:00:36 -0700 Received: from kbuild by bee with local (Exim 4.84_2) (envelope-from ) id 1fYCs0-000Kh5-4y; Thu, 28 Jun 2018 00:00:36 +0800 Date: Wed, 27 Jun 2018 23:59:37 +0800 From: kbuild test robot To: Ming Lei Cc: kbuild-all@01.org, Jens Axboe , Christoph Hellwig , Kent Overstreet , David Sterba , Huang Ying , Mike Snitzer , linux-kernel@vger.kernel.org, linux-block@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-mm@kvack.org, Theodore Ts'o , "Darrick J . Wong" , Coly Li , Filipe Manana , Randy Dunlap , Ming Lei Subject: Re: [PATCH V7 10/24] block: introduce multipage page bvec helpers Message-ID: <201806272319.0p98i2TZ%fengguang.wu@intel.com> References: <20180627124548.3456-11-ming.lei@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180627124548.3456-11-ming.lei@redhat.com> User-Agent: Mutt/1.5.23 (2014-03-12) X-SA-Exim-Connect-IP: X-SA-Exim-Mail-From: lkp@intel.com X-SA-Exim-Scanned: No (on bee); SAEximRunCond expanded to false Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Ming, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on linus/master] [also build test WARNING on v4.18-rc2] [cannot apply to next-20180627] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Ming-Lei/block-support-multipage-bvec/20180627-214022 reproduce: # apt-get install sparse make ARCH=x86_64 allmodconfig make C=1 CF=-D__CHECK_ENDIAN__ sparse warnings: (new ones prefixed by >>) net/ceph/messenger.c:842:25: sparse: expression using sizeof(void) net/ceph/messenger.c:842:25: sparse: expression using sizeof(void) net/ceph/messenger.c:847:9: sparse: expression using sizeof(void) net/ceph/messenger.c:847:9: sparse: expression using sizeof(void) net/ceph/messenger.c:847:9: sparse: expression using sizeof(void) net/ceph/messenger.c:847:9: sparse: expression using sizeof(void) net/ceph/messenger.c:847:9: sparse: expression using sizeof(void) net/ceph/messenger.c:847:9: sparse: expression using sizeof(void) net/ceph/messenger.c:847:9: sparse: expression using sizeof(void) net/ceph/messenger.c:847:9: sparse: expression using sizeof(void) net/ceph/messenger.c:847:9: sparse: expression using sizeof(void) net/ceph/messenger.c:847:9: sparse: expression using sizeof(void) net/ceph/messenger.c:847:9: sparse: expression using sizeof(void) net/ceph/messenger.c:847:9: sparse: expression using sizeof(void) net/ceph/messenger.c:847:9: sparse: expression using sizeof(void) net/ceph/messenger.c:847:9: sparse: expression using sizeof(void) net/ceph/messenger.c:848:47: sparse: expression using sizeof(void) net/ceph/messenger.c:848:47: sparse: expression using sizeof(void) net/ceph/messenger.c:848:47: sparse: expression using sizeof(void) net/ceph/messenger.c:848:47: sparse: expression using sizeof(void) net/ceph/messenger.c:848:47: sparse: expression using sizeof(void) net/ceph/messenger.c:848:47: sparse: expression using sizeof(void) net/ceph/messenger.c:848:47: sparse: expression using sizeof(void) net/ceph/messenger.c:848:47: sparse: expression using sizeof(void) net/ceph/messenger.c:848:47: sparse: expression using sizeof(void) net/ceph/messenger.c:848:47: sparse: expression using sizeof(void) net/ceph/messenger.c:848:47: sparse: expression using sizeof(void) net/ceph/messenger.c:848:47: sparse: expression using sizeof(void) net/ceph/messenger.c:848:47: sparse: expression using sizeof(void) net/ceph/messenger.c:848:47: sparse: expression using sizeof(void) net/ceph/messenger.c:855:29: sparse: expression using sizeof(void) net/ceph/messenger.c:855:29: sparse: expression using sizeof(void) net/ceph/messenger.c:855:29: sparse: expression using sizeof(void) net/ceph/messenger.c:855:29: sparse: expression using sizeof(void) net/ceph/messenger.c:855:29: sparse: expression using sizeof(void) net/ceph/messenger.c:855:29: sparse: expression using sizeof(void) net/ceph/messenger.c:855:29: sparse: expression using sizeof(void) net/ceph/messenger.c:855:29: sparse: expression using sizeof(void) net/ceph/messenger.c:855:29: sparse: expression using sizeof(void) net/ceph/messenger.c:855:29: sparse: expression using sizeof(void) net/ceph/messenger.c:855:29: sparse: expression using sizeof(void) net/ceph/messenger.c:855:29: sparse: expression using sizeof(void) net/ceph/messenger.c:855:29: sparse: expression using sizeof(void) net/ceph/messenger.c:855:29: sparse: expression using sizeof(void) net/ceph/messenger.c:869:9: sparse: expression using sizeof(void) net/ceph/messenger.c:869:9: sparse: expression using sizeof(void) net/ceph/messenger.c:869:9: sparse: expression using sizeof(void) net/ceph/messenger.c:869:9: sparse: expression using sizeof(void) net/ceph/messenger.c:869:9: sparse: expression using sizeof(void) net/ceph/messenger.c:869:9: sparse: expression using sizeof(void) net/ceph/messenger.c:869:9: sparse: expression using sizeof(void) net/ceph/messenger.c:869:9: sparse: expression using sizeof(void) net/ceph/messenger.c:869:9: sparse: expression using sizeof(void) net/ceph/messenger.c:869:9: sparse: expression using sizeof(void) net/ceph/messenger.c:869:9: sparse: expression using sizeof(void) net/ceph/messenger.c:869:9: sparse: expression using sizeof(void) net/ceph/messenger.c:869:9: sparse: expression using sizeof(void) net/ceph/messenger.c:869:9: sparse: expression using sizeof(void) include/linux/bvec.h:139:37: sparse: expression using sizeof(void) include/linux/bvec.h:139:37: sparse: expression using sizeof(void) include/linux/bvec.h:139:37: sparse: expression using sizeof(void) include/linux/bvec.h:139:37: sparse: expression using sizeof(void) include/linux/bvec.h:139:37: sparse: expression using sizeof(void) include/linux/bvec.h:139:37: sparse: expression using sizeof(void) include/linux/bvec.h:139:37: sparse: expression using sizeof(void) include/linux/bvec.h:139:37: sparse: expression using sizeof(void) include/linux/bvec.h:139:37: sparse: expression using sizeof(void) include/linux/bvec.h:139:37: sparse: expression using sizeof(void) include/linux/bvec.h:139:37: sparse: expression using sizeof(void) include/linux/bvec.h:139:37: sparse: expression using sizeof(void) include/linux/bvec.h:139:37: sparse: expression using sizeof(void) include/linux/bvec.h:139:37: sparse: expression using sizeof(void) include/linux/bvec.h:140:32: sparse: expression using sizeof(void) include/linux/bvec.h:140:32: sparse: expression using sizeof(void) net/ceph/messenger.c:889:9: sparse: expression using sizeof(void) net/ceph/messenger.c:889:9: sparse: expression using sizeof(void) net/ceph/messenger.c:889:9: sparse: expression using sizeof(void) net/ceph/messenger.c:889:9: sparse: expression using sizeof(void) net/ceph/messenger.c:889:9: sparse: expression using sizeof(void) net/ceph/messenger.c:889:9: sparse: expression using sizeof(void) net/ceph/messenger.c:889:9: sparse: expression using sizeof(void) net/ceph/messenger.c:889:9: sparse: expression using sizeof(void) net/ceph/messenger.c:889:9: sparse: expression using sizeof(void) net/ceph/messenger.c:889:9: sparse: expression using sizeof(void) net/ceph/messenger.c:889:9: sparse: expression using sizeof(void) net/ceph/messenger.c:889:9: sparse: expression using sizeof(void) net/ceph/messenger.c:889:9: sparse: expression using sizeof(void) net/ceph/messenger.c:889:9: sparse: expression using sizeof(void) net/ceph/messenger.c:890:47: sparse: expression using sizeof(void) net/ceph/messenger.c:890:47: sparse: expression using sizeof(void) net/ceph/messenger.c:890:47: sparse: expression using sizeof(void) net/ceph/messenger.c:890:47: sparse: expression using sizeof(void) net/ceph/messenger.c:890:47: sparse: expression using sizeof(void) net/ceph/messenger.c:890:47: sparse: expression using sizeof(void) net/ceph/messenger.c:890:47: sparse: expression using sizeof(void) net/ceph/messenger.c:890:47: sparse: expression using sizeof(void) net/ceph/messenger.c:890:47: sparse: expression using sizeof(void) net/ceph/messenger.c:890:47: sparse: expression using sizeof(void) net/ceph/messenger.c:890:47: sparse: expression using sizeof(void) >> net/ceph/messenger.c:890:47: sparse: too many warnings vim +890 net/ceph/messenger.c 6aaa4511 Alex Elder 2013-03-06 862 8ae4f4f5 Alex Elder 2013-03-14 863 static bool ceph_msg_data_bio_advance(struct ceph_msg_data_cursor *cursor, 8ae4f4f5 Alex Elder 2013-03-14 864 size_t bytes) 6aaa4511 Alex Elder 2013-03-06 865 { 5359a17d Ilya Dryomov 2018-01-20 866 struct ceph_bio_iter *it = &cursor->bio_iter; 6aaa4511 Alex Elder 2013-03-06 867 5359a17d Ilya Dryomov 2018-01-20 868 BUG_ON(bytes > cursor->resid); 5359a17d Ilya Dryomov 2018-01-20 869 BUG_ON(bytes > bio_iter_len(it->bio, it->iter)); 25aff7c5 Alex Elder 2013-03-11 870 cursor->resid -= bytes; 5359a17d Ilya Dryomov 2018-01-20 871 bio_advance_iter(it->bio, &it->iter, bytes); f38a5181 Kent Overstreet 2013-08-07 872 5359a17d Ilya Dryomov 2018-01-20 873 if (!cursor->resid) { 5359a17d Ilya Dryomov 2018-01-20 874 BUG_ON(!cursor->last_piece); 5359a17d Ilya Dryomov 2018-01-20 875 return false; /* no more data */ 5359a17d Ilya Dryomov 2018-01-20 876 } f38a5181 Kent Overstreet 2013-08-07 877 5359a17d Ilya Dryomov 2018-01-20 878 if (!bytes || (it->iter.bi_size && it->iter.bi_bvec_done)) 6aaa4511 Alex Elder 2013-03-06 879 return false; /* more bytes to process in this segment */ 6aaa4511 Alex Elder 2013-03-06 880 5359a17d Ilya Dryomov 2018-01-20 881 if (!it->iter.bi_size) { 5359a17d Ilya Dryomov 2018-01-20 882 it->bio = it->bio->bi_next; 5359a17d Ilya Dryomov 2018-01-20 883 it->iter = it->bio->bi_iter; 5359a17d Ilya Dryomov 2018-01-20 884 if (cursor->resid < it->iter.bi_size) 5359a17d Ilya Dryomov 2018-01-20 885 it->iter.bi_size = cursor->resid; 25aff7c5 Alex Elder 2013-03-11 886 } 6aaa4511 Alex Elder 2013-03-06 887 5359a17d Ilya Dryomov 2018-01-20 888 BUG_ON(cursor->last_piece); 5359a17d Ilya Dryomov 2018-01-20 889 BUG_ON(cursor->resid < bio_iter_len(it->bio, it->iter)); 5359a17d Ilya Dryomov 2018-01-20 @890 cursor->last_piece = cursor->resid == bio_iter_len(it->bio, it->iter); 6aaa4511 Alex Elder 2013-03-06 891 return true; 6aaa4511 Alex Elder 2013-03-06 892 } ea96571f Alex Elder 2013-04-05 893 #endif /* CONFIG_BLOCK */ df6ad1f9 Alex Elder 2012-06-11 894 :::::: The code at line 890 was first introduced by commit :::::: 5359a17d2706b86da2af83027343d5eb256f7670 libceph, rbd: new bio handling code (aka don't clone bios) :::::: TO: Ilya Dryomov :::::: CC: Ilya Dryomov --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation