[so] [SO] Contestatie Tema3 Linux

[Andrei Simion-Constantinescu]

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 <ciorneiioana at gmail.com>:
> 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