Currently sys/cachectl.h is used unconditionally on MIPS although it is
only available on Linux and will fail the build when targetting baremetal
Details
Diff Detail
- Build Status
Buildable 12628 Build 12628: arc lint + arc unit
Event Timeline
I checked Linux, FreeBSD and MacOS and only found it one Linux so I assumed it was Linux specific (especially since the commit that introduced it (r228767) only mentions Linux.
Should I just add || defined(__NetBSD__) or are there any other systems that might have it?
It's also unclear for me what is 'bare metal' about. FreeBSD should have the same header.
I grepped the FreeBSD source tree for cachectl and the only matches are LLVM and compiler-rt.
By baremetal I mean no operating system and only newlib running on MIPS qemu.
The original change (r228767) was tested on Linux only (including Android). I have never tested this part of the code on any *BSD system.
OK, so we can reschedule it for later. Excuse me for noise. It looks like compiler-rt wasn't built (in a while) on NetBSD/evbmips (release engineering machines test x86_64, i386, ppc, sparc, sparc64, arm). Once it will be done, we will address it.