Page MenuHomeFreeBSD

ahc(4): resolve some minor nits
ClosedPublic

Authored by kevans on Apr 19 2024, 2:05 AM.
Tags
None
Referenced Files
F102987280: D44859.diff
Tue, Nov 19, 12:38 PM
Unknown Object (File)
Wed, Nov 13, 9:34 AM
Unknown Object (File)
Fri, Oct 25, 5:50 PM
Unknown Object (File)
Fri, Oct 25, 5:49 PM
Unknown Object (File)
Fri, Oct 25, 5:49 PM
Unknown Object (File)
Fri, Oct 25, 5:37 PM
Unknown Object (File)
Oct 18 2024, 12:22 PM
Unknown Object (File)
Oct 1 2024, 12:40 PM
Subscribers
None

Details

Summary

In ahc_init(), qoutfifo is already assigned to effectively the same
value a couple lines up, except in the first assignment it uses the
proper definition; keep the more descriptive assignment.

ahc_targetcmd_offset() gets the offset wrong entirely; as per the
area of ahc_init() this diff also touches, targetcmds is laid out first
in the shared map and it's followed by the qoutfifo. As a result, we'd
generally be getting negative offsets here. We can't actually do a
partial sync anyways, so there was no consequence to getting this wrong.

Diff Detail

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

Event Timeline

kevans created this revision.

I think this is good... but mav will know for sure if we're missing something

This revision is now accepted and ready to land.Apr 19 2024, 2:12 AM

Looks good to me, though seems only cosmetic.

This revision was automatically updated to reflect the committed changes.