svn commit: r294017 - stable/10/sys/dev/sfxge/common

[Andrew Rybchenko]

Author: arybchik
Date: Thu Jan 14 16:28:21 2016
New Revision: 294017
URL: https://svnweb.freebsd.org/changeset/base/294017

Log:
  MFC r293817
  
  sfxge: cleanup: prefast fixes in common code
  
  Submitted by:   Andrew Lee <alee at solarflare.com>
  Sponsored by:   Solarflare Communications, Inc.

Modified:
  stable/10/sys/dev/sfxge/common/efx.h
  stable/10/sys/dev/sfxge/common/efx_filter.c
  stable/10/sys/dev/sfxge/common/efx_impl.h
  stable/10/sys/dev/sfxge/common/hunt_impl.h
  stable/10/sys/dev/sfxge/common/hunt_mcdi.c
  stable/10/sys/dev/sfxge/common/hunt_nvram.c
  stable/10/sys/dev/sfxge/common/siena_impl.h
  stable/10/sys/dev/sfxge/common/siena_mcdi.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/sys/dev/sfxge/common/efx.h
==============================================================================
--- stable/10/sys/dev/sfxge/common/efx.h	Thu Jan 14 16:27:43 2016	(r294016)
+++ stable/10/sys/dev/sfxge/common/efx.h	Thu Jan 14 16:28:21 2016	(r294017)
@@ -2235,14 +2235,14 @@ efx_filter_supported_filters(
 
 extern			void
 efx_filter_spec_init_rx(
-	__inout		efx_filter_spec_t *spec,
+	__out		efx_filter_spec_t *spec,
 	__in		efx_filter_priority_t priority,
 	__in		efx_filter_flag_t flags,
 	__in		efx_rxq_t *erp);
 
 extern			void
 efx_filter_spec_init_tx(
-	__inout		efx_filter_spec_t *spec,
+	__out		efx_filter_spec_t *spec,
 	__in		efx_txq_t *etp);
 
 extern	__checkReturn	efx_rc_t

Modified: stable/10/sys/dev/sfxge/common/efx_filter.c
==============================================================================
--- stable/10/sys/dev/sfxge/common/efx_filter.c	Thu Jan 14 16:27:43 2016	(r294016)
+++ stable/10/sys/dev/sfxge/common/efx_filter.c	Thu Jan 14 16:28:21 2016	(r294017)
@@ -292,7 +292,7 @@ fail1:
 
 		void
 efx_filter_spec_init_rx(
-	__inout		efx_filter_spec_t *spec,
+	__out		efx_filter_spec_t *spec,
 	__in		efx_filter_priority_t priority,
 	__in		efx_filter_flag_t flags,
 	__in		efx_rxq_t *erp)
@@ -311,7 +311,7 @@ efx_filter_spec_init_rx(
 
 		void
 efx_filter_spec_init_tx(
-	__inout		efx_filter_spec_t *spec,
+	__out		efx_filter_spec_t *spec,
 	__in		efx_txq_t *etp)
 {
 	EFSYS_ASSERT3P(spec, !=, NULL);

Modified: stable/10/sys/dev/sfxge/common/efx_impl.h
==============================================================================
--- stable/10/sys/dev/sfxge/common/efx_impl.h	Thu Jan 14 16:27:43 2016	(r294016)
+++ stable/10/sys/dev/sfxge/common/efx_impl.h	Thu Jan 14 16:28:21 2016	(r294017)
@@ -1120,7 +1120,7 @@ efx_vpd_hunk_next(
 	__in				size_t size,
 	__out				efx_vpd_tag_t *tagp,
 	__out				efx_vpd_keyword_t *keyword,
-	__out_bcount_opt(*paylenp)	unsigned int *payloadp,
+	__out_opt			unsigned int *payloadp,
 	__out_opt			uint8_t *paylenp,
 	__inout				unsigned int *contp);
 

Modified: stable/10/sys/dev/sfxge/common/hunt_impl.h
==============================================================================
--- stable/10/sys/dev/sfxge/common/hunt_impl.h	Thu Jan 14 16:27:43 2016	(r294016)
+++ stable/10/sys/dev/sfxge/common/hunt_impl.h	Thu Jan 14 16:28:21 2016	(r294017)
@@ -289,10 +289,10 @@ ef10_mcdi_poll_response(
 
 extern			void
 ef10_mcdi_read_response(
-	__in		efx_nic_t *enp,
-	__out		void *bufferp,
-	__in		size_t offset,
-	__in		size_t length);
+	__in			efx_nic_t *enp,
+	__out_bcount(length)	void *bufferp,
+	__in			size_t offset,
+	__in			size_t length);
 
 extern			void
 ef10_mcdi_request_copyout(

Modified: stable/10/sys/dev/sfxge/common/hunt_mcdi.c
==============================================================================
--- stable/10/sys/dev/sfxge/common/hunt_mcdi.c	Thu Jan 14 16:27:43 2016	(r294016)
+++ stable/10/sys/dev/sfxge/common/hunt_mcdi.c	Thu Jan 14 16:28:21 2016	(r294017)
@@ -307,10 +307,10 @@ ef10_mcdi_poll_response(
 
 			void
 ef10_mcdi_read_response(
-	__in		efx_nic_t *enp,
-	__out		void *bufferp,
-	__in		size_t offset,
-	__in		size_t length)
+	__in			efx_nic_t *enp,
+	__out_bcount(length)	void *bufferp,
+	__in			size_t offset,
+	__in			size_t length)
 {
 	const efx_mcdi_transport_t *emtp = enp->en_mcdi.em_emtp;
 	efsys_mem_t *esmp = emtp->emt_dma_mem;

Modified: stable/10/sys/dev/sfxge/common/hunt_nvram.c
==============================================================================
--- stable/10/sys/dev/sfxge/common/hunt_nvram.c	Thu Jan 14 16:27:43 2016	(r294016)
+++ stable/10/sys/dev/sfxge/common/hunt_nvram.c	Thu Jan 14 16:28:21 2016	(r294017)
@@ -240,7 +240,7 @@ fail1:
 
 static				efx_rc_t
 tlv_init_cursor(
-	__in	tlv_cursor_t	*cursor,
+	__out	tlv_cursor_t	*cursor,
 	__in	uint32_t	*block,
 	__in	uint32_t	*limit)
 {
@@ -255,7 +255,7 @@ tlv_init_cursor(
 
 static				efx_rc_t
 tlv_init_cursor_from_size(
-	__in	tlv_cursor_t	*cursor,
+	__out	tlv_cursor_t	*cursor,
 	__in	uint8_t	*block,
 	__in	size_t		size)
 {

Modified: stable/10/sys/dev/sfxge/common/siena_impl.h
==============================================================================
--- stable/10/sys/dev/sfxge/common/siena_impl.h	Thu Jan 14 16:27:43 2016	(r294016)
+++ stable/10/sys/dev/sfxge/common/siena_impl.h	Thu Jan 14 16:28:21 2016	(r294017)
@@ -127,10 +127,10 @@ siena_mcdi_poll_response(
 
 extern			void
 siena_mcdi_read_response(
-	__in		efx_nic_t *enp,
-	__out		void *bufferp,
-	__in		size_t offset,
-	__in		size_t length);
+	__in			efx_nic_t *enp,
+	__out_bcount(length)	void *bufferp,
+	__in			size_t offset,
+	__in			size_t length);
 
 extern			void
 siena_mcdi_request_copyout(

Modified: stable/10/sys/dev/sfxge/common/siena_mcdi.c
==============================================================================
--- stable/10/sys/dev/sfxge/common/siena_mcdi.c	Thu Jan 14 16:27:43 2016	(r294016)
+++ stable/10/sys/dev/sfxge/common/siena_mcdi.c	Thu Jan 14 16:28:21 2016	(r294017)
@@ -216,10 +216,10 @@ siena_mcdi_poll_response(
 
 			void
 siena_mcdi_read_response(
-	__in		efx_nic_t *enp,
-	__out		void *bufferp,
-	__in		size_t offset,
-	__in		size_t length)
+	__in			efx_nic_t *enp,
+	__out_bcount(length)	void *bufferp,
+	__in			size_t offset,
+	__in			size_t length)
 {
 	efx_mcdi_iface_t *emip = &(enp->en_mcdi.em_emip);
 	unsigned int pdur;