opensolaris macros: drop INLINE_FLS checks
ClosedPublic
Actions

Authored by dougm on Jun 21 2023, 7:04 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Sep 24, 4:30 AM

	Unknown Object (File)
	Mon, Sep 23, 5:36 PM

	Unknown Object (File)
	Thu, Sep 19, 8:53 PM

	Unknown Object (File)
	Tue, Sep 17, 6:40 AM

	Unknown Object (File)
	Mon, Sep 16, 1:00 PM

	Unknown Object (File)
	Sun, Sep 15, 1:30 AM

	Unknown Object (File)
	Tue, Sep 10, 12:58 AM

	Unknown Object (File)
	Sun, Sep 1, 11:45 AM

View All 49 Files

Subscribers

imp

Details

Reviewers

mhorne
avg

Commits

rGe08776813087: inline_fls: remove redundant INLINE_FLS test

Summary

After D40698 is committed, HAS_INLINE_FLS and similar macros will be defined always. This change just removes the tests for these always-true conditions.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

dougm requested review of this revision.Jun 21 2023, 7:04 PM

dougm created this revision.

I am not sure where/if this file is used anymore. It originated as part of the Solaris/illumos ZFS implementation, which has been superseded by OpenZFS.

I tried quickly adding an #error directive to the header, but compiling a GENERIC amd64 kernel still succeeded.

In any case, making this change should be fine.

This revision is now accepted and ready to land.Jun 22 2023, 7:52 PM

Closed by commit rGe08776813087: inline_fls: remove redundant INLINE_FLS test (authored by dougm). · Explain WhyJul 6 2023, 6:32 PM

This revision was automatically updated to reflect the committed changes.

dougm added a commit: rGe08776813087: inline_fls: remove redundant INLINE_FLS test.

Herald added a subscriber: imp. · View Herald TranscriptJul 6 2023, 6:32 PM