From: kernel test robot <lkp@intel.com>
To: Chuck Lever <chuck.lever@oracle.com>, linux-nfs@vger.kernel.org
Cc: kbuild-all@lists.01.org
Subject: Re: [PATCH v1 02/11] NFSD: Shrink size of struct nfsd4_copy
Date: Sun, 24 Jul 2022 14:30:41 +0800 [thread overview]
Message-ID: <202207241457.kBfLhxI3-lkp@intel.com> (raw)
In-Reply-To: <165852114903.11403.8544275041086740489.stgit@manet.1015granger.net>
Hi Chuck,
I love your patch! Yet something to improve:
[auto build test ERROR on linus/master]
[also build test ERROR on v5.19-rc7 next-20220722]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Chuck-Lever/Put-struct-nfsd4_copy-on-a-diet/20220723-042113
base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 70664fc10c0d722ec79d746d8ac1db8546c94114
config: sh-allmodconfig (https://download.01.org/0day-ci/archive/20220724/202207241457.kBfLhxI3-lkp@intel.com/config)
compiler: sh4-linux-gcc (GCC) 12.1.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/intel-lab-lkp/linux/commit/1968aac547167ef15b3634429e0cfac0c5b38419
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Chuck-Lever/Put-struct-nfsd4_copy-on-a-diet/20220723-042113
git checkout 1968aac547167ef15b3634429e0cfac0c5b38419
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=sh SHELL=/bin/bash fs/nfsd/
If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
fs/nfsd/nfs4proc.c: In function 'nfsd4_setup_inter_ssc':
>> fs/nfsd/nfs4proc.c:1553:41: error: passing argument 1 of 'nfsd4_interssc_connect' from incompatible pointer type [-Werror=incompatible-pointer-types]
1553 | status = nfsd4_interssc_connect(©->cp_src, rqstp, mount);
| ^~~~~~~~~~~~~
| |
| struct nl4_server **
fs/nfsd/nfs4proc.c:1428:43: note: expected 'struct nl4_server *' but argument is of type 'struct nl4_server **'
1428 | nfsd4_interssc_connect(struct nl4_server *nss, struct svc_rqst *rqstp,
| ~~~~~~~~~~~~~~~~~~~^~~
cc1: some warnings being treated as errors
vim +/nfsd4_interssc_connect +1553 fs/nfsd/nfs4proc.c
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1527
f2453978a4f2dd Chuck Lever 2020-04-06 1528 /*
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1529 * Verify COPY destination stateid.
f2453978a4f2dd Chuck Lever 2020-04-06 1530 *
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1531 * Connect to the source server with NFSv4.1.
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1532 * Create the source struct file for nfsd_copy_range.
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1533 * Called with COPY cstate:
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1534 * SAVED_FH: source filehandle
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1535 * CURRENT_FH: destination filehandle
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1536 */
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1537 static __be32
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1538 nfsd4_setup_inter_ssc(struct svc_rqst *rqstp,
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1539 struct nfsd4_compound_state *cstate,
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1540 struct nfsd4_copy *copy, struct vfsmount **mount)
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1541 {
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1542 struct svc_fh *s_fh = NULL;
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1543 stateid_t *s_stid = ©->cp_src_stateid;
b8290ca250fb77 Olga Kornievskaia 2019-12-04 1544 __be32 status = nfserr_inval;
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1545
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1546 /* Verify the destination stateid and set dst struct file*/
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1547 status = nfs4_preprocess_stateid_op(rqstp, cstate, &cstate->current_fh,
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1548 ©->cp_dst_stateid,
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1549 WR_STATE, ©->nf_dst, NULL);
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1550 if (status)
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1551 goto out;
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1552
ce0887ac96d35c Olga Kornievskaia 2019-10-09 @1553 status = nfsd4_interssc_connect(©->cp_src, rqstp, mount);
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1554 if (status)
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1555 goto out;
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1556
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1557 s_fh = &cstate->save_fh;
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1558
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1559 copy->c_fh.size = s_fh->fh_handle.fh_size;
d8b26071e65e80 NeilBrown 2021-09-02 1560 memcpy(copy->c_fh.data, &s_fh->fh_handle.fh_raw, copy->c_fh.size);
3f9544ca62bc13 Olga Kornievskaia 2019-12-04 1561 copy->stateid.seqid = cpu_to_be32(s_stid->si_generation);
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1562 memcpy(copy->stateid.other, (void *)&s_stid->si_opaque,
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1563 sizeof(stateid_opaque_t));
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1564
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1565 status = 0;
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1566 out:
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1567 return status;
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1568 }
ce0887ac96d35c Olga Kornievskaia 2019-10-09 1569
--
0-DAY CI Kernel Test Service
https://01.org/lkp
next prev parent reply other threads:[~2022-07-24 6:31 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-07-22 20:18 [PATCH v1 00/11] Put struct nfsd4_copy on a diet Chuck Lever
2022-07-22 20:19 ` [PATCH v1 01/11] NFSD: Shrink size of struct nfsd4_copy_notify Chuck Lever
2022-07-25 14:19 ` Olga Kornievskaia
2022-07-25 14:36 ` Chuck Lever III
2022-07-22 20:19 ` [PATCH v1 02/11] NFSD: Shrink size of struct nfsd4_copy Chuck Lever
2022-07-24 6:30 ` kernel test robot [this message]
2022-07-22 20:19 ` [PATCH v1 03/11] NFSD: Reorder the fields in struct nfsd4_op Chuck Lever
2022-07-22 20:19 ` [PATCH v1 04/11] NFSD: Make nfs4_put_copy() static Chuck Lever
2022-07-22 20:19 ` [PATCH v1 05/11] NFSD: Make boolean fields in struct nfsd4_copy into atomic bit flags Chuck Lever
2022-07-22 20:19 ` [PATCH v1 06/11] NFSD: Refactor nfsd4_cleanup_inter_ssc() (1/2) Chuck Lever
2022-07-22 20:19 ` [PATCH v1 07/11] NFSD: Refactor nfsd4_cleanup_inter_ssc() (2/2) Chuck Lever
2022-07-22 20:19 ` [PATCH v1 08/11] NFSD: Refactor nfsd4_do_copy() Chuck Lever
2022-07-22 20:19 ` [PATCH v1 09/11] NFSD: Remove kmalloc from nfsd4_do_async_copy() Chuck Lever
2022-07-22 20:19 ` [PATCH v1 10/11] NFSD: Add nfsd4_send_cb_offload() Chuck Lever
2022-07-22 20:20 ` [PATCH v1 11/11] NFSD: Move copy offload callback arguments into a separate structure Chuck Lever
2022-07-26 19:45 ` [PATCH v1 00/11] Put struct nfsd4_copy on a diet Olga Kornievskaia
2022-07-27 16:18 ` Olga Kornievskaia
2022-07-27 16:38 ` dai.ngo
2022-07-27 17:15 ` Chuck Lever III
2022-07-27 17:52 ` Olga Kornievskaia
2022-07-27 18:04 ` Chuck Lever III
2022-07-27 18:21 ` Olga Kornievskaia
2022-07-27 18:48 ` Olga Kornievskaia
2022-07-27 19:23 ` dai.ngo
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=202207241457.kBfLhxI3-lkp@intel.com \
--to=lkp@intel.com \
--cc=chuck.lever@oracle.com \
--cc=kbuild-all@lists.01.org \
--cc=linux-nfs@vger.kernel.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.