From: kbuild test robot <lkp@intel.com>
To: James Simmons <jsimmons@infradead.org>
Cc: kbuild-all@01.org,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
devel@driverdev.osuosl.org,
Andreas Dilger <andreas.dilger@intel.com>,
Oleg Drokin <oleg.drokin@intel.com>, Ben Evans <bevans@cray.com>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Lustre Development List <lustre-devel@lists.lustre.org>
Subject: Re: [PATCH 3/6] staging: lustre: obdclass: Create a header for obdo related functions
Date: Sat, 3 Dec 2016 21:27:31 +0800 [thread overview]
Message-ID: <201612032108.E93gbaZn%fengguang.wu@intel.com> (raw)
In-Reply-To: <1480707650-24089-4-git-send-email-jsimmons@infradead.org>
[-- Attachment #1: Type: text/plain, Size: 5241 bytes --]
Hi Ben,
[auto build test ERROR on staging/staging-testing]
[also build test ERROR on next-20161202]
[cannot apply to v4.9-rc7]
[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/James-Simmons/Fix-ups-to-make-lustre_idl-h-a-proper-UAPI-header/20161203-112301
config: s390-allmodconfig (attached as .config)
compiler: s390x-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705
reproduce:
wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
make.cross ARCH=s390
Note: the linux-review/James-Simmons/Fix-ups-to-make-lustre_idl-h-a-proper-UAPI-header/20161203-112301 HEAD c2346c2b941862f21c17e2c11dfd2ed0e66f72ea builds fine.
It only hurts bisectibility.
All errors (new ones prefixed by >>):
drivers/staging/lustre/lustre/osc/osc_request.c: In function 'osc_pack_req_body':
>> drivers/staging/lustre/lustre/osc/osc_request.c:114:2: error: implicit declaration of function 'lustre_set_wire_obdo' [-Werror=implicit-function-declaration]
lustre_set_wire_obdo(&req->rq_import->imp_connect_data, &body->oa, oa);
^~~~~~~~~~~~~~~~~~~~
drivers/staging/lustre/lustre/osc/osc_request.c: In function 'osc_getattr':
>> drivers/staging/lustre/lustre/osc/osc_request.c:149:2: error: implicit declaration of function 'lustre_get_wire_obdo' [-Werror=implicit-function-declaration]
lustre_get_wire_obdo(&req->rq_import->imp_connect_data, oa,
^~~~~~~~~~~~~~~~~~~~
cc1: some warnings being treated as errors
vim +/lustre_set_wire_obdo +114 drivers/staging/lustre/lustre/osc/osc_request.c
d7e09d039 Peng Tao 2013-05-02 108 {
d7e09d039 Peng Tao 2013-05-02 109 struct ost_body *body;
d7e09d039 Peng Tao 2013-05-02 110
d7e09d039 Peng Tao 2013-05-02 111 body = req_capsule_client_get(&req->rq_pill, &RMF_OST_BODY);
d7e09d039 Peng Tao 2013-05-02 112 LASSERT(body);
d7e09d039 Peng Tao 2013-05-02 113
48a2a12af John L. Hammond 2016-10-02 @114 lustre_set_wire_obdo(&req->rq_import->imp_connect_data, &body->oa, oa);
d7e09d039 Peng Tao 2013-05-02 115 }
d7e09d039 Peng Tao 2013-05-02 116
d7e09d039 Peng Tao 2013-05-02 117 static int osc_getattr(const struct lu_env *env, struct obd_export *exp,
48a2a12af John L. Hammond 2016-10-02 118 struct obdo *oa)
d7e09d039 Peng Tao 2013-05-02 119 {
d7e09d039 Peng Tao 2013-05-02 120 struct ptlrpc_request *req;
d7e09d039 Peng Tao 2013-05-02 121 struct ost_body *body;
d7e09d039 Peng Tao 2013-05-02 122 int rc;
d7e09d039 Peng Tao 2013-05-02 123
d7e09d039 Peng Tao 2013-05-02 124 req = ptlrpc_request_alloc(class_exp2cliimp(exp), &RQF_OST_GETATTR);
7f1ae4c06 Oleg Drokin 2016-02-16 125 if (!req)
0a3bdb007 Greg Kroah-Hartman 2013-08-03 126 return -ENOMEM;
d7e09d039 Peng Tao 2013-05-02 127
d7e09d039 Peng Tao 2013-05-02 128 rc = ptlrpc_request_pack(req, LUSTRE_OST_VERSION, OST_GETATTR);
d7e09d039 Peng Tao 2013-05-02 129 if (rc) {
d7e09d039 Peng Tao 2013-05-02 130 ptlrpc_request_free(req);
0a3bdb007 Greg Kroah-Hartman 2013-08-03 131 return rc;
d7e09d039 Peng Tao 2013-05-02 132 }
d7e09d039 Peng Tao 2013-05-02 133
48a2a12af John L. Hammond 2016-10-02 134 osc_pack_req_body(req, oa);
d7e09d039 Peng Tao 2013-05-02 135
d7e09d039 Peng Tao 2013-05-02 136 ptlrpc_request_set_replen(req);
d7e09d039 Peng Tao 2013-05-02 137
d7e09d039 Peng Tao 2013-05-02 138 rc = ptlrpc_queue_wait(req);
d7e09d039 Peng Tao 2013-05-02 139 if (rc)
26c4ea46a Tina Johnson 2014-09-21 140 goto out;
d7e09d039 Peng Tao 2013-05-02 141
d7e09d039 Peng Tao 2013-05-02 142 body = req_capsule_server_get(&req->rq_pill, &RMF_OST_BODY);
7f1ae4c06 Oleg Drokin 2016-02-16 143 if (!body) {
26c4ea46a Tina Johnson 2014-09-21 144 rc = -EPROTO;
26c4ea46a Tina Johnson 2014-09-21 145 goto out;
26c4ea46a Tina Johnson 2014-09-21 146 }
d7e09d039 Peng Tao 2013-05-02 147
d7e09d039 Peng Tao 2013-05-02 148 CDEBUG(D_INODE, "mode: %o\n", body->oa.o_mode);
48a2a12af John L. Hammond 2016-10-02 @149 lustre_get_wire_obdo(&req->rq_import->imp_connect_data, oa,
3b2f75fd5 wang di 2013-06-03 150 &body->oa);
d7e09d039 Peng Tao 2013-05-02 151
48a2a12af John L. Hammond 2016-10-02 152 oa->o_blksize = cli_brw_size(exp->exp_obd);
:::::: The code at line 114 was first introduced by commit
:::::: 48a2a12afc54aa1ebfdb1e821fc3dc7893816986 staging: lustre: osc: remove remaining bits for capa support
:::::: TO: John L. Hammond <john.hammond@intel.com>
:::::: CC: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 43289 bytes --]
next prev parent reply other threads:[~2016-12-03 13:29 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-12-02 19:40 [PATCH 0/6] Fix ups to make lustre_idl.h a proper UAPI header James Simmons
2016-12-02 19:40 ` [PATCH 1/6] staging: lustre: headers: move swab functions to new header files James Simmons
2016-12-02 19:40 ` [PATCH 2/6] staging: lustre: headers: sort headers affected by swab move James Simmons
2016-12-06 9:32 ` Greg Kroah-Hartman
2016-12-02 19:40 ` [PATCH 3/6] staging: lustre: obdclass: Create a header for obdo related functions James Simmons
2016-12-03 13:27 ` kbuild test robot [this message]
2016-12-05 20:50 ` Dan Carpenter
2016-12-05 21:55 ` [lustre-devel] " Dilger, Andreas
2016-12-05 22:06 ` Dan Carpenter
2016-12-07 19:18 ` James Simmons
2016-12-06 9:36 ` Greg Kroah-Hartman
2016-12-07 19:13 ` James Simmons
2016-12-02 19:40 ` [PATCH 4/6] staging: lustre: headers: sort headers affected by obdo move James Simmons
2016-12-02 19:40 ` [PATCH 5/6] staging: lustre: headers: Move functions out of lustre_idl.h James Simmons
2016-12-05 20:54 ` Dan Carpenter
2016-12-02 19:40 ` [PATCH 6/6] staging: lustre: headers: use proper byteorder functions in lustre_idl.h James Simmons
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=201612032108.E93gbaZn%fengguang.wu@intel.com \
--to=lkp@intel.com \
--cc=andreas.dilger@intel.com \
--cc=bevans@cray.com \
--cc=devel@driverdev.osuosl.org \
--cc=gregkh@linuxfoundation.org \
--cc=jsimmons@infradead.org \
--cc=kbuild-all@01.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lustre-devel@lists.lustre.org \
--cc=oleg.drokin@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®