[pso] tema5 teste linux

[Adriana Draghici]

am lasat cu rw lock in loc de spinlock_bh pt ca Adriana S mi-a spus ca a
folosit spinlock_bh si tot ii pica testele si atunci nu am mai dat
importanta sa schimb lock-urile folosite, ba la un upload le si comentasem
parca.


2009/6/8 Bogdan Tenea <tenea.bogdan at gmail.com>

> Eu de exemplu am folosit copy_to_user (doh), spinlock_bh si GFP_KERNEL
> acolo unde era cazul si tot am problemele astea.
> Daca am scris despre problema noastra in readme mai trebuie sa trimitem
> email ?
>
> 2009/6/8 Razvan Deaconescu <razvan.deaconescu at cs.pub.ro>
>
> On Sun, 2009-06-07 at 15:06 +0300, Razvan Deaconescu wrote:
>> > Voi reveni cu un raspuns oficial pana diseara.
>>
>> Am crezut ca voi detecta problema, dar nu o pot reproduce pe
>> implementarea "de referinta" :-)
>>
>> M-am uitat peste codul Adrianei si am observat urmatoarele "greseli":
>> * foloseste read_lock, read_unlock nu read_lock_bh, read_unlock_bh in
>> ioctl
>> * foloseste GFP_ATOMIC in ioctl? de ce? nu se afla in context
>> intrerupere si nici nu detine un spinlock
>>
>> Intrucat nu avem integrata optiunea de afisare a mesajelor de kernel in
>> vmchecker nu se poate face debugging usor prin masina virtuala. Astfel
>> ca, daca tema trece pe sistemul vostru dar nu pe site, o puteti lasa asa
>> si ii puteti trimite e-mail responsabilului de tema (in cazul de fata
>> Dragos) in care sa precizati "problema" voastra.
>>
>> Razvan
>>
>>
>> --
>> This message has been scanned for viruses and
>> dangerous content by MailScanner, and is
>> believed to be clean.
>>
>> _______________________________________________
>> pso mailing list
>> pso at cursuri.cs.pub.ro
>> http://cursuri.cs.pub.ro/cgi-bin/mailman/listinfo/pso
>>
>
>
> _______________________________________________
> pso mailing list
> pso at cursuri.cs.pub.ro
> http://cursuri.cs.pub.ro/cgi-bin/mailman/listinfo/pso
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://cursuri.cs.pub.ro/pipermail/pso/attachments/20090608/ea4f8165/attachment.htm>