Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1973)

Unified Diff: device/generic_sensor/public/cpp/sensor_reading.h

Issue 2395853003: [Sensors] Improvements in shared buffer managing (Closed)
Patch Set: Test compilation fix + comment from Ken Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « device/generic_sensor/public/cpp/BUILD.gn ('k') | device/generic_sensor/public/cpp/sensor_reading.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: device/generic_sensor/public/cpp/sensor_reading.h
diff --git a/device/generic_sensor/public/cpp/sensor_reading.h b/device/generic_sensor/public/cpp/sensor_reading.h
new file mode 100644
index 0000000000000000000000000000000000000000..6d0a42fd9afab504b7998d9fdb580a6d8f25a43a
--- /dev/null
+++ b/device/generic_sensor/public/cpp/sensor_reading.h
@@ -0,0 +1,62 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef DEVICE_GENERIC_SENSOR_PUBLIC_CPP_SENSOR_READING_H_
+#define DEVICE_GENERIC_SENSOR_PUBLIC_CPP_SENSOR_READING_H_
+
+#include "device/base/synchronization/one_writer_seqlock.h"
+#include "device/generic_sensor/public/interfaces/sensor.mojom.h"
+
+namespace device {
+
+// This class is guarantied to have a fixed size of 64 bits on every platform.
+// It is introduce to simplify sensors shared buffer memory calculation.
+template <typename Data>
+class SensorReadingField {
+ public:
+ static_assert(sizeof(Data) <= sizeof(int64_t),
+ "The field size must be <= 64 bits.");
+ SensorReadingField() = default;
+ SensorReadingField(Data value) { storage_.value = value; }
+ SensorReadingField& operator=(Data value) {
+ storage_.value = value;
+ return *this;
+ }
+ Data& value() { return storage_.value; }
+ operator Data() const { return storage_.value; }
+
+ private:
+ union Storage {
+ int64_t unused;
+ Data value;
+ Storage() { new (&value) Data(); }
+ ~Storage() { value.~Data(); }
+ };
+ Storage storage_;
+};
+
+// This structure represents sensor reading data: timestamp and 3 values.
+struct SensorReading {
+ SensorReading();
+ ~SensorReading();
+ SensorReading(const SensorReading& other);
+ SensorReadingField<double> timestamp;
+ SensorReadingField<double> values[3];
+};
+
+// This structure represents sensor reading buffer: sensor reading and seqlock
+// for synchronization.
+struct SensorReadingSharedBuffer {
+ SensorReadingSharedBuffer();
+ ~SensorReadingSharedBuffer();
+ SensorReadingField<OneWriterSeqLock> seqlock;
+ SensorReading reading;
+
+ // Gets the shared reading buffer offset for the given sensor type.
+ static uint64_t GetOffset(mojom::SensorType type);
+};
+
+} // namespace device
+
+#endif // DEVICE_GENERIC_SENSOR_PUBLIC_CPP_SENSOR_READING_H_
« no previous file with comments | « device/generic_sensor/public/cpp/BUILD.gn ('k') | device/generic_sensor/public/cpp/sensor_reading.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698