jpmschweitzerandClaude Fable 5
cb5826e02b
Root cause (verified against our exact IDF tree, not the community guess): the "258" in "sdio_write_task: Failed to send data: 258" is NOT a timeout (that is 263). 258 = 0x102 = ESP_ERR_INVALID_ARG. On the ESP32-P4, block- mode CMD53 writes require the SOURCE buffer to be 64-byte (cache-line) aligned; the IDF sdmmc driver rejects a misaligned source with INVALID_ARG BEFORE any bus activity. esp_hosts write loop then declares "Unrecoverable host sdio state" and reboots the whole P4. The audio TX payload is not 64-aligned, so streaming mic audio wedged on the very FIRST frame (which is exactly what we saw: listening -> instant Failed to send -> reboot). This also explains why buffer/queue/clock/retry tuning all did nothing: the write never reached the bus. And why our symptom was instant, not after ~100 writes (the community block-mode-desync theory) — it is the first misaligned buffer, every time. Fix: vendored esp_hosted 2.12.11 as an editable local component (overrides the registry copy) and bounce a misaligned TX payload through one aligned DMA scratch buffer in hosted_sdio_write_block (port_esp_hosted_host_sdio.c). TX is serialized by the bus lock so a single static bounce buffer is safe; freed in hosted_sdio_deinit. Host-only change — no C6 reflash. VERIFIED ON HARDWARE (autonomous self-test): 40s of continuous mic-audio upstream streaming — the traffic that previously wedged on the first frame — ran clean, zero timeouts, zero reboots. A guarded SDIO_TX_SELFTEST harness is kept (compiled out) for future SDIO stress testing. Credit: root cause + patch designed via multi-agent investigation; the precise 258=INVALID_ARG decode (correcting the upstream community timeout assumption) came from checking our actual esp_err.h. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
87 lines
3.2 KiB
Protocol Buffer
87 lines
3.2 KiB
Protocol Buffer
/*
|
|
* Copyright (c) 2021, the protobuf-c authors.
|
|
* All rights reserved.
|
|
*
|
|
* Redistribution and use in source and binary forms, with or without
|
|
* modification, are permitted provided that the following conditions are
|
|
* met:
|
|
*
|
|
* * Redistributions of source code must retain the above copyright
|
|
* notice, this list of conditions and the following disclaimer.
|
|
*
|
|
* * Redistributions in binary form must reproduce the above
|
|
* copyright notice, this list of conditions and the following disclaimer
|
|
* in the documentation and/or other materials provided with the
|
|
* distribution.
|
|
*
|
|
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
|
|
* "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
|
|
* LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
|
|
* A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
|
|
* OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
|
|
* SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
|
|
* LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
|
|
* DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
|
|
* THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
|
|
* (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
|
|
* OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
|
|
*/
|
|
|
|
syntax = "proto2";
|
|
import "google/protobuf/descriptor.proto";
|
|
|
|
// We never need to generate protobuf-c.pb-c.{c,h}, the options are used
|
|
// only by protoc-gen-c, never by the protobuf-c runtime itself
|
|
option (pb_c_file).no_generate = true;
|
|
|
|
message ProtobufCFileOptions {
|
|
// Suppresses pb-c.{c,h} file output completely.
|
|
optional bool no_generate = 1 [default = false];
|
|
|
|
// Generate helper pack/unpack functions?
|
|
// For backwards compatibility, if this field is not explicitly set,
|
|
// only top-level message pack/unpack functions will be generated
|
|
optional bool gen_pack_helpers = 2 [default = true];
|
|
|
|
// Generate helper init message functions?
|
|
optional bool gen_init_helpers = 3 [default = true];
|
|
|
|
// Use const char * instead of char * for string fields
|
|
optional bool const_strings = 4 [default = false];
|
|
|
|
// For oneof fields, set ProtobufCFieldDescriptor name field to the
|
|
// name of the containing oneof, instead of the field name
|
|
optional bool use_oneof_field_name = 5 [default = false];
|
|
|
|
// Overrides the package name, if present
|
|
optional string c_package = 6;
|
|
}
|
|
|
|
extend google.protobuf.FileOptions {
|
|
optional ProtobufCFileOptions pb_c_file = 1019;
|
|
}
|
|
|
|
message ProtobufCMessageOptions {
|
|
// Overrides the parent setting only if present
|
|
optional bool gen_pack_helpers = 1 [default = false];
|
|
|
|
// Overrides the parent setting only if present
|
|
optional bool gen_init_helpers = 2 [default = true];
|
|
|
|
// Reserved base message field name
|
|
optional string base_field_name = 3 [default = "base"];
|
|
}
|
|
|
|
extend google.protobuf.MessageOptions {
|
|
optional ProtobufCMessageOptions pb_c_msg = 1019;
|
|
}
|
|
|
|
message ProtobufCFieldOptions {
|
|
// Treat string as bytes in generated code
|
|
optional bool string_as_bytes = 1 [default = false];
|
|
}
|
|
|
|
extend google.protobuf.FieldOptions {
|
|
optional ProtobufCFieldOptions pb_c_field = 1019;
|
|
}
|