Date: Mon, 21 May 2018 13:49:24 -0700 From: Matthew Macy <mmacy@freebsd.org> To: Jonathan Looney <jonlooney@gmail.com> Cc: src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r333911 - head/sys/netinet Message-ID: <CAPrugNr2YUFtpMbXWQpxfqjPWmQeQGuN2B0zbp7F14J3Se9x8g@mail.gmail.com> In-Reply-To: <CADrOrmsP6e_nepCzv__qeknPPy7evB%2Bgbb5pbU_-z6Lurs7nCg@mail.gmail.com> References: <201805200227.w4K2Rwhh007423@repo.freebsd.org> <CADrOrmsP6e_nepCzv__qeknPPy7evB%2Bgbb5pbU_-z6Lurs7nCg@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, May 21, 2018 at 6:15 AM, Jonathan Looney <jonlooney@gmail.com> wrote: > On Sat, May 19, 2018 at 10:27 PM, Matt Macy <mmacy@freebsd.org> wrote: >> >> + il = malloc(sizeof(struct in_pcblist) + n * sizeof(struct inpcb >> *), M_TEMP, M_WAITOK|M_ZERO); >> + inp_list = il->il_inp_list; > > > Why does this need M_ZERO? It allows me to assert that it hasn't already been passed to epoch_call. However, it doesn't make sense to pass it on non-INVARIANTS builds. -M
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAPrugNr2YUFtpMbXWQpxfqjPWmQeQGuN2B0zbp7F14J3Se9x8g>