Date: Thu, 13 Mar 2014 22:13:41 +0000 From: "Meyer, Conrad" <conrad.meyer@isilon.com> To: John Baldwin <jhb@freebsd.org>, "freebsd-hackers@freebsd.org" <freebsd-hackers@freebsd.org> Subject: RE: [PATCH] x86/mca.c: malloc with correct pointer type Message-ID: <A3CAF0E84A34A540B4C74454358E003F1292E8FF@MX103CL02.corp.emc.com> In-Reply-To: <201403131357.07665.jhb@freebsd.org> References: <1394728529-22495-1-git-send-email-conrad.meyer@isilon.com>, <201403131357.07665.jhb@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
> From: John Baldwin [jhb@freebsd.org]=0A= > Sent: Thursday, March 13, 2014 10:57 AM=0A= > To: freebsd-hackers@freebsd.org=0A= > Cc: Meyer, Conrad=0A= > Subject: Re: [PATCH] x86/mca.c: malloc with correct pointer type=0A= > =0A= > > - cmc_state =3D malloc((mp_maxid + 1) * sizeof(struct cmc_state **)= ,=0A= > > - M_MCA, M_WAITOK);=0A= > > + cmc_state =3D malloc((mp_maxid + 1) * sizeof(*cmc_state), M_MCA,= =0A= > > + M_WAITOK);=0A= > > for (i =3D 0; i <=3D mp_maxid; i++)=0A= > > cmc_state[i] =3D malloc(sizeof(struct cmc_state) * mca_ba= nks,=0A= > > M_MCA, M_WAITOK | M_ZERO);=0A= > =0A= > I would rather do 'struct cmc_state *' as I think it is clearer to read, = but=0A= > the change is correct. Thanks!=0A= > =0A= > --=0A= > John Baldwin=0A= =0A= Sounds fine to me.=0A= =0A= Thanks,=0A= Conrad=
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?A3CAF0E84A34A540B4C74454358E003F1292E8FF>