afilgueira / hilolay Goto Github PK

Así como está y según las instrucciones del repo, no compila en la VM de Lubuntu de la cátedra (la que tiene GUI).
Error: declaración de una variable dentro de la condición del for (Sólo se permite en C99)

ults.c: In function ‘test’:
ults.c:5:5: error: ‘for’ loop initial declarations are only allowed in C99 mode
     for (int i = 0; i < 20; i++) {
     ^
ults.c:5:5: note: use option -std=c99 or -std=gnu99 to compile your code
ults.c: In function ‘main’:
ults.c:18:5: error: ‘for’ loop initial declarations are only allowed in C99 mode
     for(int i=0; i < 5; i++) {
     ^

Viendo los "yield"s y quizas alguna cosa mas (las e/s? algo asi?) estaria bueno trackear los tiempos y poder mostrar el gantt de lo que se ejecuto

https://github.com/afilgueira/ults/blob/3fb87b6887ad80c554931dddae49ef326ebc040c/hilolay/hilolay.c#L91-L98

Alternatives:

    for (new_ult = &ults[0];new_ult->state != FREE; new_ult++) {
        if (new_ult == &ults[MAX_ULTS]) {
            lib_log("Cannot create a new ULTs!");
            return ERROR_TOO_MANY_ULTS;
        }
    }

    for (new_ult = &ults[0]; new_ult != &ults[MAX_ULTS] && new_ult->state != FREE; new_ult++);
    if (new_ult == &ults[MAX_ULTS]) {
        lib_log("Cannot create a new ULTs!");
        return ERROR_TOO_MANY_ULTS;
    }

    new_ult = &ults[0];
    while (new_ult != &ults[MAX_ULTS] && new_ult->state != FREE){
       new_ult++;
    }
    if (new_ult == &ults[MAX_ULTS]) {
        lib_log("Cannot create a new ULTs!");
        return ERROR_TOO_MANY_ULTS;
    }

Hoy en dia no la soporta. Chequear esto

Linkeada a #6

La funcion de init deberia permitir elegir el algoritmo y alguna cosa mas de configuracion

Que hagan yield al finalizarlas. Me gustaria tener una funcion generica, que reciba otra funcion por parametro, y de ahi descomponer en un set de 8-10 funciones "nuestras" que la usen

Esto es muy amplio, lo se... veamos que onda

Actualmente usa el vector de TCBs como "cola" y busca asi el siguiente que este "ready"

La planificacion actual chequea si no hay otros ults en ready. Deberiamos evitar eso, y si es necesario hacer un cambio de contexto "con el mismo", en favor de simplificar el codigo

Depende de #8
La biblioteca podría "ignorar" la señal (y dejarla encolada por ej) mientras nosotros "deshabilitamos las interrupciones"

Veamos que se puede hacer con esto