On Tue, Jun 23, 2015 at 09:08:25PM -0600, Theo de Raadt wrote: > - bcopy(sc->tulip_setupdata, sc->tulip_setupbuf, > - sizeof(sc->tulip_setupbuf)); > + bcopy(sc->tulip_setupdata, sc->tulip_setupbuf, TULIP_SETUP);
> + sc->tulip_setupbuf = dma_alloc(TULIP_SETUP, PR_WAITOK); > + sc->tulip_setupdata = malloc(TULIP_SETUP, M_DEVBUF, M_WAITOK); > - u_int32_t tulip_setupbuf[192/sizeof(u_int32_t)]; > - u_int32_t tulip_setupdata[192/sizeof(u_int32_t)]; > +#define TULIP_SETUP (192 / sizeof(u_int32_t)) FWIW, change that to: +#define TULIP_SETUP 192 > + u_int32_t *tulip_setupbuf; > + u_int32_t *tulip_setupdata;