Acum am dat reply all. Nu le vazusem pe cele din
vm_page_fault_handler. M-am uitat mai atent, iar la malloc in general
am uitat sa folosesc DIE, ai dreptate.
Mersi ca te-ai uitat pe cod.
O zi buna in continuare,
Andrei
2017-05-19 11:40 GMT+03:00 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 :
>>> Salut,
>>>
>>>
Am uitat sa precizez id-ul de pe vmchecker: andrei.simion
2017-05-19 10:53 GMT+03:00 Andrei Simion-Constantinescu
:
> 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
___
http://ocw.cs.pub.ro/courses/so/info/lista-discutii