sys/vm/uma_core.c:zone_import() Wrap stripe variable in #ifdef NUMA
ClosedPublic
Actions

Authored by sbruno on Jul 3 2018, 4:12 PM.

Tags

None

Referenced Files

	F153054832: D16113.id.diff
	Sat, Apr 18, 9:16 PM

	Unknown Object (File)
	Sun, Apr 12, 11:59 PM

	Unknown Object (File)
	Fri, Apr 10, 6:54 PM

	Unknown Object (File)
	Mon, Apr 6, 4:05 AM

	Unknown Object (File)
	Sat, Apr 4, 9:32 PM

	Unknown Object (File)
	Thu, Mar 26, 12:28 AM

	Unknown Object (File)
	Thu, Mar 26, 12:28 AM

	Unknown Object (File)
	Wed, Mar 25, 9:44 AM

View All 88 Files

Subscribers

imp

Details

Reviewers

jeff
kib
mjg

Commits

rS336063: Wrap the declaration and assignment of "stripe" with #ifdef NUMA declarations

Summary

stripe seems to be unused on targets that don't support NUMA.
Wrap this variable completely inside of #ifdef NUMA. Found with gcc.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

sbruno created this revision.Jul 3 2018, 4:12 PM

Herald added a subscriber: imp. · View Herald TranscriptJul 3 2018, 4:12 PM

Harbormaster completed remote builds in B17816: Diff 44809.Jul 3 2018, 4:12 PM

/home/sbruno/bsd/fbsd_head/sys/vm/uma_core.c: In function 'zone_import':
/home/sbruno/bsd/fbsd_head/sys/vm/uma_core.c:2774:6: warning: variable 'stripe' set but not used [-Wunused-but-set-variable]
  int stripe;

This revision was not accepted when it landed; it landed in state Needs Review.Jul 7 2018, 1:38 PM

Closed by commit rS336063: Wrap the declaration and assignment of "stripe" with #ifdef NUMA declarations (authored by sbruno). · Explain Why

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

vm/

uma_core.c

4 lines

Diff 44993

View Options

sys/vm/uma_core.c:zone_import() Wrap stripe variable in #ifdef NUMAClosedPublicActions