使用Threads进行分段错误(核心转储)

时间:2014-12-11 02:30:46

标签: c multithreading

当我尝试将两个矩阵与线程相乘时,我收到了分段错误(核心转储)错误 我在C中制作了这个程序并且在TAM< = 200时工作正常但是当我插入一个高值时,线程不起作用。 这是代码:

#include <pthread.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <time.h>
#include "tamanio.h"
#define NUM_THREADS 4

int matrizA[TAM][TAM];
int matrizB[TAM][TAM];
int matrizR[TAM][TAM];
int x,y,z;
int m=TAM,o=TAM,p=TAM;
clock_t start_t, end_t;
double total_t;

//THREADS
void *mul(void *threadid){

long tid; 
tid = (long) threadid;
int OTHERCONTROL=tid*TAM/NUM_THREADS;
int VARCONTROLTHREAD=(tid+1)*TAM/NUM_THREADS;
printf("Iniciando hilo %ld ...\n", tid);
//multiply
for(x=OTHERCONTROL; x<VARCONTROLTHREAD; x++){
    for(y=0; y<o; y++){
        for(z=0;z<p;z++){
        matrizR[x][y]+=matrizA[x][z]*matrizB[z][y];
        }
    }
}

printf("Hilo %ld terminado.\n", tid);

pthread_exit((void*) threadid);}


int main (int argc, char **argv){
//variables
FILE *entrada;
char *nombre;

//Read 
    if (argc > 1){
        nombre =argv[1];        
        entrada =fopen(nombre, "r");
        if(entrada == NULL){
            printf("Hubo problemas al intentar abrir el archivo de entrada\n");
            exit(1);
        }
    }
// MatrizA
for(x=0; x<m; x++){
        for(y=0; y<o; y++){
            fscanf(entrada,"%d\t",&(matrizA[x][y]));
        }

    }
// MatrizB
for(x=0; x<m; x++){
        for(y=0; y<o; y++){
            fscanf(entrada,"%d\t",&(matrizB[x][y]));
        }

    }

    fclose(entrada);
    printf("Se termina la lectura de datos\n");

   start_t=clock();

//**THREADS**
    pthread_t threads[NUM_THREADS];
    int rc;
    long t;
    void *status;

    for(t=0;t<NUM_THREADS;t++){
        printf("creando hilo%ld\n",t);
        rc=pthread_create(&threads[t],NULL,mul,(void *)t);
        if(rc){
            printf("ERROR\n");
            exit(-1);
        }
    }


    for(t=0; t<NUM_THREADS; t++) {
      rc = pthread_join(threads[t], &status);
      if (rc) {
         printf("ERROR; El codigo de retorno de pthread_join() es %d\n", rc);
         exit(-1);
         }
      printf("Main: se completo la union con el hilo %ld regresando un estado de %ld\n",t,(long)status);
      }
    end_t=clock();
    total_t = (double) (end_t - start_t) / CLOCKS_PER_SEC;
    printf("Tiempo de ejecucion: %f \n",total_t);

    printf("END MAIN\n");
    pthread_exit(NULL);
    return 0;
}

tamanio.h只包含一个名为TAM的变量

#define TAM 1000

有问题吗?

1 个答案:

答案 0 :(得分:2)

您不应将x, y, z作为全局变量。如果你这样做,他们将在你的线程中共享,并且可能发生奇怪的事情。您应该将这些变量声明为每个函数中的局部变量。您还应该考虑将m, o, p替换为本地或仅TAM,尽管这些永远不会分配给它们,因此它不那么重要。