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
F157293057: D38711.id117708.diff
Wed, May 20, 2:18 AM
Unknown Object (File)
Sat, May 16, 11:44 PM
Unknown Object (File)
Sat, May 16, 11:44 PM
Unknown Object (File)
Sat, May 16, 11:44 PM
Unknown Object (File)
Sat, May 16, 11:43 PM
Unknown Object (File)
Thu, May 7, 1:47 PM
Unknown Object (File)
Apr 18 2026, 3:47 PM
Unknown Object (File)
Apr 18 2026, 3:46 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...