PERFORCE change 157132 for review

Hans Petter Selasky hselasky at c2i.net
Wed Feb 4 06:12:14 PST 2009


On Wednesday 04 February 2009, Weongyo Jeong wrote:
>         bzero(desc, sizeof(struct uath_tx_desc));
>         desc->msglen = htobe32(sizeof(struct uath_tx_desc));
> -       desc->msgid  = data_idx + 1;    /* don't care about endianness */
> +       desc->msgid  = (sc->sc_msgid++) + 1;    /* don't care about
> endianness */ desc->type   = htobe32(WDCMSG_FLUSH);
>         desc->txqid  = htobe32(0);
>         desc->connid = htobe32(0);

Why not allocate an mbuf to hold the "descriptor" data. Then you use the 
m_next field of the header mbuf to link with the "data" mbuf ? Then you only 
need one [mbuf] queue for TX ? You can look at if_ural2.c and if_rum2.c 
in /sys/dev/usb2/wlan .

--HPS


More information about the p4-projects mailing list