11493 aggr needs support for multiple pseudo rx groups

Review Request #2201 — Created July 23, 2019 and discarded

rm
illumos-gate
master
11493
2200
general

11493 aggr needs support for multiple pseudo rx groups
Portions contributed by: Dan McDonald <danmcd@joyent.com>
Reviewed by: Patrick Mooney <patrick.mooney@joyent.com>
Reviewed by: Jerry Jelinek <jerry.jelinek@joyent.com>
Reviewed by: Robert Mustacchi <rm@joyent.com>



  • 2
  • 0
  • 0
  • 0
  • 2
Description From Last Updated
Can this just be an else (with no "if") but have ASSERT(rtype == MAC_RING_TYPE_TX) in the "then" clause? I'm suprised ... pwinder pwinder
For consistency with other checks can this be: if ((mcip->mci_state_flags & MCIS_IS_AGGR_PORT) != 0) { pwinder pwinder
pwinder
  1. 
      
  2. This will have to change now!
    pbchk will catch this and any others

    1. Hey Paul, thanks for taking a look at this. I moved this review over to gerritt.

      https://code.illumos.org/c/illumos-gate/+/336

      If you could re-apply your comments there that would be great. Though, I plan to leave the copyrights alone since they accurately reflect when the code was written (with the exception of any code I update, of course).

  3. usr/src/uts/common/io/mac/mac.c (Diff revision 1)
     
     

    Can this just be an else (with no "if") but have

    ASSERT(rtype == MAC_RING_TYPE_TX)
    

    in the "then" clause? I'm suprised the compiler hasn't moaned about "grp" maybe used uninitialized.

  4. For consistency with other checks can this be:

    if ((mcip->mci_state_flags & MCIS_IS_AGGR_PORT) != 0) {
    
  5. 
      
rm
Review request changed

Status: Discarded

Loading...