Skip to content

bnx2x: fixes

Dmitry Kravkov netdev 5 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v5 Latest5 patches 10 repliesIn review
  2. v46 patches 3 repliesSuperseded
  3. v36 patches 2 repliesSuperseded
Earlier versions · 2
VersionPostedPatchesRepliesStatus
v26 patches1 replySuperseded
v16 patches5 repliesSuperseded

Changes in v5 from the cover letter

* patch 2/6: replace proprietary locking with semaphore
* droped 1/6: since adds redundant code from Benjamin Poirier

Patches in v5 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/5bnx2x: protect different statistics flowsdrivers/net/ethernet/broadcom/bnx2x/bnx2x.h drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c drivers/net/ethernet/broadcom/bnx2x/bnx2x_stats.c+57 -124 replies
2/5bnx2x: update fairness parameters following DCB negotiationdrivers/net/ethernet/broadcom/bnx2x/bnx2x.h drivers/net/ethernet/broadcom/bnx2x/bnx2x_dcb.c drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c+24 -120 replies
3/5bnx2x: fix memory leak in VFdrivers/net/ethernet/broadcom/bnx2x/bnx2x_sriov.c+1 -10 replies
4/5bnx2x: fix PTE write access errordrivers/net/ethernet/broadcom/bnx2x/bnx2x.h drivers/net/ethernet/broadcom/bnx2x/bnx2x_hsi.h drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c+15 -00 replies
5/5bnx2x: prevent crash in shutdown flow with CNICdrivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c+1 -20 replies

Reviews trailers per patch

1 of 5 patches has a Reviewed-by or Acked-by.

PatchNeal Cardwell
1/5A-b
2/5
3/5
4/5
5/5

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v5 thread

Cover letter [PATCH net v5 0/5] bnx2x: fixes

Hello Dave

Please consider applying the series of bnx2x fixes to net:
* statistics may cause FW assert
* missing fairness configuration in DCB flow
* memory leak in sriov related part
* Illegal PTE access
* Pagefault crash in shutdown flow with cnic
v1->v2
* fixed sparse error pointed by Joe Perches
* added missing signed-off from Sergei Shtylyov
v2->v3
* added missing signed-off from Sergei Shtylyov
* fixed formatting from Sergei Shtylyov
v3->v4
* patch 1/6: fixed declaration order
* patch 2/6 replaced with: protect flows using set_bit constraints
v4->v5
* patch 2/6: replace proprietary locking with semaphore
* droped 1/6: since adds redundant code from Benjamin Poirier

Thanks
Dmitry