search

gwolf / sistop

Fundamentos de sistemas operativos
68 stars 23 forks source link

codigo: Añade Makefile y hace los ejemplos completos compilables #14

Closed emilio closed 7 years ago

emilio commented 8 years ago

Especialmente preocupante era el siguiente fragmento de código:

pthread_create(&th[i], attr, filosofo, (int*) &i);

Donde se le pasa un puntero a una variable de la pila a un hilo, cuando esa variable cambia.

Cabe la posibilidad de que ese hilo no empiece a correr antes de que la siguiente iteración del bucle se produzca, lo que generaría que varios filósofos acabaran con la misma id.

He estandarizado el indentado a 4 espacios, pero no hay problema en cambiarlo a 2 si se quiere.

emilio commented 8 years ago

ping?

gwolf commented 8 years ago

Emilio Cobos Álvarez dijo [Tue, Mar 29, 2016 at 02:43:13PM -0700]:

ping?

¡Hola Emilio!

Se me debería caer la cara de vergüenza... No me olvido de este parche. Pero estoy demasiado saturado de trabajo por ahora. Lo voy a incorporar, pero necesito un par de semanas para salir de algunos compromisos.

Saludos,

emilio commented 8 years ago

Pero estoy demasiado saturado de trabajo por ahora. Lo voy a incorporar, pero necesito un par de semanas para salir de algunos compromisos.

No hay problema! :)

Un saludo: Emilio

gwolf commented 7 years ago

...Mucho, mucho tiempo pasó :-( Como sea, ¡muchas gracias! Incorporé la mayor parte de tus cambios en el commit 96b372d.

emilio commented 7 years ago

Whoops... NP! :)