Считать лог. Опции

[Jenya7]

я логирую данные во внешнюю память.
размерность памяти 8 бит. каждая запись 16 бит то есть каждая запись берет 2 байта. текущий адрес записи я храню в глобальной переменной - param1.logNum.
и тут я задумался над оптимальным алгоритмом - считать последних n записей. причем мне эти записи надо послать пакетами по 18 байт каждый пакет.
Кое что я набросал.

Код

void BLUENRG_SendLog(void)  //send n deltas from the last
{

    uint32_t chank = 18;  //DELTA_SIZE * AVAILABLE_BUF_SIZE;
    uint32_t last_log_addr = param1.logNum * DELTA_SIZE;
    uint32_t packet_size;
    uint32_t tx_num=0;
    uint32_t current_addr = last_log_addr - (chank * tx_num);

    //check if the overflow occurred
    while (current_addr > START_LOG_ADRESS)
    {
               packet_size = chank;

               uint8_t buf[packet_size];

               LOGGER_Readlog(current_addr, packet_size, buf);

        SendBuf(buf);

        tx_num++;
        
        current_addr = last_log_addr - (chank * tx_num);

        
    }
    // send the rest of data ???

}

Но мне это не нравиться. Может лучше отсчитать и начать с младшего адреса? Проще и не надо проверять ушел ли я за границы начального адреса.

Сообщение отредактировал Jenya7 - Oct 19 2015, 07:24

[Jenya7]

Код

uint32_t chunk = CHUNK_SIZE;
uint32_t last_log_addr = (logNum * DELTA_SIZE) + START_LOG_ADRESS;
uint32_t packet_size;
int32_t current_addr = last_log_addr-chunk;  
      
    while (current_addr > START_LOG_ADRESS)
    {
        if(current_addr - START_LOG_ADRESS >= chunk)
            packet_size = chunk;
        else
            packet_size = current_addr - START_LOG_ADRESS;

        uint8_t buf[packet_size];

        LOGGER_Read(current_addr,packet_size,buf);

        SendBuf(buf);

        current_addr -= chunk;
    }

если не лень функцию можно сделать абсолютно генерик стайл без привязки к какой либо железке кольцевая она там или нет.

Сообщение отредактировал Jenya7 - Oct 19 2015, 14:29

[Alexashka]

Код

...
  int32_t current_addr = last_log_addr-chunk;  
...
if(current_addr - START_LOG_ADRESS >= chunk)
            packet_size = chunk;
        else
            packet_size = current_addr - START_LOG_ADRESS;
...
         current_addr -= chunk;
     }

если не лень функцию можно сделать абсолютно генерик стайл без привязки к какой либо железке кольцевая она там или нет.

Закольцовка имеет смысл если записи идут и идут, а считывание всё откладывается. Тогда закольцевав буфер сохраняем последние записи (ессно затерев самые старые из них).
В конце наверное нужно вычитать не chunk, а packet_size (на случай если блок был укороченный). И похоже нужно смотреть разницу не между текущим адресом и стартовым, а между конечным и стартовым (для примера можно посмотреть что будет если в буфере было 5 записей а считывается 18 -при первом же заходе в функцию будем сразу из нее вылетать, т.к current_addr < START_LOG_ADRESS). Не понятно зачем нужна переменная chunk, если она используется только как копия константы CHUNK

[Jenya7]

Закольцовка имеет смысл если записи идут и идут, а считывание всё откладывается. Тогда закольцевав буфер сохраняем последние записи (ессно затерев самые старые из них).
В конце наверное нужно вычитать не chunk, а packet_size (на случай если блок был укороченный). И похоже нужно смотреть разницу не между текущим адресом и стартовым, а между конечным и стартовым (для примера можно посмотреть что будет если в буфере было 5 записей а считывается 18 -при первом же заходе в функцию будем сразу из нее вылетать, т.к current_addr < START_LOG_ADRESS). Не понятно зачем нужна переменная chunk, если она используется только как копия константы CHUNK

спасибо.