Evolution-x/hardware_interfaces
1
0
Fork 0
mirror of https://github.com/Evolution-X/hardware_interfaces synced 2026-02-02 02:42:35 +00:00
Code Issues Packages Projects Releases Wiki Activity

Bluetooth: Sleep instead of dying on EOF

Browse Source 
am: 0d63f8af7c

Change-Id: I8f86685ba2000817dcfbdf566449826b121936dd
This commit is contained in:
Myles Watson
2018-01-10 01:38:39 +00:00
committed by android-build-merger
parent fc0e8c6f48 0d63f8af7c
commit e63d6a08ce
2 changed files with 18 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
bluetooth/1.0/default/h4_protocol.cc
View File

@@ -71,8 +71,10 @@ void H4Protocol::OnDataReady(int fd) {
ssize_t bytes_read = TEMP_FAILURE_RETRY(read(fd, buffer, 1)); ssize_t bytes_read = TEMP_FAILURE_RETRY(read(fd, buffer, 1));
if (bytes_read != 1) { if (bytes_read != 1) {
if (bytes_read == 0) { if (bytes_read == 0) {
LOG_ALWAYS_FATAL("%s: Unexpected EOF reading the packet type!", // This is only expected if the UART got closed when shutting down.
__func__); ALOGE("%s: Unexpected EOF reading the packet type!", __func__);
sleep(5); // Expect to be shut down within 5 seconds.
return;
} else if (bytes_read < 0) { } else if (bytes_read < 0) {
LOG_ALWAYS_FATAL("%s: Read packet type error: %s", __func__, LOG_ALWAYS_FATAL("%s: Read packet type error: %s", __func__,
strerror(errno)); strerror(errno));

21
bluetooth/1.0/default/hci_packetizer.cc
View File

@@ -56,9 +56,13 @@ void HciPacketizer::OnDataReady(int fd, HciPacketType packet_type) {
ssize_t bytes_read = TEMP_FAILURE_RETRY( ssize_t bytes_read = TEMP_FAILURE_RETRY(
read(fd, preamble_ + bytes_read_, read(fd, preamble_ + bytes_read_,
preamble_size_for_type[packet_type] - bytes_read_)); preamble_size_for_type[packet_type] - bytes_read_));
if (bytes_read <= 0) { if (bytes_read == 0) {
LOG_ALWAYS_FATAL_IF((bytes_read == 0), // This is only expected if the UART got closed when shutting down.
"%s: Unexpected EOF reading the header!", __func__); ALOGE("%s: Unexpected EOF reading the header!", __func__);
sleep(5); // Expect to be shut down within 5 seconds.
return;
}
if (bytes_read < 0) {
LOG_ALWAYS_FATAL("%s: Read header error: %s", __func__, LOG_ALWAYS_FATAL("%s: Read header error: %s", __func__,
strerror(errno)); strerror(errno));
} }
@@ -80,10 +84,13 @@ void HciPacketizer::OnDataReady(int fd, HciPacketType packet_type) {
fd, fd,
packet_.data() + preamble_size_for_type[packet_type] + bytes_read_, packet_.data() + preamble_size_for_type[packet_type] + bytes_read_,
bytes_remaining_)); bytes_remaining_));
if (bytes_read <= 0) { if (bytes_read == 0) {
LOG_ALWAYS_FATAL_IF((bytes_read == 0), // This is only expected if the UART got closed when shutting down.
"%s: Unexpected EOF reading the payload!", ALOGE("%s: Unexpected EOF reading the payload!", __func__);
__func__); sleep(5); // Expect to be shut down within 5 seconds.
return;
}
if (bytes_read < 0) {
LOG_ALWAYS_FATAL("%s: Read payload error: %s", __func__, LOG_ALWAYS_FATAL("%s: Read payload error: %s", __func__,
strerror(errno)); strerror(errno));
} }