[PATCH] fwip(4) panic on attach
Sam Leffler
sam at errno.com
Sun Jul 31 17:13:22 GMT 2005
Christian Brueffer wrote:
> Hi,
>
> in CURRENT and RELENG_6 I get a reproducible panic with fwip(4) when
> plugging in my firewire cardbus card. The following patch fixes it,
> but I'm not sure if it's correct.
>
> The interface works after applying the patch.
>
> Index: if_fwip.c
> ===================================================================
> RCS file: /data/ncvs/freebsd/src/sys/dev/firewire/if_fwip.c,v
> retrieving revision 1.7
> diff -u -r1.7 if_fwip.c
> --- if_fwip.c 10 Jun 2005 16:49:08 -0000 1.7
> +++ if_fwip.c 31 Jul 2005 14:01:52 -0000
> @@ -171,11 +171,13 @@
>
> fwip = ((struct fwip_softc *)device_get_softc(dev));
> unit = device_get_unit(dev);
> +
> + bzero(fwip, sizeof(struct fwip_softc));
> +
> ifp = fwip->fw_softc.fwip_ifp = if_alloc(IFT_IEEE1394);
> if (ifp == NULL)
> return (ENOSPC);
>
> - bzero(fwip, sizeof(struct fwip_softc));
> /* XXX */
> fwip->dma_ch = -1;
>
I thought the softc was guaranteed to be zero'd; can't you just delete
the bzero?
Sam
More information about the freebsd-current
mailing list