Merge "Remove Foreground Boost CPUSet"
diff --git a/BoardConfig.mk b/BoardConfig.mk
index 2c78166..a616551 100644
--- a/BoardConfig.mk
+++ b/BoardConfig.mk
@@ -49,8 +49,8 @@
TARGET_RECOVERY_PIXEL_FORMAT := BGRA_8888
PRESENT_TIME_OFFSET_FROM_VSYNC_NS := 0
-VSYNC_EVENT_PHASE_OFFSET_NS := 1000000
-SF_VSYNC_EVENT_PHASE_OFFSET_NS := 1000000
+VSYNC_EVENT_PHASE_OFFSET_NS := 7500000
+SF_VSYNC_EVENT_PHASE_OFFSET_NS := 5000000
TARGET_USES_HWC2 := true
TARGET_USERIMAGES_USE_EXT4 := true
@@ -120,3 +120,7 @@
# Testing related defines
BOARD_PERFSETUP_SCRIPT := platform_testing/scripts/perf-setup/dragon-setup.sh
+
+# Vendor Interface Manifest
+DEVICE_MANIFEST_FILE := device/google/dragon/manifest.xml
+DEVICE_MATRIX_FILE := device/google/dragon/compatibility_matrix.xml
diff --git a/audio/hal/audio_hw.c b/audio/hal/audio_hw.c
index e83aff3..e2faafd 100644
--- a/audio/hal/audio_hw.c
+++ b/audio/hal/audio_hw.c
@@ -1023,12 +1023,7 @@
ssize_t frames_wr = 0; /* Number of frames actually read */
size_t bytes_per_sample = audio_bytes_per_sample(stream->common.get_format(&stream->common));
void *proc_buf_out = buffer;
-#ifdef PREPROCESSING_ENABLED
- audio_buffer_t in_buf;
- audio_buffer_t out_buf;
- int i;
- bool has_processing = in->num_preprocessors != 0;
-#endif
+
/* Additional channels might be added on top of main_channels:
* - aux_channels (by processing effects)
* - extra channels due to HW limitations
@@ -1037,7 +1032,34 @@
size_t src_channels = in->config.channels;
size_t dst_channels = audio_channel_count_from_in_mask(in->main_channels);
bool channel_remapping_needed = (dst_channels != src_channels);
- size_t src_buffer_size = frames_num * src_channels * bytes_per_sample;
+ const size_t src_frame_size = src_channels * bytes_per_sample;
+
+#ifdef PREPROCESSING_ENABLED
+ const bool has_processing = in->num_preprocessors != 0;
+#else
+ const bool has_processing = false;
+#endif
+
+ /* With additional channels or processing, we need intermediate buffers */
+ if (channel_remapping_needed || has_processing) {
+ const size_t src_buffer_size = frames_num * src_frame_size;
+
+ if (in->proc_buf_size < src_buffer_size) {
+ in->proc_buf_size = src_buffer_size;
+#ifdef PREPROCESSING_ENABLED
+ /* we always reallocate both buffers in case # of effects change dynamically. */
+ in->proc_buf_in = realloc(in->proc_buf_in, src_buffer_size);
+ ALOG_ASSERT((in->proc_buf_in != NULL),
+ "process_frames() failed to reallocate proc_buf_in");
+#endif
+ in->proc_buf_out = realloc(in->proc_buf_out, src_buffer_size);
+ ALOG_ASSERT((in->proc_buf_out != NULL),
+ "process_frames() failed to reallocate proc_buf_out");
+ }
+ if (channel_remapping_needed) {
+ proc_buf_out = in->proc_buf_out;
+ }
+ }
#ifdef PREPROCESSING_ENABLED
if (has_processing) {
@@ -1046,24 +1068,10 @@
while (frames_wr < frames_num) {
/* first reload enough frames at the end of process input buffer */
if (in->proc_buf_frames < (size_t)frames_num) {
- ssize_t frames_rd;
- if (in->proc_buf_size < (size_t)frames_num) {
- in->proc_buf_size = (size_t)frames_num;
- in->proc_buf_in = realloc(in->proc_buf_in, src_buffer_size);
- ALOG_ASSERT((in->proc_buf_in != NULL),
- "process_frames() failed to reallocate proc_buf_in");
- if (channel_remapping_needed) {
- in->proc_buf_out = realloc(in->proc_buf_out, src_buffer_size);
- ALOG_ASSERT((in->proc_buf_out != NULL),
- "process_frames() failed to reallocate proc_buf_out");
- proc_buf_out = in->proc_buf_out;
- }
- }
- frames_rd = read_frames(in,
- in->proc_buf_in +
- in->proc_buf_frames * src_channels * bytes_per_sample,
- frames_num - in->proc_buf_frames);
- if (frames_rd < 0) {
+ ssize_t frames_rd = read_frames(in,
+ (char *)in->proc_buf_in + in->proc_buf_frames * src_frame_size,
+ frames_num - in->proc_buf_frames);
+ if (frames_rd < 0) {
/* Return error code */
frames_wr = frames_rd;
break;
@@ -1073,17 +1081,20 @@
/* in_buf.frameCount and out_buf.frameCount indicate respectively
* the maximum number of frames to be consumed and produced by process() */
+ audio_buffer_t in_buf;
+ audio_buffer_t out_buf;
+
in_buf.frameCount = in->proc_buf_frames;
- in_buf.s16 = in->proc_buf_in;
+ in_buf.s16 = in->proc_buf_in; /* currently assumes PCM 16 effects */
out_buf.frameCount = frames_num - frames_wr;
- out_buf.s16 = (int16_t *)proc_buf_out + frames_wr * in->config.channels;
+ out_buf.s16 = (int16_t *)proc_buf_out + frames_wr * src_channels;
/* FIXME: this works because of current pre processing library implementation that
* does the actual process only when the last enabled effect process is called.
* The generic solution is to have an output buffer for each effect and pass it as
* input to the next.
*/
- for (i = 0; i < in->num_preprocessors; i++) {
+ for (int i = 0; i < in->num_preprocessors; i++) {
(*in->preprocessors[i].effect_itfe)->process(in->preprocessors[i].effect_itfe,
&in_buf,
&out_buf);
@@ -1096,8 +1107,8 @@
if (in->proc_buf_frames) {
memcpy(in->proc_buf_in,
- in->proc_buf_in + in_buf.frameCount * src_channels * bytes_per_sample,
- in->proc_buf_frames * in->config.channels * audio_bytes_per_sample(in_get_format(in)));
+ (char *)in->proc_buf_in + in_buf.frameCount * src_frame_size,
+ in->proc_buf_frames * src_frame_size);
}
/* if not enough frames were passed to process(), read more and retry. */
@@ -1120,23 +1131,14 @@
#endif //PREPROCESSING_ENABLED
{
/* No processing effects attached */
- if (channel_remapping_needed) {
- /* With additional channels, we cannot use original buffer */
- if (in->proc_buf_size < src_buffer_size) {
- in->proc_buf_size = src_buffer_size;
- in->proc_buf_out = realloc(in->proc_buf_out, src_buffer_size);
- ALOG_ASSERT((in->proc_buf_out != NULL),
- "process_frames() failed to reallocate proc_buf_out");
- }
- proc_buf_out = in->proc_buf_out;
- }
frames_wr = read_frames(in, proc_buf_out, frames_num);
ALOG_ASSERT(frames_wr <= frames_num, "read more frames than requested");
}
- if (channel_remapping_needed) {
+ /* check negative frames_wr (error) before channel remapping to avoid overwriting memory. */
+ if (channel_remapping_needed && frames_wr > 0) {
size_t ret = adjust_channels(proc_buf_out, src_channels, buffer, dst_channels,
- bytes_per_sample, frames_wr * src_channels * bytes_per_sample);
+ bytes_per_sample, frames_wr * src_frame_size);
ALOG_ASSERT(ret == (frames_wr * dst_channels * bytes_per_sample));
}
diff --git a/compatibility_matrix.xml b/compatibility_matrix.xml
new file mode 100644
index 0000000..3082485
--- /dev/null
+++ b/compatibility_matrix.xml
@@ -0,0 +1,59 @@
+<compatibility-matrix version="1.0" type="device">
+ <hal format="hidl" optional="false">
+ <name>android.frameworks.schedulerservice</name>
+ <version>1.0</version>
+ <interface>
+ <name>ISchedulingPolicyService</name>
+ <instance>default</instance>
+ </interface>
+ </hal>
+ <hal format="hidl" optional="false">
+ <name>android.frameworks.sensorservice</name>
+ <version>1.0</version>
+ <interface>
+ <name>ISensorManager</name>
+ <instance>default</instance>
+ </interface>
+ </hal>
+ <hal format="hidl" optional="false">
+ <name>android.hidl.allocator</name>
+ <version>1.0</version>
+ <interface>
+ <name>IAllocator</name>
+ <instance>ashmem</instance>
+ </interface>
+ </hal>
+ <hal format="hidl" optional="false">
+ <name>android.hidl.manager</name>
+ <version>1.0</version>
+ <interface>
+ <name>IServiceManager</name>
+ <instance>default</instance>
+ </interface>
+ </hal>
+ <hal format="hidl" optional="false">
+ <name>android.hidl.memory</name>
+ <version>1.0</version>
+ <interface>
+ <name>IMapper</name>
+ <instance>ashmem</instance>
+ </interface>
+ </hal>
+ <hal format="hidl" optional="false">
+ <name>android.hidl.token</name>
+ <version>1.0</version>
+ <interface>
+ <name>ITokenManager</name>
+ <instance>default</instance>
+ </interface>
+ </hal>
+ <hal format="hidl" optional="false">
+ <name>android.system.wifi.keystore</name>
+ <version>1.0</version>
+ <interface>
+ <name>IKeystore</name>
+ <instance>default</instance>
+ </interface>
+ </hal>
+
+</compatibility-matrix>
diff --git a/device.mk b/device.mk
index 3d8f38e..917e755 100644
--- a/device.mk
+++ b/device.mk
@@ -87,6 +87,7 @@
frameworks/native/data/etc/android.hardware.camera.full.xml:system/etc/permissions/android.hardware.camera.full.xml \
frameworks/native/data/etc/android.hardware.wifi.xml:system/etc/permissions/android.hardware.wifi.xml \
frameworks/native/data/etc/android.hardware.wifi.direct.xml:system/etc/permissions/android.hardware.wifi.direct.xml \
+ frameworks/native/data/etc/android.hardware.wifi.passpoint.xml:system/etc/permissions/android.hardware.wifi.passpoint.xml \
frameworks/native/data/etc/android.hardware.ethernet.xml:system/etc/permissions/android.hardware.ethernet.xml \
frameworks/native/data/etc/android.hardware.usb.accessory.xml:system/etc/permissions/android.hardware.usb.accessory.xml \
frameworks/native/data/etc/android.hardware.usb.host.xml:system/etc/permissions/android.hardware.usb.host.xml \
@@ -139,10 +140,6 @@
PRODUCT_COPY_FILES += \
device/google/dragon/audio_effects.conf:system/etc/audio_effects.conf
-# Vendor Interface Manifest
-PRODUCT_COPY_FILES += \
- $(LOCAL_PATH)/manifest.xml:vendor/manifest.xml
-
PRODUCT_AAPT_CONFIG := normal large xlarge hdpi xhdpi xxhdpi
PRODUCT_AAPT_PREF_CONFIG := xhdpi
diff --git a/dumpstate/DumpstateDevice.cpp b/dumpstate/DumpstateDevice.cpp
index a7a2361..c99a5ee 100644
--- a/dumpstate/DumpstateDevice.cpp
+++ b/dumpstate/DumpstateDevice.cpp
@@ -35,7 +35,7 @@
// Methods from ::android::hardware::dumpstate::V1_0::IDumpstateDevice follow.
Return<void> DumpstateDevice::dumpstateBoard(const hidl_handle& handle) {
- if (handle->numFds < 1) {
+ if (handle == nullptr || handle->numFds < 1) {
ALOGE("no FDs\n");
return Void();
}
diff --git a/overlay/frameworks/base/core/res/res/values/config.xml b/overlay/frameworks/base/core/res/res/values/config.xml
index 506f6ad..1b13f00 100644
--- a/overlay/frameworks/base/core/res/res/values/config.xml
+++ b/overlay/frameworks/base/core/res/res/values/config.xml
@@ -58,9 +58,6 @@
<!-- Boolean indicating whether the wifi chipset has dual frequency band support -->
<bool translatable="false" name="config_wifi_dual_band_support">true</bool>
- <!-- Boolean indicating whether Hotspot 2.0/Passpoint and ANQP queries is enabled -->
- <bool translatable="false" name="config_wifi_hotspot2_enabled">true</bool>
-
<!-- Boolean indicating whether the wifi chipset has background scan support -->
<bool translatable="false" name="config_wifi_background_scan_support">true</bool>
