From: Xin Long <lucien.xin@gmail.com>
To: kernel test robot <lkp@intel.com>
Cc: network dev <netdev@vger.kernel.org>, davem <davem@davemloft.net>,
Jakub Kicinski <kuba@kernel.org>,
Marcelo Ricardo Leitner <marcelo.leitner@gmail.com>,
"linux-sctp @ vger . kernel . org" <linux-sctp@vger.kernel.org>,
kbuild-all@lists.01.org
Subject: Re: [PATCH net-next 06/14] sctp: do the basic send and recv for PLPMTUD probe
Date: Mon, 21 Jun 2021 21:13:59 -0400 [thread overview]
Message-ID: <CADvbK_dCgaqrr=pxZGfBmm=3m31+eKgcfkU7AowFbnZAcC92bQ@mail.gmail.com> (raw)
In-Reply-To: <202106211151.QDS54KHu-lkp@intel.com>
This is a "set but not used" warning, I can fix it after.
Thanks.
On Sun, Jun 20, 2021 at 11:50 PM kernel test robot <lkp@intel.com> wrote:
>
> Hi Xin,
>
> Thank you for the patch! Perhaps something to improve:
>
> [auto build test WARNING on net-next/master]
>
> url: https://github.com/0day-ci/linux/commits/Xin-Long/sctp-implement-RFC8899-Packetization-Layer-Path-MTU-Discovery-for-SCTP-transport/20210621-094007
> base: https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git adc2e56ebe6377f5c032d96aee0feac30a640453
> config: i386-randconfig-r023-20210620 (attached as .config)
> compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
> reproduce (this is a W=1 build):
> # https://github.com/0day-ci/linux/commit/fcac1d6488c8bc7cb69af9e8051686a674d94fc3
> git remote add linux-review https://github.com/0day-ci/linux
> git fetch --no-tags linux-review Xin-Long/sctp-implement-RFC8899-Packetization-Layer-Path-MTU-Discovery-for-SCTP-transport/20210621-094007
> git checkout fcac1d6488c8bc7cb69af9e8051686a674d94fc3
> # save the attached .config to linux build tree
> make W=1 ARCH=i386
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
>
> All warnings (new ones prefixed by >>):
>
> >> net/sctp/output.c:215:16: warning: no previous prototype for 'sctp_packet_bundle_pad' [-Wmissing-prototypes]
> 215 | enum sctp_xmit sctp_packet_bundle_pad(struct sctp_packet *pkt, struct sctp_chunk *chunk)
> | ^~~~~~~~~~~~~~~~~~~~~~
> net/sctp/output.c: In function 'sctp_packet_bundle_pad':
> >> net/sctp/output.c:219:20: warning: variable 'sp' set but not used [-Wunused-but-set-variable]
> 219 | struct sctp_sock *sp;
> | ^~
>
>
> vim +/sctp_packet_bundle_pad +215 net/sctp/output.c
>
> 213
> 214 /* Try to bundle a pad chunk into a packet with a heartbeat chunk for PLPMTUTD probe */
> > 215 enum sctp_xmit sctp_packet_bundle_pad(struct sctp_packet *pkt, struct sctp_chunk *chunk)
> 216 {
> 217 struct sctp_transport *t = pkt->transport;
> 218 struct sctp_chunk *pad;
> > 219 struct sctp_sock *sp;
> 220 int overhead = 0;
> 221
> 222 if (!chunk->pmtu_probe)
> 223 return SCTP_XMIT_OK;
> 224
> 225 sp = sctp_sk(t->asoc->base.sk);
> 226
> 227 /* calculate the Padding Data size for the pad chunk */
> 228 overhead += sizeof(struct sctphdr) + sizeof(struct sctp_chunkhdr);
> 229 overhead += sizeof(struct sctp_sender_hb_info) + sizeof(struct sctp_pad_chunk);
> 230 pad = sctp_make_pad(t->asoc, t->pl.probe_size - overhead);
> 231 if (!pad)
> 232 return SCTP_XMIT_DELAY;
> 233
> 234 list_add_tail(&pad->list, &pkt->chunk_list);
> 235 pkt->size += SCTP_PAD4(ntohs(pad->chunk_hdr->length));
> 236 chunk->transport = t;
> 237
> 238 return SCTP_XMIT_OK;
> 239 }
> 240
>
> ---
> 0-DAY CI Kernel Test Service, Intel Corporation
> https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
next prev parent reply other threads:[~2021-06-22 1:14 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-06-21 1:38 [PATCH net-next 00/14] sctp: implement RFC8899: Packetization Layer Path MTU Discovery for SCTP transport Xin Long
2021-06-21 1:38 ` [PATCH net-next 01/14] sctp: add pad chunk and its make function and event table Xin Long
2021-06-21 1:38 ` [PATCH net-next 02/14] sctp: add probe_interval in sysctl and sock/asoc/transport Xin Long
2021-06-21 1:38 ` [PATCH net-next 03/14] sctp: add SCTP_PLPMTUD_PROBE_INTERVAL sockopt for sock/asoc/transport Xin Long
2021-06-21 1:38 ` [PATCH net-next 04/14] sctp: add the constants/variables and states and some APIs for transport Xin Long
2021-06-21 1:38 ` [PATCH net-next 05/14] sctp: add the probe timer in transport for PLPMTUD Xin Long
2021-06-21 1:38 ` [PATCH net-next 06/14] sctp: do the basic send and recv for PLPMTUD probe Xin Long
2021-06-21 3:49 ` kernel test robot
2021-06-22 1:13 ` Xin Long [this message]
2021-06-22 17:02 ` David Miller
2021-06-21 1:38 ` [PATCH net-next 07/14] sctp: do state transition when PROBE_COUNT == MAX_PROBES on HB send path Xin Long
2021-06-21 1:38 ` [PATCH net-next 08/14] sctp: do state transition when a probe succeeds on HB ACK recv path Xin Long
2021-06-21 1:38 ` [PATCH net-next 09/14] sctp: do state transition when receiving an icmp TOOBIG packet Xin Long
2021-06-21 1:38 ` [PATCH net-next 10/14] sctp: enable PLPMTUD when the transport is ready Xin Long
2021-06-21 1:38 ` [PATCH net-next 11/14] sctp: remove the unessessary hold for idev in sctp_v6_err Xin Long
2021-06-21 1:38 ` [PATCH net-next 12/14] sctp: extract sctp_v6_err_handle function from sctp_v6_err Xin Long
2021-06-21 1:38 ` [PATCH net-next 13/14] sctp: extract sctp_v4_err_handle function from sctp_v4_err Xin Long
2021-06-21 1:38 ` [PATCH net-next 14/14] sctp: process sctp over udp icmp err on sctp side Xin Long
2021-06-22 1:30 ` [PATCH net-next 00/14] sctp: implement RFC8899: Packetization Layer Path MTU Discovery for SCTP transport Marcelo Ricardo Leitner
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='CADvbK_dCgaqrr=pxZGfBmm=3m31+eKgcfkU7AowFbnZAcC92bQ@mail.gmail.com' \
--to=lucien.xin@gmail.com \
--cc=davem@davemloft.net \
--cc=kbuild-all@lists.01.org \
--cc=kuba@kernel.org \
--cc=linux-sctp@vger.kernel.org \
--cc=lkp@intel.com \
--cc=marcelo.leitner@gmail.com \
--cc=netdev@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).