Disable background_threads support on windows

Background Threads are unsupported on windows in jemalloc, due to
requiring pthreads (which aren't available on windows, at least on the
msvc target). Trying to enable it will cause a runtime error causing the
program to immediately abort.

As such, to avoid problems, we set windows as a target that does not
support background_threads.

Signed-off-by: roblabla <[email protected]>

Author: roblabla

.github/workflows/main.yml

@@ -18,7 +18,7 @@ jobs:
         include:
           - name: x86_64-pc-windows-msvc
             target: x86_64-pc-windows-msvc
-            nobgt: 0
+            nobgt: 1
             no_tests: 1
             tag: windows-latest
           - name: x86_64-apple-darwin

jemalloc-ctl/src/lib.rs

@@ -141,7 +141,7 @@ option! {
     /// # static ALLOC: tikv_jemallocator::Jemalloc = tikv_jemallocator::Jemalloc;
     /// #
     /// # fn main() {
-    /// # #[cfg(not(target_os = "macos"))] {
+    /// # #[cfg(not(any(target_os = "macos", windows)))] {
     /// #
     /// use tikv_jemalloc_ctl::background_thread;
     /// let bg = background_thread::mib().unwrap();
@@ -171,7 +171,7 @@ option! {
     /// # static ALLOC: tikv_jemallocator::Jemalloc = tikv_jemallocator::Jemalloc;
     /// #
     /// # fn main() {
-    /// # #[cfg(not(target_os = "macos"))] {
+    /// # #[cfg(not(any(target_os = "macos", windows)))] {
     /// #
     /// use tikv_jemalloc_ctl::max_background_threads;
     /// let m = max_background_threads::mib().unwrap();

jemalloc-ctl/src/macros.rs

@@ -69,7 +69,7 @@ macro_rules! r {
                 match stringify!($id) {
                     "background_thread" |
                     "max_background_threads"
-                    if cfg!(target_os = "macos") => return,
+                    if cfg!(any(target_os = "macos", windows)) => return,
                     _ => (),
                 }
 
@@ -117,7 +117,7 @@ macro_rules! w {
                 match stringify!($id) {
                     "background_thread" |
                     "max_background_threads"
-                        if cfg!(target_os = "macos") => return,
+                        if cfg!(any(target_os = "macos", windows)) => return,
                     _ => (),
                 }
 
@@ -167,7 +167,7 @@ macro_rules! u {
                 match stringify!($id) {
                     "background_thread" |
                     "max_background_threads"
-                        if cfg!(target_os = "macos") => return,
+                        if cfg!(any(target_os = "macos", windows)) => return,
                     _ => (),
                 }
 

jemalloc-sys/src/env.rs

@@ -12,7 +12,7 @@ pub static UNSUPPORTED_TARGETS: &[&str] = &[
 pub static UNTESTED_TARGETS: &[&str] = &["openbsd", "msvc"];
 
 /// `jemalloc`'s background_thread support is known not to work on these targets:
-pub static NO_BG_THREAD_TARGETS: &[&str] = &["musl"];
+pub static NO_BG_THREAD_TARGETS: &[&str] = &["musl", "windows"];
 
 /// targets that don't support unprefixed `malloc`
 // “it was found that the `realpath` function in libc would allocate with libc malloc

jemallocator/tests/background_thread_enabled.rs

@@ -2,7 +2,7 @@
 //! library was compiled with background thread run-time support.
 #![cfg(feature = "background_threads_runtime_support")]
 #![cfg(not(feature = "unprefixed_malloc_on_supported_platforms"))]
-#![cfg(not(target_env = "musl"))]
+#![cfg(not(any(windows, target_env = "musl")))]
 
 use tikv_jemallocator::Jemalloc;