Date: Tue, 24 Jun 2014 08:19:47 -0700 From: Sean Bruno <sbruno@ignoranthack.me> To: Adrian Chadd <adrian@freebsd.org> Cc: "freebsd-wireless@freebsd.org" <freebsd-wireless@freebsd.org> Subject: Re: wireless mcast updates, ic->ic_update_mcast Message-ID: <1403623187.7781.6.camel@bruno> In-Reply-To: <CAJ-Vmo=1ZstSanGP5zK8tnnOrE_sO4MKNFRcGvCJPJ8MZZL0yA@mail.gmail.com> References: <1403366123.39384.10.camel@bruno> <CAJ-Vmo=1ZstSanGP5zK8tnnOrE_sO4MKNFRcGvCJPJ8MZZL0yA@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, 2014-06-21 at 12:47 -0700, Adrian Chadd wrote: > Well, it depends on what the hardware may want or desire to function > correctly. I've no idea what theose chips require for multicast > behaviour. > > > -a > > I think that was kind of my point of bringing this up. Only a handful of cards really implement the appropriate code. Most of the rest, implement stub functions that hide a console print message indicating missing functionality. I propose nuking most of these stub functions and wrapping the console printf in a bootverbose. sean > On 21 June 2014 08:55, Sean Bruno <sbruno@ignoranthack.me> wrote: > > My T61 has a wpi(4) card in it. I noted that the console message "needs > > muticast update" is emitted as it doesn't implement the required > > function in its code and uses the default of the net80211 stack. I > > haven't noted any issues, but I took a journey through some of the > > wireless drivers to see what the status of implementation was. It seems > > most of the drivers stub out the required function to get rid of the > > warning. > > > > I perceive that with 6 exceptions, most of our wireless adapters don't > > have an implementation, or worse, they implement a stub to squash the > > warning. Shouldn't these stub functions be removed? Perhaps wrapp the > > net80211 default behavior in bootverbose? > > > > -- implemented ic_update_mcast() > > ath/if_ath.c -- implemented and working > > usb/wlan/if_zyd.c -- implemented and working > > usb/wlan/if_wi.c -- implemented and working > > > > -- sets a flag to default o/s behvavior? > > mwl/if_mlw.c -- function coded out, most code if 0'd out > > usb/wlan/if_upgt.c -- looks to default to whatever the o/s can do. > > usr/wlan/if_run.c -- looks to default to whatever the o/s can do > > > > -- stubbed out function to squash warning (bug?) > > iwn/if_iwn.c -- stubbed out in if_iwn to remove default error > > usb/wlan/if_rum.c -- stubbed out to remove default error > > usb/wlan/if_urtwn.c -- stubbed out to remove default error > > usb/wlan/if_urtw.c -- stubbed out to remove default error > > usr/wlan/if_uath.c -- stubbed out to remove default error > > usb/wlan/if_rsu.c -- stubbed out to remove default error > > wtap/if_wtap.c -- stubbed out to remove default error > > > > -- undefined, emits error correctly > > wpi/if_wpi.c > > bwi/if_bwi.c > > bwn/if_bwn.c > > ipw/if_ipw.c > > malo/if_malo.c > > ral/rt2560.c > > ral/rt2860.c > > ral/rt2661.c > > wi/if_wi.c > > usb/wlan/if_ural.c > > > > _______________________________________________ > > freebsd-wireless@freebsd.org mailing list > > http://lists.freebsd.org/mailman/listinfo/freebsd-wireless > > To unsubscribe, send any mail to "freebsd-wireless-unsubscribe@freebsd.org"
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1403623187.7781.6.camel>