Page MenuHomeFreeBSD

vchi: rename bitset macros to avoid collision with bitset(9)
ClosedPublic

Authored by scottph on Aug 24 2020, 11:50 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 9, 4:34 AM
Unknown Object (File)
Mar 23 2024, 12:24 AM
Unknown Object (File)
Mar 23 2024, 12:24 AM
Unknown Object (File)
Mar 22 2024, 10:15 AM
Unknown Object (File)
Jan 10 2024, 4:46 PM
Unknown Object (File)
Dec 20 2023, 2:01 AM
Unknown Object (File)
Dec 12 2023, 12:27 AM
Unknown Object (File)
Nov 11 2023, 1:06 AM

Details

Summary

An upcoming change to include bitset(9) macros from vm_page.h
causes a macro name collision with vchi's custom bitset macros.

This change was performed mechanically by:

sed -i .orig s/BITSET/VCHI_BITSET/g $(grep -rl BITSET sys/contrib/vchiq)

MFC after: 1 week
Sponsored by: Ampere Computing, Inc.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Aug 25 2020, 8:44 AM

I wonder if we can upstream this too

I wonder if we can upstream this too

It looks like upstream has moved pretty far in the years since this code was imported. I believe the diff is on the order of:

3702 insertions(+), 7735 deletions(-)

(Where the sys/contrib/vchi has around 14k lines). So I think unless someone plans to feed and water the import, getting the change upstream won't really be of any benefit to us.

I wonder if we can upstream this too

It looks like upstream has moved pretty far in the years since this code was imported. I believe the diff is on the order of:

3702 insertions(+), 7735 deletions(-)

(Where the sys/contrib/vchi has around 14k lines). So I think unless someone plans to feed and water the import, getting the change upstream won't really be of any benefit to us.

Sure. I mean it shouldn't hold up this change in our tree, just that it probably makes sense to do on general principle.