SegFault в программе Cache Sim C - PullRequest
Новая
       9

SegFault в программе Cache Sim C

0 голосов
/ 16 декабря 2011

РЕДАКТИРОВАТЬ: так что вам, ребята, легче найти, где в моем коде, я думаю, происходит segfault, загляните в прокомментированный раздел под названием /* Cache Structure */

EDIT2: допустил огромную ошибку с GDB в правильном выводекогда он отклоняется от gdb: 

(gdb) run
Starting program: /.autofs/ilab/ilab_users/petejodo/Assignment4/a.out 
1
/.autofs/ilab/ilab_users/petejodo/Assignment4/a.out

Program received signal SIGSEGV, Segmentation fault.
0x00449453 in strlen () from /lib/libc.so.6
(gdb)

EDIT3: после обращения к тактике n00b, такой как печать строк, я обнаружил, что он печатает раньше: 

file = fopen(purefile, "r");
    if (file == 0){
        printf ("Could not find file!\n");
        return 0;
    }

, но не печатает "test"У меня под ним есть

ORIGINAL

Поэтому, проверив все потоки на segfault и используя gdb, я не могу понять, почему моя программа segfaulting в этой строке.Я пишу кеш-симулятор (в настоящее время в моем коде есть только сквозная запись), и я не включил свои методы, потому что они в порядке, это основной метод. 

int main(int argc, char **argv) {
    FILE* file;

    /* Counter variables */
    int i;
    int j;

    /* Helper Variables */
    int setAdd;
    int totalSet;
    int trash;
    int size;
    int extra;
    char rw;

    /* Necessary Character Arrays */
    char hex[100];
    char bin[100];
    char origTag[100];
    char bbits[100];
    char sbits[100];
    char tbits[100];

    /* Cache Info Variables */
    int setNumber = 4096; /* cacheSize/blockSize : (16,384/4) */
    int setBits = 12; /* log(setNumber)/log(2) : (log(4096)/log(2)) */
    int tagSize = 18; /* 32-(blockBits + setBits) **blockBits = log(blockSize)/log(2)** : (32 - (2 + 12) */

    /* Results */
    int cacheHit = 0;
    int cacheMiss = 0;
    int write = 0;
    int read = 0;

    /* Cache Structure */
    tempLine cache[4096];

    char* style;
    char* purefile;
    if (strcmp(argv[1], "-h")==0) 
    {
        puts("Usage: sim <write policy> <trace file>");
        return 0;
    } 
    style = argv[1];
    purefile = argv[2];
    file = fopen(purefile, "r");/* HYPOTHESIZED SEGFAULT HERE */
    if (file == 0){
        printf ("Could not find file!\n");
        return 0;
    }
    printf("test1");
    /* Setting Structure Default Values */
    for(i = 0; i < setNumber; i++)
    {
        cache[i].tag = (char *)malloc(sizeof(char)*(tagSize + 1));
        for(j = 0; j < tagSize; j++)
        {
            cache[i].tag[j] = '0';
        }
        cache[i].valid = 0;
    }

    /* Main Loop */
    while(fgetc(file) != '#')
    {
        setAdd = 0;
        totalSet = 0;

        fseek(file, -1, SEEK_CUR);
        fscanf(file, "%d: %c %s\n", &trash, &rw, origTag);

        /* Cutting off '0x' off from address '0x00000000' and adding 0's if necessary */
        size = strlen(origTag);
        extra = (10 - size);
        for(i = 0; i < extra; i++)
            hex[i] = '0';
        for(i = extra, j = 0; i < (size-(2-extra)); i++, j++)
            hex[i] = origTag[j + 2];

        hex[8] = '\0';

        hex2bin(hex, bin);

        split(bin, bbits, sbits, tbits);

        /* Changing cArray into int */
        for(i = 0, j = (setBits - 1); i < setBits; i++, j--)
        {
            if (sbits[i] == '1')
                setAdd = 1;
            if (sbits[i] == '0')
                setAdd = 0;
            setAdd = setAdd * pow(2, j);
            totalSet += setAdd;
        }

        /* Calculating Hits and Misses */
        if (cache[totalSet].valid == 0)
        {
            cache[totalSet].valid = 1;
            strcpy(cache[totalSet].tag, tbits);
        }

        if ((cache[totalSet].valid == 1) && (strcmp(cache[totalSet].tag, tbits) == 0))
        {
            /* HIT */
            if (rw == 'W')
            {
                cacheHit++;        
                write++;
            }
            if (rw == 'R')
                cacheHit++;    
         }
         else
         {
             /* MISS */
            if (rw == 'R')
            {
                cacheMiss++;
                read++;
            }
            if (rw == 'W')
            {
                cacheMiss++;
                read++;
                write++;
            }
            cache[totalSet].valid = 1;
            strcpy(cache[totalSet].tag, tbits);
        }
        /* End Calculations */  
    }
    printResult(cacheHit, cacheMiss, read, write);

    return 0;
}

И что я получилиз GDB было это: 

**INCORRECT**

(gdb) run
Starting program: /.autofs/ilab/ilab_users/petejodo/Assignment4/a.out 

Program received signal SIGSEGV, Segmentation fault.
0x080489b6 in main (argc=1, argv=0xbfffe954) at sim.c:128
128             if (strcmp(argv[1], "-h")==0)
(gdb) bt

#0  0x080489b6 in main (argc=1, argv=0xbfffe954) at sim.c:128

Я как бы потерялся, любая помощь будет отличной.Спасибо!

О, а также проверяя значение argv[1] это NULL или 0x0, потому что я думаю, что это из-за этого.argv[1] должен содержать либо флаг справки, либо запись, либо обратную запись, я просто еще не указал, проверять ли запись или обратную запись.

1 Ответ

1 голос
/ 16 декабря 2011

Вам нужно проверить, если argc > 1, если вы хотите получить доступ к argv[1].Программа всегда получает имя, по которому она была вызвана (например, ./a.out в вашем случае) как argv[0], поэтому argc >= 1 всегда имеет значение true.Чтобы получить доступ к первому реальному аргументу - то есть ко второму элементу argv - вам нужно проверить, есть ли хотя бы два аргумента.Тем не менее, проверка наличия > n элементов при доступе к argv[n] более удобна для чтения, поскольку оба числа одинаковы.

Вот пример того, как изменить код: 

if (argc < 3 || (argv > 1 && strcmp(argv[1], "-h")==0))
{
    puts("Usage: sim <write policy> <trace file>");
    return 0;
}

В этом случае вы увидите ошибку, если недостаточно аргументов, чтобы иметь действительные элементы argv[1] и argv[2] или если первый аргумент -h (в случае, если кто-то ставит вещи после -h - иначе это 'будет обрабатываться проверкой «слишком мало аргументов»).

Кстати, вы должны взглянуть на getopt() - это делает анализ аргументов / переключателей намного проще и чище.

...