Page MenuHomeFreeBSD
Differential D38711

mlx5: Enter network epoch when using if_foreach()
ClosedPublic
Actions

Authored by jhibbits on Feb 21 2023, 9:09 PM.
Tags
None
Referenced Files
F175500989: D38711.id117708.diff
Sun, Oct 11, 7:09 AM
F175485025: D38711.diff
Sun, Oct 11, 4:55 AM
F175386732: D38711.id118400.diff
Sat, Oct 10, 11:36 AM
Unknown Object (File)
Thu, Oct 1, 3:06 AM
Unknown Object (File)
Wed, Sep 30, 11:19 PM
Unknown Object (File)
Wed, Sep 30, 10:11 PM
Unknown Object (File)
Sun, Sep 27, 6:47 AM
Unknown Object (File)
Sat, Sep 26, 4:38 PM
View All Files
Subscribers
ae
imp
melifaro

Details

Reviewers
glebius
• hselasky
Group Reviewers
network
Commits
rG70510800d1ad: mlx5: Enter network epoch when using if_foreach()
Summary

IFNET_RLOCK() is not sufficient, the epoch needs entered.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 49931
Build 46823: arc lint + arc unit

Event Timeline

jhibbits created this revision.Feb 21 2023, 9:09 PM
Herald added subscribers: ae, imp. · View Herald TranscriptFeb 21 2023, 9:09 PM
jhibbits requested review of this revision.Feb 21 2023, 9:09 PM
Harbormaster completed remote builds in B49931: Diff 117708.Feb 21 2023, 9:09 PM
• hselasky added a comment.Feb 22 2023, 10:04 AM

Hi,

I'm not sure if that will work, because there is a write_lock() inside mlx5_enable_roce_if_cb(), and that cannot be used inside an EPOCH section!

#define write_lock(_l)          rw_wlock(&(_l)->rw)
• hselasky added a comment.Feb 22 2023, 10:13 AM
In D38711#881214, @hselasky wrote:

Hi,

I'm not sure if that will work, because there is a write_lock() inside mlx5_enable_roce_if_cb(), and that cannot be used inside an EPOCH section!

#define write_lock(_l)          rw_wlock(&(_l)->rw)

Or am I wrong.

--HPS

melifaro added a subscriber: melifaro.Feb 22 2023, 10:27 AM
In D38711#881221, @hselasky wrote:
In D38711#881214, @hselasky wrote:

Hi,

I'm not sure if that will work, because there is a write_lock() inside mlx5_enable_roce_if_cb(), and that cannot be used inside an EPOCH section!

#define write_lock(_l)          rw_wlock(&(_l)->rw)

Or am I wrong.

--HPS

It's OK to use rwlocks/mutexes (and basically anything that don't sleep) in the (network) epoch.

• hselasky accepted this revision.Feb 22 2023, 10:42 AM

I'm good with this patch!

This revision is now accepted and ready to land.Feb 22 2023, 10:42 AM
Closed by commit rG70510800d1ad: mlx5: Enter network epoch when using if_foreach() (authored by jhibbits). · Explain WhyMar 6 2023, 4:24 PM
This revision was automatically updated to reflect the committed changes.
jhibbits added a commit: rG70510800d1ad: mlx5: Enter network epoch when using if_foreach().

Revision Contents
Changeset List

PathSize
sys/
dev/
mlx5/
mlx5_ib/
5 lines

Diff 117708

View Options

sys/dev/mlx5/mlx5_ib/mlx5_ib_main.c

Loading...