Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 14 Dec 2003 19:38:53 +0000 (GMT)
From:      Gavin Atkinson <gavin+freebsdc@ury.york.ac.uk>
To:        Don Lewis <truckman@freebsd.org>
Cc:        current@freebsd.org
Subject:   Re: cvs commit: src/sys/kern subr_mbpool.c subr_mbuf.c
Message-ID:  <20031214180746.Y23299@ury.york.ac.uk>
In-Reply-To: <200312072153.hB7LrfTi004032@repoman.freebsd.org>
References:  <200312072153.hB7LrfTi004032@repoman.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Sun, 7 Dec 2003, Don Lewis wrote:
> truckman    2003/12/07 13:53:41 PST
>
>   Modified files:
>     sys/kern             subr_mbpool.c subr_mbuf.c
>   Log:
>   Pass MTX_DEF as the last argument to mtx_init() instead of 0.  This
>   is not a functional change.  The code happened to work properly only
>   because MTX_DEF is defined as 0.

Can I also offer the following patches which do the same to other parts of
the tree? I believe these patches close the last places where mtx_init is
passed incorrect arguments. They have survived a kernel build.

http://www.devrandom.co.uk/freebsd/ata-mtx.diff
http://www.devrandom.co.uk/freebsd/ng_uni-mtx.diff
http://www.devrandom.co.uk/freebsd/pst-mtx.diff

(sos cc'd due to ATA patches)

Gavin



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