Page MenuHomeFreeBSD

geom: Remove unused fields from struct bio
ClosedPublic

Authored by markj on Thu, Sep 24, 1:55 PM.
Tags
None
Referenced Files
F173431860: D59986.id187602.diff
Fri, Sep 25, 9:53 PM
F173425704: D59986.id.diff
Fri, Sep 25, 9:01 PM
F173422076: D59986.id187690.diff
Fri, Sep 25, 8:34 PM
F173410005: D59986.diff
Fri, Sep 25, 7:01 PM
F173386432: D59986.id187602.diff
Fri, Sep 25, 3:11 PM
F173381209: D59986.diff
Fri, Sep 25, 2:15 PM
F173380085: D59986.id187602.diff
Fri, Sep 25, 2:02 PM
F173295427: D59986.id187602.diff
Fri, Sep 25, 2:01 AM
Subscribers

Details

Summary

No functional change intended.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

markj requested review of this revision.Thu, Sep 24, 1:55 PM

So some history. phk added this twice: once in 2003 and again in 2004. Søren (sos) used them in his non-cam ata rework also in 2004. This was the only in-tree driver that used them. It was removed in 2013 in the move to the CAM-based ata framework that mav did. trasz removed the machinery and functions that used this in 2016.

So this appears to be totally dead and unused and can be removed.

This revision is now accepted and ready to land.Thu, Sep 24, 2:21 PM
In D59986#1376309, @imp wrote:

So some history. phk added this twice: once in 2003 and again in 2004. Søren (sos) used them in his non-cam ata rework also in 2004. This was the only in-tree driver that used them. It was removed in 2013 in the move to the CAM-based ata framework that mav did. trasz removed the machinery and functions that used this in 2016.

So this appears to be totally dead and unused and can be removed.

Thanks, I did a bit of digging too but forgot to update the commit log message. :(

This revision was automatically updated to reflect the committed changes.