饱和输入造成的数据丢失 #983

五角星

请只提交最新的主要或开发分支的错误。您可以检查启动消息中的版本号并将其与grbl.h中的版本进行比较

请回答以下问题。

您使用的是什么版本的固件？

版本 1.3a
构建 20210424

问题是否可重复？

是的

什么情况下会出现bug？

每当 i/o 输入速率超过 SERIAL 以外的客户端上的消耗速率时。

这个问题似乎是由于 Serial.cpp 中的 clientCheckTask 是如何运作的

        while ((client = getClientChar(&data)) != CLIENT_ALL) {
            if (is_realtime_command(data)) {
                execute_realtime_command(static_cast<Cmd>(data), client);
            } else {
                    // ...
                    client_buffer[client].write(data);
                    // ...
            }
        }  // if something available

该循环不断获取客户端数据，然后尝试写入每个客户端的 InputBuffer，而不考虑可用容量。

当有可用输入但没有输出容量时，这会导致数据丢失。

这种设计的原因似乎是允许提取实时命令。

如果我们看一下 getClientChar

static uint8_t getClientChar(uint8_t* data) {
    int res;
#ifdef REVERT_TO_ARDUINO_SERIAL
    if (client_buffer[CLIENT_SERIAL].availableforwrite() && (res = Serial.read()) != -1) {
#else
    if (client_buffer[CLIENT_SERIAL].availableforwrite() && (res = Uart0.read()) != -1) {
#endif
        *data = res;
        return CLIENT_SERIAL;
    }
    // ...
}