diff options
| author | Manish Goregaokar <manishsmail@gmail.com> | 2016-01-14 04:22:18 +0530 |
|---|---|---|
| committer | Manish Goregaokar <manishsmail@gmail.com> | 2016-01-14 11:04:40 +0530 |
| commit | 6f4ede44dec50d58d10349c12256de5f3b254ade (patch) | |
| tree | fe09a4d3960574d5d861f52031c2aa1555d780c4 /src/liballoc | |
| parent | 6581563f7a7cafd40d17eb1921726febc10f8628 (diff) | |
| parent | 98bef2b81823483a23beef48c7999a67206d261d (diff) | |
| download | rust-6f4ede44dec50d58d10349c12256de5f3b254ade.tar.gz rust-6f4ede44dec50d58d10349c12256de5f3b254ade.zip | |
Rollup merge of #30801 - Amanieu:oom_print, r=alexcrichton
This adds the ability to override the default OOM behavior by setting a handler function. This is used by libstd to print a message when running out of memory instead of crashing with an obscure "illegal hardware instruction" error (at least on Linux). Fixes #14674
Diffstat (limited to 'src/liballoc')
| -rw-r--r-- | src/liballoc/lib.rs | 14 | ||||
| -rw-r--r-- | src/liballoc/oom.rs | 42 |
2 files changed, 45 insertions, 11 deletions
diff --git a/src/liballoc/lib.rs b/src/liballoc/lib.rs index 69f74ba73bb..ffa0ec4917c 100644 --- a/src/liballoc/lib.rs +++ b/src/liballoc/lib.rs @@ -92,6 +92,7 @@ #![feature(unsize)] #![feature(drop_in_place)] #![feature(fn_traits)] +#![feature(const_fn)] #![feature(needs_allocator)] @@ -134,15 +135,6 @@ mod boxed_test; pub mod arc; pub mod rc; pub mod raw_vec; +pub mod oom; -/// Common out-of-memory routine -#[cold] -#[inline(never)] -#[unstable(feature = "oom", reason = "not a scrutinized interface", - issue = "27700")] -pub fn oom() -> ! { - // FIXME(#14674): This really needs to do something other than just abort - // here, but any printing done must be *guaranteed* to not - // allocate. - unsafe { core::intrinsics::abort() } -} +pub use oom::oom; diff --git a/src/liballoc/oom.rs b/src/liballoc/oom.rs new file mode 100644 index 00000000000..d355d59185e --- /dev/null +++ b/src/liballoc/oom.rs @@ -0,0 +1,42 @@ +// Copyright 2014-2015 The Rust Project Developers. See the COPYRIGHT +// file at the top-level directory of this distribution and at +// http://rust-lang.org/COPYRIGHT. +// +// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or +// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license +// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your +// option. This file may not be copied, modified, or distributed +// except according to those terms. + +use core::sync::atomic::{AtomicPtr, Ordering}; +use core::mem; +use core::intrinsics; + +static OOM_HANDLER: AtomicPtr<()> = AtomicPtr::new(default_oom_handler as *mut ()); + +fn default_oom_handler() -> ! { + // The default handler can't do much more since we can't assume the presence + // of libc or any way of printing an error message. + unsafe { intrinsics::abort() } +} + +/// Common out-of-memory routine +#[cold] +#[inline(never)] +#[unstable(feature = "oom", reason = "not a scrutinized interface", + issue = "27700")] +pub fn oom() -> ! { + let value = OOM_HANDLER.load(Ordering::SeqCst); + let handler: fn() -> ! = unsafe { mem::transmute(value) }; + handler(); +} + +/// Set a custom handler for out-of-memory conditions +/// +/// To avoid recursive OOM failures, it is critical that the OOM handler does +/// not allocate any memory itself. +#[unstable(feature = "oom", reason = "not a scrutinized interface", + issue = "27700")] +pub fn set_oom_handler(handler: fn() -> !) { + OOM_HANDLER.store(handler as *mut (), Ordering::SeqCst); +} |
