Page MenuHomeFreeBSD
Differential D59680

mlx4: convert to timer_setup() LinuxKPI
AcceptedPublic
Actions

Authored by bz on Mon, Sep 14, 10:54 PM.

Details

Reviewers
kib
Summary

The Linux driver switched away from using this timer in e22979d96a55d.
Locally adjust the code to use timer_setup/from_timer in order to get
us out of Linux version 4.15 (and earlier) KPIs.

MFC after: 3 days

Test Plan

Please add others to the review as you see fit.
I CANNOT TEST THIS.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 76906
Build 73789: arc lint + arc unit

Event Timeline

bz created this revision.Mon, Sep 14, 10:54 PM
Herald added a subscriber: imp. · View Herald TranscriptMon, Sep 14, 10:54 PM
bz requested review of this revision.Mon, Sep 14, 10:54 PM
Harbormaster completed remote builds in B76903: Diff 186733.Mon, Sep 14, 10:55 PM
bz added a parent revision: D58834: LinuxKPI: rework struct timer_list (*function[_415]).Mon, Sep 14, 10:55 PM
bz added inline comments.Mon, Sep 14, 11:10 PM
sys/dev/mlx4/mlx4_en/mlx4_en_tx.c
438

Switch to timer_container_of() right away.

bz updated this revision to Diff 186736.Tue, Sep 15, 12:00 AM

Also apply 41cb08555c416 from_timer -> timer_container_of.

Harbormaster completed remote builds in B76906: Diff 186736.Tue, Sep 15, 12:00 AM
bz added a child revision: D59688: LinuxKPI: remove timer KPI from Linux version before 4.15.Tue, Sep 15, 1:35 AM
kib accepted this revision.Tue, Sep 15, 5:29 AM

Nvidia cannot test this either. Consider my review as just a fellow committer looking at the same code.

This revision is now accepted and ready to land.Tue, Sep 15, 5:29 AM

Revision Contents
Changeset List

PathSize
sys/
dev/
mlx4/
mlx4_en/
2 lines
4 lines
4 lines

Diff 186736

View Options

sys/dev/mlx4/mlx4_en/en.h

Loading...
View Options

sys/dev/mlx4/mlx4_en/mlx4_en_cq.c

Loading...
View Options

sys/dev/mlx4/mlx4_en/mlx4_en_tx.c

Loading...