Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 25 Dec 2018 07:07:34 +0000 (UTC)
From:      Andrew Rybchenko <arybchik@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-11@freebsd.org
Subject:   svn commit: r342430 - stable/11/sys/dev/sfxge/common
Message-ID:  <201812250707.wBP77Y8q035136@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: arybchik
Date: Tue Dec 25 07:07:34 2018
New Revision: 342430
URL: https://svnweb.freebsd.org/changeset/base/342430

Log:
  MFC r340888
  
  sfxge(4): fix warnings from VS2015 C compiler (C4100)
  
  Fix multiple level 4 warnings
  "C4100: 'xxx': unreferenced formal parameter"
  no functional changes.
  
  The _NOTE(ARGUNUSED(xxx)) annotations are being exposed to the Visual
  Studio 2015 C compiler with the following:
  
    #define _NOTE_ARGUNUSED(...) UNREFERENCED_PARAMETER((__VA_ARGS__));
    #define _NOTE(_annotation)   _NOTE_ ## _annotation
  
  Submitted by:   Andrew Lee <alee at solarflare.com>
  Sponsored by:   Solarflare Communications, Inc.
  Differential Revision:  https://reviews.freebsd.org/D18122

Modified:
  stable/11/sys/dev/sfxge/common/ef10_nvram.c
  stable/11/sys/dev/sfxge/common/ef10_rx.c
  stable/11/sys/dev/sfxge/common/ef10_tx.c
  stable/11/sys/dev/sfxge/common/efx_ev.c
  stable/11/sys/dev/sfxge/common/efx_lic.c
  stable/11/sys/dev/sfxge/common/efx_mcdi.c
Directory Properties:
  stable/11/   (props changed)

Modified: stable/11/sys/dev/sfxge/common/ef10_nvram.c
==============================================================================
--- stable/11/sys/dev/sfxge/common/ef10_nvram.c	Tue Dec 25 07:03:52 2018	(r342429)
+++ stable/11/sys/dev/sfxge/common/ef10_nvram.c	Tue Dec 25 07:07:34 2018	(r342430)
@@ -675,6 +675,7 @@ ef10_nvram_buffer_validate(
 	int pos;
 	efx_rc_t rc;
 
+	_NOTE(ARGUNUSED(enp, partn))
 	EFX_STATIC_ASSERT(sizeof (*header) <= EF10_NVRAM_CHUNK);
 
 	if ((partn_data == NULL) || (partn_size == 0)) {
@@ -1284,6 +1285,8 @@ ef10_nvram_buf_read_tlv(
 	size_t length;
 	caddr_t value;
 	efx_rc_t rc;
+
+	_NOTE(ARGUNUSED(enp))
 
 	if ((seg_data == NULL) || (max_seg_size == 0)) {
 		rc = EINVAL;

Modified: stable/11/sys/dev/sfxge/common/ef10_rx.c
==============================================================================
--- stable/11/sys/dev/sfxge/common/ef10_rx.c	Tue Dec 25 07:03:52 2018	(r342429)
+++ stable/11/sys/dev/sfxge/common/ef10_rx.c	Tue Dec 25 07:07:34 2018	(r342430)
@@ -657,6 +657,8 @@ ef10_rx_qpost(
 	unsigned int offset;
 	unsigned int id;
 
+	_NOTE(ARGUNUSED(completed))
+
 	/* The client driver must not overfill the queue */
 	EFSYS_ASSERT3U(added - completed + n, <=,
 	    EFX_RXQ_LIMIT(erp->er_mask + 1));

Modified: stable/11/sys/dev/sfxge/common/ef10_tx.c
==============================================================================
--- stable/11/sys/dev/sfxge/common/ef10_tx.c	Tue Dec 25 07:03:52 2018	(r342429)
+++ stable/11/sys/dev/sfxge/common/ef10_tx.c	Tue Dec 25 07:07:34 2018	(r342430)
@@ -609,6 +609,8 @@ ef10_tx_qdesc_dma_create(
 	__in	boolean_t eop,
 	__out	efx_desc_t *edp)
 {
+	_NOTE(ARGUNUSED(etp))
+
 	/* No limitations on boundary crossing */
 	EFSYS_ASSERT(size <= etp->et_enp->en_nic_cfg.enc_tx_dma_desc_size_max);
 
@@ -632,6 +634,8 @@ ef10_tx_qdesc_tso_create(
 	__in	uint8_t  tcp_flags,
 	__out	efx_desc_t *edp)
 {
+	_NOTE(ARGUNUSED(etp))
+
 	EFSYS_PROBE4(tx_desc_tso_create, unsigned int, etp->et_index,
 		    uint16_t, ipv4_id, uint32_t, tcp_seq,
 		    uint8_t, tcp_flags);
@@ -654,6 +658,8 @@ ef10_tx_qdesc_tso2_create(
 	__out_ecount(count)	efx_desc_t *edp,
 	__in			int count)
 {
+	_NOTE(ARGUNUSED(etp, count))
+
 	EFSYS_PROBE4(tx_desc_tso2_create, unsigned int, etp->et_index,
 		    uint16_t, ipv4_id, uint32_t, tcp_seq,
 		    uint16_t, tcp_mss);
@@ -683,6 +689,8 @@ ef10_tx_qdesc_vlantci_create(
 	__in	uint16_t  tci,
 	__out	efx_desc_t *edp)
 {
+	_NOTE(ARGUNUSED(etp))
+
 	EFSYS_PROBE2(tx_desc_vlantci_create, unsigned int, etp->et_index,
 		    uint16_t, tci);
 

Modified: stable/11/sys/dev/sfxge/common/efx_ev.c
==============================================================================
--- stable/11/sys/dev/sfxge/common/efx_ev.c	Tue Dec 25 07:03:52 2018	(r342429)
+++ stable/11/sys/dev/sfxge/common/efx_ev.c	Tue Dec 25 07:07:34 2018	(r342430)
@@ -1419,6 +1419,8 @@ efx_ev_qstat_name(
 	__in	efx_nic_t *enp,
 	__in	unsigned int id)
 {
+	_NOTE(ARGUNUSED(enp))
+
 	EFSYS_ASSERT3U(enp->en_magic, ==, EFX_NIC_MAGIC);
 	EFSYS_ASSERT3U(id, <, EV_NQSTATS);
 

Modified: stable/11/sys/dev/sfxge/common/efx_lic.c
==============================================================================
--- stable/11/sys/dev/sfxge/common/efx_lic.c	Tue Dec 25 07:03:52 2018	(r342429)
+++ stable/11/sys/dev/sfxge/common/efx_lic.c	Tue Dec 25 07:07:34 2018	(r342430)
@@ -587,7 +587,7 @@ efx_lic_v1v2_read_key(
 {
 	efx_rc_t rc;
 
-	_NOTE(ARGUNUSED(enp))
+	_NOTE(ARGUNUSED(enp, buffer_size))
 	EFSYS_ASSERT(length <= (EFX_LICENSE_V1V2_PAYLOAD_LENGTH_MAX +
 	    EFX_LICENSE_V1V2_HEADER_LENGTH));
 
@@ -659,7 +659,7 @@ efx_lic_v1v2_delete_key(
 	uint32_t move_start = offset + length;
 	uint32_t move_length = end - move_start;
 
-	_NOTE(ARGUNUSED(enp))
+	_NOTE(ARGUNUSED(enp, buffer_size))
 	EFSYS_ASSERT(end <= buffer_size);
 
 	/* Shift everything after the key down */
@@ -678,7 +678,7 @@ efx_lic_v1v2_create_partition(
 	__in			size_t buffer_size
 	)
 {
-	_NOTE(ARGUNUSED(enp))
+	_NOTE(ARGUNUSED(enp, buffer_size))
 	EFSYS_ASSERT(EFX_LICENSE_V1V2_HEADER_LENGTH <= buffer_size);
 
 	/* Write terminator */

Modified: stable/11/sys/dev/sfxge/common/efx_mcdi.c
==============================================================================
--- stable/11/sys/dev/sfxge/common/efx_mcdi.c	Tue Dec 25 07:03:52 2018	(r342429)
+++ stable/11/sys/dev/sfxge/common/efx_mcdi.c	Tue Dec 25 07:07:34 2018	(r342430)
@@ -812,6 +812,8 @@ efx_mcdi_get_proxy_handle(
 {
 	efx_rc_t rc;
 
+	_NOTE(ARGUNUSED(enp))
+
 	/*
 	 * Return proxy handle from MCDI request that returned with error
 	 * MC_MCD_ERR_PROXY_PENDING. This handle is used to wait for a matching



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201812250707.wBP77Y8q035136>