[so] [SO] Contestatie Tema3 Linux

[Ioana Ciornei]

Am adaugat inapoi lista. Te rog sa dai de fiecare data reply-all.

> Nu folosesc DIE la malloc in populate_page_table intrucat cele 2
> variabile sunt definite local in while.

Si cum afecteaza asta malloc-ul?

 Unde chiar am uitat sa
> folosesc DIE pe malloc este in save_memory_zone la
> mem_info->frame_table. In toate celelate cazuri folosesc si la malloc
> macro-ul DIE. Este destul de greu sa nu iti mai scape cate un apel de
> sistem neverificat, in toate celelate cazuri se vede ca mi-am format
> deprinderea de a verifica valorile de return.

Si sa nu uitam de vm_page_fault_handler.

Ioana

>
> Andrei
>
>
> 2017-05-19 11:11 GMT+03:00 Ioana Ciornei <ciorneiioana at gmail.com>:
>> Salut,
>>
>>
>>> Am uitat sa precizez id-ul de pe vmchecker: andrei.simion
>>>
>>>
>>> 2017-05-19 10:53 GMT+03:00 Andrei Simion-Constantinescu
>>> <andrei.simion.c at gmail.com>:
>>>> Buna ziua,
>>>>
>>>> In urma corectarii temei 3, am primit urmatoarea depunctare:
>>>> -0.2: nu sunt verificate valorile de retur ale funcțiilor; recomandăm
>>>> folosirea macro-ului DIE, prezent în scheletul de laborator
>>>>
>>>> In tema mea, pentru toate apelurile de sistem folosesc macro-ul DIE
>>>> (mkstemp, unlink,read, write, lseek, mmap, munmap, mprotect etc), nu
>>>> vad de unde ar putea sa provina depunctarea. Singurele valori de
>>>> return pe care nu le verific sunt w_get_exception_handler,
>>>> w_set_exception_handler, intrucat vmsim_init si vmsim_cleanup intorc
>>>> TRUE/FALSE, iar daca cele 2 apeluri dau fail cele 2 functii ale
>>>> librarie vmsim vor intoarce FALSE. Mi se pare natural sa fie asa,
>>>> folosirea lui DIE acolo
>>>> mi se parea fortata.
>>>> Daca ar putea cineva sa se uite pe tema mea, as aprecia.
>>
>> M-am uitat eu inca o data pe tema ta.
>> Nu toate valorile de return sunt verificate. Ca exemplu, DIE nu e
>> folosit pentru a verifica ce returneaza malloc.
>> Avand in vedere ca inca de la tema1 depunctam acest tip de greseli, mi
>> s-ar fi parut normal ca pana acum numarul lor sa scada.
>>
>> O zi faina,
>> Ioana
>>
>>
>>>>
>>>> Multumesc,
>>>> Simion-Constantinescu Andrei
>>>> 333CA