cvs commit: src/sys/dev/em if_em.c if_em.h
Gleb Smirnoff
glebius at FreeBSD.org
Mon Aug 7 11:46:09 UTC 2006
Prafulla,
On Fri, Aug 04, 2006 at 04:21:49PM +0000, Prafulla Deuskar wrote:
P> > Thanks. Any objections about merging the HEAD to RELENG_6 soon?
P>
P> Jack Vogel is doing the merge - I will send it out to you for review.
P> So let us plan on committing early next week?
The merge is very simple:
1) take the driver from HEAD
2) rename VLAN_INPUT_TAG() macro to VLAN_INPUT_TAG_NEW()
3) revert revision 1.105, since this functionality isn't yet in RELENG_6
I have prepared the patch last week and posted it on STABLE, and several
people have already reported that they successfully run it. So, we can
commit to RELENG_6, I think.
P> Is there a report on what coverty found with if_em_hw.c?
Coverity complains that the return value of em_read_phy_reg() is
somtimes checked and sometimes is not. Can you please tell, whether
this is important report or can it be ignored?
--
Totus tuus, Glebius.
GLEBIUS-RIPN GLEB-RIPE
More information about the cvs-src
mailing list