Помощь - Поиск - Пользователи - Календарь
Полная версия этой страницы: Зависания на printf
Форум разработчиков электроники ELECTRONIX.ru > Сайт и форум > В помощь начинающему > ARM, 32bit
athlon64
Процессор SAM7X512, IAR 5.4.
В проекте используются порты COM0, COM1, PIT, TWI. Основой для проекта был пример из IAR "basic-usart-hw-handshaking-project". Приём в порты COM0, COM1 сделан через регистры побайтно, вывод - через DMA. Нужно для вывода отладочного лога в DBGU форматированный вывод.

При инициализации DBGU:
Код
  PIO_Configure  (DBGU_pins, PIO_LISTSIZE(DBGU_pins));
  DBGU_Configure (DBGU_STANDARD, 115200, BOARD_MCK);

и попытке вывести любую строку при помощи
Код
printf("TEST");
процессор зависает.
То же самое при выводе с помощью
Код
TRACE_INFO("TEST");
и соответствующей инициализацией из trace.c порта DBGU.
При этом в порт ничего не попадает.
А вот
Код
DBGU_PutChar(byte);
работает прекрасно.

Куда смотреть?
jorikdima
В размер стэка смотреть. Перво-наперво попробовать увеличить.
athlon64
Ошибочка вышла - если выводим только текст, не виснет.
Так виснет:
Код
printf("-- TEST --\n\r");

а так - нет, но в порт ничего не выводится:
Код
printf("-- TEST --");

Цитата(jorikdima @ Jul 5 2010, 13:40) *
В размер стэка смотреть. Перво-наперво попробовать увеличить.

Если можно, для чайников - где он изменяется?
athlon64
Увеличил размер стека в icf-файле с 4Кб до 8Кб. Не полегчало
athlon64
Решения так и не нашёл, пока вывожу в цикле посимвольно пару строк раз в секунду.
Хотелось бы выводить полноценный лог в порт DBGU, для чего форматированный вывод очень нужен.
Может какие то принципиальные куски кода привести?
MALLOY2
Попробуйте так
Код
char format_buff[512];        //Сюда будет копироваться отформатированная строка

void out_fdbg(const char *args, ...)
{
   u32_t len;
   va_list ap;
   va_start(ap, args);
   len = vsprintf(format_buff,args,ap);
   va_end(ap);
   Uart1_SendData(format_buff,len);
}
athlon64
Цитата(MALLOY2 @ Jul 15 2010, 11:44) *
Попробуйте так
Код
char format_buff[512];        //Сюда будет копироваться отформатированная строка

void out_fdbg(const char *args, ...)
{
   u32_t len;
   va_list ap;
   va_start(ap, args);
   len = vsprintf(format_buff,args,ap);
   va_end(ap);
   Uart1_SendData(format_buff,len);
}

IAR ругается на va_list, va_start, va_end. Где они лежат не понял sad.gif
Что кроме stdio.h нужно инклюдить?
MALLOY2
Цитата
#include <stdarg.h>


Смотрите, размер format_buff полностью на вашей ответственности, если результирующая строка не влезет в него сами виноваты.
aaarrr
Цитата(MALLOY2 @ Jul 15 2010, 11:47) *
Смотрите, размер format_buff полностью на вашей ответственности, если результирующая строка не влезет в него сами виноваты.

Поэтому проще использовать vsnprintf вместо vsprintf.
MALLOY2
Согласен, забыл про нее, давно пользуюсь своей функцией
athlon64
Цитата(MALLOY2 @ Jul 15 2010, 13:47) *
Смотрите, размер format_buff полностью на вашей ответственности, если результирующая строка не влезет в него сами виноваты.
это понятно

так и не понял как пользоваться
что такое args и ap в вашем коде?
MALLOY2
Это расписано почти в каждой книжке по С, stdarg.h
athlon64
Код
void DBGU_fprint(const char *args, ...)
{
  char format_buff[512];        //Сюда будет копироваться отформатированная строка
  unsigned short int len,i_len;
  
  va_list ap;
  va_start(ap, args);
  len = vsprintf(format_buff, args, ap);
  va_end(ap);
  
  for (i_len=0; i_len < len; i_len++)
    DBGU_PutChar(format_buff[i_len]);
}

...
// Вызов:
  DBGU_fprint("TEST\n");

Во, заработало!! Спасибо
Пока этот вариант более чем устраивает smile.gif
aaarrr
Цитата(athlon64 @ Jul 15 2010, 13:45) *
Пока этот вариант более чем устраивает smile.gif

Сделайте лучше:
Код
len = vsnprintf(format_buff, sizeof(format_buff), args, ap);
MALLOY2
format_buff нестоит делать локальной!!!!! стека не напасетесь!!!!

Стоит также заменить vsprintf на vsnprintf, но есть НО, почитайте о результатах которые возвращет эта функция, она может вернуть -1, для вашего кода это будет баг. Надо перед отправкой в порт проверить значение len и тип len сменить на знаковый.
athlon64
Цитата(MALLOY2 @ Jul 15 2010, 18:43) *
format_buff нестоит делать локальной!!!!! стека не напасетесь!!!!
Сделал

Цитата(MALLOY2 @ Jul 15 2010, 18:43) *
Стоит также заменить vsprintf на vsnprintf, но есть НО, почитайте о результатах которые возвращет эта функция, она может вернуть -1, для вашего кода это будет баг. Надо перед отправкой в порт проверить значение len и тип len сменить на знаковый.

Думаю, теперь верно?
Код
void DBGU_fprint(const char *args, ...)
{
  char len,i_len;
  
   va_list ap;
   va_start(ap, args);
   len = vsnprintf(format_buff, sizeof(format_buff), args, ap);
   va_end(ap);
  
   if (len > 0)
     for (i_len=0; i_len < len; i_len++)
       DBGU_PutChar(format_buff[i_len]);
}
aaarrr
Цитата(athlon64 @ Jul 15 2010, 19:09) *
Думаю, теперь верно?

Нет. Еще char на int замените.

Вообще, старайтесь не использовать переменные не полной размерности там, где в этом нет необходимости.
athlon64
Цитата(aaarrr @ Jul 15 2010, 21:18) *
Нет. Еще char на int замените.

Вообще, старайтесь не использовать переменные не полной размерности там, где в этом нет необходимости.

У меня буфер коротенький - 100 байт smile.gif мне этого достаточно
aaarrr
Цитата(athlon64 @ Jul 16 2010, 07:34) *
У меня буфер коротенький - 100 байт smile.gif мне этого достаточно

С int'ом код получится короче и быстрее, размер буфера ни при чем.
sergeeff
Цитата(athlon64 @ Jul 15 2010, 18:09) *
if (len > 0)


Код
if
тут совсем лишний
athlon64
Цитата(sergeeff @ Jul 16 2010, 16:35) *
Код
if
тут совсем лишний

в смысле ">0" лишний?
Код
if (len)


Цитата(aaarrr @ Jul 16 2010, 11:38) *
С int'ом код получится короче и быстрее, размер буфера ни при чем.

Хм, почему то всегда думал что с char экономится ОЗУ и код выполняется быстрее smile.gif
sergeeff
Цитата(athlon64 @ Jul 18 2010, 18:46) *
в смысле ">0" лишний?
Код
if (len)



Хм, почему то всегда думал что с char экономится ОЗУ и код выполняется быстрее smile.gif


Именно проверка >0 лишняя.

Надо не думать, а знать. ARM - 32-битный процессор.
aaarrr
Цитата(athlon64 @ Jul 18 2010, 19:46) *
в смысле ">0" лишний?
Код
if (len)

Нет, совсем лишний: при len <= 0 цикл не выполнится ни разу.

Цитата(athlon64 @ Jul 18 2010, 19:46) *
Хм, почему то всегда думал что с char экономится ОЗУ и код выполняется быстрее smile.gif

ОЗУ в данном случае задействовано не будет, под локальные переменные будут использованы регистры процессора. А так как они 32 битные, то на эмуляции 8 битности получите оверхед по размеру кода и скорости.
athlon64
Цитата(aaarrr @ Jul 18 2010, 22:14) *
Нет, совсем лишний: при len <= 0 цикл не выполнится ни разу.
Хм, логично smile.gif

Цитата(sergeeff @ Jul 18 2010, 22:11) *
Надо не думать, а знать. ARM - 32-битный процессор.
Ну что ж, учту smile.gif



defunct
Цитата(MALLOY2 @ Jul 15 2010, 08:44) *
Попробуйте так
Код
char format_buff[512];        //Сюда будет копироваться отформатированная строка

void out_fdbg(const char *args, ...)
{
....
}

А что, очень даже талантливо. ©
Сделать совершенно мертвую память "ни под что".. под один только dbg_printf...
По хорошему лучше вообще без доп буфера, чтобы putc выводил сразу туда куда надо.
sergeeff
Цитата(defunct @ Jul 20 2010, 02:06) *
А что, очень даже талантливо. ©
Сделать совершенно мертвую память "ни под что".. под один только dbg_printf...
По хорошему лучше вообще без доп буфера, чтобы putc выводил сразу туда куда надо.


Ловите:

Код
/*
    Copyright 2001, 2002 Georges Menie (www.menie.org)

    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU Lesser General Public License as published by
    the Free Software Foundation; either version 2 of the License, or
    (at your option) any later version.

    This program is distributed in the hope that it will be useful,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
    GNU Lesser General Public License for more details.

    You should have received a copy of the GNU Lesser General Public License
    along with this program; if not, write to the Free Software
    Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
*/

/*
    putchar is the only external dependency for this file,
    if you have a working putchar, just remove the following
    define. If the function should be called something else,
    replace outbyte(c) by your own function call.
*/
#define putchar(c) outbyte(c)

static void printchar(char **str, int c)
{
    extern int putchar(int c);
    if (str) {
        **str = c;
        ++(*str);
    }
    else (void)putchar(c);
}

#define PAD_RIGHT 1
#define PAD_ZERO 2

static int prints(char **out, const char *string, int width, int pad)
{
    register int pc = 0, padchar = ' ';

    if (width > 0) {
        register int len = 0;
        register const char *ptr;
        for (ptr = string; *ptr; ++ptr) ++len;
        if (len >= width) width = 0;
        else width -= len;
        if (pad & PAD_ZERO) padchar = '0';
    }
    if (!(pad & PAD_RIGHT)) {
        for (; width > 0; --width) {
            printchar (out, padchar);
            ++pc;
        }
    }
    for (; *string; ++string) {
        printchar (out, *string);
        ++pc;
    }
    for (; width > 0; --width) {
        printchar (out, padchar);
        ++pc;
    }

    return pc;
}

/* the following should be enough for 32 bit int */
#define PRINT_BUF_LEN 12

static int printi(char **out, int i, int b, int sg, int width, int pad, int letbase)
{
    char print_buf[PRINT_BUF_LEN];
    register char *s;
    register int t, neg = 0, pc = 0;
    register unsigned int u = i;

    if (i == 0) {
        print_buf[0] = '0';
        print_buf[1] = '\0';
        return prints (out, print_buf, width, pad);
    }

    if (sg && b == 10 && i < 0) {
        neg = 1;
        u = -i;
    }

    s = print_buf + PRINT_BUF_LEN-1;
    *s = '\0';

    while (u) {
        t = u % b;
        if( t >= 10 )
            t += letbase - '0' - 10;
        *--s = t + '0';
        u /= b;
    }

    if (neg) {
        if( width && (pad & PAD_ZERO) ) {
            printchar (out, '-');
            ++pc;
            --width;
        }
        else {
            *--s = '-';
        }
    }

    return pc + prints (out, s, width, pad);
}

static int print(char **out, int *varg)
{
    register int width, pad;
    register int pc = 0;
    register char *format = (char *)(*varg++);
    char scr[2];

    for (; *format != 0; ++format) {
        if (*format == '%') {
            ++format;
            width = pad = 0;
            if (*format == '\0') break;
            if (*format == '%') goto out;
            if (*format == '-') {
                ++format;
                pad = PAD_RIGHT;
            }
            while (*format == '0') {
                ++format;
                pad |= PAD_ZERO;
            }
            for (; *format >= '0' && *format <= '9'; ++format) {
                width *= 10;
                width += *format - '0';
            }
            if( *format == 's' ) {
                register char *s = *((char **)varg++);
                pc += prints (out, s?s:"(null)", width, pad);
                continue;
            }
            if( *format == 'd' ) {
                pc += printi (out, *varg++, 10, 1, width, pad, 'a');
                continue;
            }
            if( *format == 'x' ) {
                pc += printi (out, *varg++, 16, 0, width, pad, 'a');
                continue;
            }
            if( *format == 'X' ) {
                pc += printi (out, *varg++, 16, 0, width, pad, 'A');
                continue;
            }
            if( *format == 'u' ) {
                pc += printi (out, *varg++, 10, 0, width, pad, 'a');
                continue;
            }
            if( *format == 'c' ) {
                /* char are converted to int then pushed on the stack */
                scr[0] = *varg++;
                scr[1] = '\0';
                pc += prints (out, scr, width, pad);
                continue;
            }
        }
        else {
        out:
            printchar (out, *format);
            ++pc;
        }
    }
    if (out) **out = '\0';
    return pc;
}

/* assuming sizeof(void *) == sizeof(int) */

int qprintf(const char *format, ...)
{
    register int *varg = (int *)(&format);
    return print(0, varg);
}

int qsprintf(char *out, const char *format, ...)
{
    register int *varg = (int *)(&format);
    return print(&out, varg);
}

#ifdef TEST_PRINTF
int main(void)
{
    char *ptr = "Hello world!";
    char *np = 0;
    int i = 5;
    unsigned int bs = sizeof(int)*8;
    int mi;
    char buf[80];

    mi = (1 << (bs-1)) + 1;
    printf("%s\n", ptr);
    printf("printf test\n");
    printf("%s is null pointer\n", np);
    printf("%d = 5\n", i);
    printf("%d = - max int\n", mi);
    printf("char %c = 'a'\n", 'a');
    printf("hex %x = ff\n", 0xff);
    printf("hex %02x = 00\n", 0);
    printf("signed %d = unsigned %u = hex %x\n", -3, -3, -3);
    printf("%d %s(s)%", 0, "message");
    printf("\n");
    printf("%d %s(s) with %%\n", 0, "message");
    sprintf(buf, "justif: \"%-10s\"\n", "left"); printf("%s", buf);
    sprintf(buf, "justif: \"%10s\"\n", "right"); printf("%s", buf);
    sprintf(buf, " 3: %04d zero padded\n", 3); printf("%s", buf);
    sprintf(buf, " 3: %-4d left justif.\n", 3); printf("%s", buf);
    sprintf(buf, " 3: %4d right justif.\n", 3); printf("%s", buf);
    sprintf(buf, "-3: %04d zero padded\n", -3); printf("%s", buf);
    sprintf(buf, "-3: %-4d left justif.\n", -3); printf("%s", buf);
    sprintf(buf, "-3: %4d right justif.\n", -3); printf("%s", buf);

    return 0;
}

/*
* if you compile this file with
*   gcc -Wall $(YOUR_C_OPTIONS) -DTEST_PRINTF -c printf.c
* you will get a normal warning:
*   printf.c:214: warning: spurious trailing `%' in format
* this line is testing an invalid % at the end of the format string.
*
* this should display (on 32bit int machine) :
*
* Hello world!
* printf test
* (null) is null pointer
* 5 = 5
* -2147483647 = - max int
* char a = 'a'
* hex ff = ff
* hex 00 = 00
* signed -3 = unsigned 4294967293 = hex fffffffd
* 0 message(s)
* 0 message(s) with %
* justif: "left      "
* justif: "     right"
*  3: 0003 zero padded
*  3: 3    left justif.
*  3:    3 right justif.
* -3: -003 zero padded
* -3: -3   left justif.
* -3:   -3 right justif.
*/

#endif
defunct
Цитата(sergeeff @ Jul 20 2010, 09:19) *
Ловите:

Вот это дело! Спасибо за правильный и полный пример!
Для просмотра полной версии этой страницы, пожалуйста, пройдите по ссылке.
Invision Power Board © 2001-2025 Invision Power Services, Inc.