Source code

Revision control

Copy as Markdown

Other Tools

// This Source Code Form is subject to the terms of the Mozilla Public
// License, v. 2.0. If a copy of the MPL was not distributed with this
// file, You can obtain one at https://mozilla.org/MPL/2.0/.
use inherent::inherent;
use std::convert::TryInto;
use std::sync::Arc;
use super::{
BaseMetricId, ChildMetricMeta, CommonMetricData, DistributionData, MemoryUnit, MetricId,
};
use glean::traits::MemoryDistribution;
use crate::ipc::{need_ipc, with_ipc_payload};
#[cfg(feature = "with_gecko")]
use super::profiler_utils::{
truncate_vector_for_marker, DistributionMetricMarker, DistributionValues,
TelemetryProfilerCategory,
};
/// A memory distribution metric.
///
/// Memory distributions are used to accumulate and store memory measurements for analyzing distributions of the memory data.
#[derive(Clone)]
pub enum MemoryDistributionMetric {
Parent {
/// The metric's ID. Used for testing and profiler markers. Memory
/// distribution metrics can be labeled, so we may have either a
/// metric ID or sub-metric ID.
id: MetricId,
inner: Arc<glean::private::MemoryDistributionMetric>,
},
Child(ChildMetricMeta),
}
define_metric_namer!(MemoryDistributionMetric);
impl MemoryDistributionMetric {
/// Create a new memory distribution metric.
pub fn new(id: BaseMetricId, meta: CommonMetricData, memory_unit: MemoryUnit) -> Self {
if need_ipc() {
MemoryDistributionMetric::Child(ChildMetricMeta::from_common_metric_data(id, meta))
} else {
let inner = glean::private::MemoryDistributionMetric::new(meta, memory_unit);
MemoryDistributionMetric::Parent {
id: id.into(),
inner: Arc::new(inner),
}
}
}
#[cfg(test)]
pub(crate) fn child_metric(&self) -> Self {
match self {
MemoryDistributionMetric::Parent { id, inner } => MemoryDistributionMetric::Child(
// SAFETY: We can unwrap here, as this code is only run in the
// context of a test. If this code is used elsewhere, the
// `unwrap` should be replaced with proper error handling of
// the `None` case.
ChildMetricMeta::from_metric_identifier(
(*id).base_metric_id().unwrap(),
inner.as_ref(),
),
),
MemoryDistributionMetric::Child(_) => {
panic!("Can't get a child metric from a child metric")
}
}
}
#[cfg(test)]
pub(crate) fn metric_id(&self) -> MetricId {
match self {
MemoryDistributionMetric::Parent { id, .. } => *id,
MemoryDistributionMetric::Child(meta) => meta.id.into(),
}
}
pub fn accumulate_samples(&self, samples: Vec<u64>) {
#[cfg(feature = "with_gecko")]
let marker_samples = truncate_vector_for_marker(&samples);
#[allow(unused)]
let id = match self {
MemoryDistributionMetric::Parent { id, inner } => {
inner.accumulate_samples(samples.into_iter().map(|s| s as _).collect());
*id
}
MemoryDistributionMetric::Child(meta) => {
with_ipc_payload(move |payload| {
if let Some(v) = payload.memory_samples.get_mut(&meta.id) {
v.extend(samples);
} else {
payload.memory_samples.insert(meta.id, samples);
}
});
MetricId::Id(meta.id)
}
};
#[cfg(feature = "with_gecko")]
gecko_profiler::lazy_add_marker!(
"MemoryDistribution::accumulate",
TelemetryProfilerCategory,
DistributionMetricMarker::<MemoryDistributionMetric, u64>::new(
id,
None,
DistributionValues::Samples(marker_samples)
)
);
}
pub fn start_buffer(&self) -> LocalMemoryDistribution<'_> {
match self {
MemoryDistributionMetric::Parent { inner, .. } => {
LocalMemoryDistribution::Parent(inner.start_buffer())
}
MemoryDistributionMetric::Child(_) => {
// TODO(bug 1920957): Buffering not implemented for child processes yet. We don't
// want to panic though.
log::warn!("Can't get a local memory distribution from a child metric. No data will be recorded.");
LocalMemoryDistribution::Child
}
}
}
}
pub enum LocalMemoryDistribution<'a> {
Parent(glean::private::LocalMemoryDistribution<'a>),
Child,
}
impl LocalMemoryDistribution<'_> {
pub fn accumulate(&mut self, sample: u64) {
match self {
LocalMemoryDistribution::Parent(p) => p.accumulate(sample),
LocalMemoryDistribution::Child => {
log::debug!("Can't accumulate local memory distribution in a child process.")
}
}
}
}
#[inherent]
impl MemoryDistribution for MemoryDistributionMetric {
/// Accumulates the provided sample in the metric.
///
/// ## Arguments
///
/// * `sample` - The sample to be recorded by the metric. The sample is assumed to be in the
/// configured memory unit of the metric.
///
/// ## Notes
///
/// Values bigger than 1 Terabyte (2<sup>40</sup> bytes) are truncated
/// and an `ErrorType::InvalidValue` error is recorded.
pub fn accumulate(&self, sample: u64) {
#[allow(unused)]
let id = match self {
MemoryDistributionMetric::Parent { id, inner } => {
// values are capped at 2**40.
// If the value doesn't fit into `i64` it's definitely to large
// and cause an error.
// glean-core handles that.
let sample = sample.try_into().unwrap_or_else(|_| {
log::warn!(
"Memory size too large to fit into into 64-bytes. Saturating at i64::MAX."
);
i64::MAX
});
inner.accumulate(sample);
*id
}
MemoryDistributionMetric::Child(meta) => {
with_ipc_payload(move |payload| {
if let Some(v) = payload.memory_samples.get_mut(&meta.id) {
v.push(sample);
} else {
payload.memory_samples.insert(meta.id, vec![sample]);
}
});
MetricId::Id(meta.id)
}
};
#[cfg(feature = "with_gecko")]
gecko_profiler::lazy_add_marker!(
"MemoryDistribution::accumulate",
TelemetryProfilerCategory,
DistributionMetricMarker::<MemoryDistributionMetric, u64>::new(
id,
None,
DistributionValues::Sample(sample)
)
);
}
/// **Exported for test purposes.**
///
/// Gets the number of recorded errors for the given error type.
///
/// # Arguments
///
/// * `error` - The type of error
/// * `ping_name` - represents the optional name of the ping to retrieve the
/// metric for. Defaults to the first value in `send_in_pings`.
///
/// # Returns
///
/// The number of errors recorded.
pub fn test_get_num_recorded_errors(&self, error: glean::ErrorType) -> i32 {
match self {
MemoryDistributionMetric::Parent { inner, .. } => {
inner.test_get_num_recorded_errors(error)
}
MemoryDistributionMetric::Child(meta) => panic!(
"Cannot get the number of recorded errors for {:?} in non-parent process!",
meta.id
),
}
}
}
#[inherent]
impl glean::TestGetValue<DistributionData> for MemoryDistributionMetric {
/// **Test-only API.**
///
/// Get the currently-stored histogram as a DistributionData of the serialized value.
/// This doesn't clear the stored value.
///
/// ## Arguments
///
/// * `ping_name` - the storage name to look into.
///
/// ## Return value
///
/// Returns the stored value or `None` if nothing stored.
pub fn test_get_value(&self, ping_name: Option<String>) -> Option<DistributionData> {
match self {
MemoryDistributionMetric::Parent { inner, .. } => inner.test_get_value(ping_name),
MemoryDistributionMetric::Child(meta) => {
panic!(
"Cannot get test value for {:?} in non-parent process!",
meta.id
)
}
}
}
}
#[cfg(test)]
mod test {
use super::*;
use crate::{common_test::*, ipc, metrics};
#[test]
fn smoke_test_memory_distribution() {
let _lock = lock_test();
let metric = MemoryDistributionMetric::new(
BaseMetricId(0),
CommonMetricData {
name: "memory_distribution_metric".into(),
category: "telemetry".into(),
send_in_pings: vec!["test-ping".into()],
disabled: false,
..Default::default()
},
MemoryUnit::Kilobyte,
);
metric.accumulate(42);
let metric_data = metric.test_get_value(Some("test-ping".to_string())).unwrap();
assert_eq!(1, metric_data.values[&42494]);
assert_eq!(43008, metric_data.sum);
}
#[test]
fn memory_distribution_child() {
let _lock = lock_test();
let parent_metric = &metrics::test_only_ipc::a_memory_dist;
parent_metric.accumulate(42);
{
let child_metric = parent_metric.child_metric();
// scope for need_ipc RAII
let _raii = ipc::test_set_need_ipc(true);
child_metric.accumulate(13 * 9);
}
let metric_data = parent_metric.test_get_value(Some("test-ping".to_string())).unwrap();
assert_eq!(1, metric_data.values[&42494]);
assert_eq!(43008, metric_data.sum);
// Single-process IPC machine goes brrrrr...
let buf = ipc::take_buf().unwrap();
assert!(buf.len() > 0);
assert!(ipc::replay_from_buf(&buf).is_ok());
let data = parent_metric.test_get_value(None).expect("must have data");
assert_eq!(2, data.values.values().fold(0, |acc, count| acc + count));
assert_eq!(1, data.values[&42494]);
assert_eq!(1, data.values[&115097]);
assert_eq!(162816, data.sum);
}
}