Date: Sat, 11 Dec 2010 10:14:08 +0000 (UTC) From: Andriy Gapon <avg@FreeBSD.org> To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: svn commit: r216373 - head/sys/dev/cxgb/ulp/tom Message-ID: <201012111014.oBBAE8ts004239@svn.freebsd.org>
next in thread | raw e-mail | index | archive | help
Author: avg Date: Sat Dec 11 10:14:08 2010 New Revision: 216373 URL: http://svn.freebsd.org/changeset/base/216373 Log: fix incorrect use of atomic_set_xxx in cxgb There is no need to use an atomic operation at structure initialization time. Note that the file changed is not connected to the build at this time. Reviewed by: jhb (general issue) Approved by: np MFC after: 2 weeks Modified: head/sys/dev/cxgb/ulp/tom/cxgb_tom.c Modified: head/sys/dev/cxgb/ulp/tom/cxgb_tom.c ============================================================================== --- head/sys/dev/cxgb/ulp/tom/cxgb_tom.c Sat Dec 11 09:38:12 2010 (r216372) +++ head/sys/dev/cxgb/ulp/tom/cxgb_tom.c Sat Dec 11 10:14:08 2010 (r216373) @@ -269,7 +269,7 @@ init_tid_tabs(struct tid_info *t, unsign t->atid_base = atid_base; t->afree = NULL; t->stids_in_use = t->atids_in_use = 0; - atomic_set_int(&t->tids_in_use, 0); + t->tids_in_use = 0; mtx_init(&t->stid_lock, "stid", NULL, MTX_DUPOK|MTX_DEF); mtx_init(&t->atid_lock, "atid", NULL, MTX_DUPOK|MTX_DEF);
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201012111014.oBBAE8ts004239>