[so2] [Teme] Tema0 & altele

[Darius-Florentin Neatu]

Mulțumesc pentru raspunsuri!

@Robert
Stiu ca se poate declara in alta parte, dar am cateva argumente impotriva.
1. Daca alocam global/static, vom avea nevoie de un mecanism de
sincronizare. Atunci absolut toate operatiile (incepand cu parsarea) s-ar
desfasura sincron.
2. Daca alocam pe heap, acelasi lucru: daca folosesc parametrul GFP_KERNEL
procesul poate fi intrerupt (si nu vreau), iar daca bag alocarea intr-un
context atomic, din nou limitez paralelismul.
Prefer stiva si pentru ca este mai rapida.

Asa cum mi-am gandit eu implementarea, in sectiunea critica am doar accesul
efectiv in lista, celelalte rezultate (ex. valoarea inserata, nodul inserat
etc) fiind deja precalculate. In acest fel am o procesare minima in
sectiunea critica.

As vrea si un raspuns din partea responsabililor daca am procedat corect.
Am pus si cele de mai sus in README, ca sa justific modificarea acelei
dimensiuni.

Darius

On Wed, 14 Mar 2018, 6:21 pm Daniel Baluta, <daniel.baluta at gmail.com> wrote:

> 2018-03-14 16:27 GMT+02:00 Darius-Florentin Neatu via so2
> <so2 at cursuri.cs.pub.ro>:
> > Salut,
> >
> > Am o intrebare referitoare la scheletul dat la Tema 0.
> > Exista un macro (#define PROCFS_MAX_SIZE     1024) care determina
> alocarea
> > pe stiva a unui buffer cu dimensiune 1024 bytes. Dupa declararea a inca
> > cateva variabile (int, pointer) primesc un warning:
> > "the frame size of 1028 bytes is larger than 1024 bytes
> > [-Wframe-larger-than=]"
> > Se poate modifica valoarea? (daca pun 1000 in loc de 1024, nu mai am
> acest
> > warning - de asemenea, implementarea trece toate testele daca pun 1000).
> >
> > Am cateva intrebari generale pentru teme.
> >
> > 0.  Am observat ca Tema 0 nu are README (nu cere readme). Este corecta
> > observatia? Se specifica undeva prin regulament altceva? (nu am gasit).
> > Observ ca acelasi scenariu apare si pe paginile temelor urmatoare.
> > Daca am pe viitor de precizat astfel de alegeri, este OK sa trec tot in
> > README, chiar daca tema nu cere un README?
>
> Nu e nevoie de README. Nu se scade daca faceti unul. :)
>
> >
> > 1. As vrea sa imi salvez temele pe Gitlab si sa pun link-ul in README.
> > Daca se permite acest lucru (si este util si celor care corecteaza), as
> vrea
> > sa stiu de unde pot obtine lista cu id-urile de LDAP ale tuturor
> > responsabililor.
> > Momentan am un repo privat[1] si am adaugat pe Razvan si pe Daniel.
>
> Alexandra?
>
> >
> > 2. Ca modalitate de folosire, ma gandeam sa pun link-ul acesta[1] in
> README,
> > iar responsabilul sa se uite in assignments/<id_assignment>.
> > Am vazut aici[2] ca se depuncteaza pentru "includerea de fișiere binare
> sau
> > irelevante în arhivă;".
> > Ramane valabila aceasta regula si pentru repo?
>
> Repo-ul a fost creat anul trecut pentru a se oferi feedback mai usor pe
> teme sau
> alte bucati de cod.
>
> In repo poti sa pui ce doresti.
>
> > Ce m-ar interesa pe mine ar fi urmatoarele:
> > - daca in assignments/<id_assignment>/ as putea sa am, de exemplu,
> scripturi
> > extra (pe care le-am folosit in faza de rezolvare si as vrea sa le
> pastrez
> > in acest loc); eventual si arhiva finala trimisa pe vmchecker ca sa nu o
> > pierd. :D
>
> In tree-ul tau poti sa faci ce vrei.
>
> > - daca pot folosi repo-ul si pentru altceva inafara de teme (Ex. in
> /lab/ as
> > vrea sa imi salvez bucati de cod de la laborator, daca voi considera ca e
> > nevoie). Practic /assignments/* e doar pentru teme, iar in restul
> repo-ului
> > sa pot pune si alte lucruri legate de so2.
>
> Poti folosi repo-ul pentru ce doresti tu. Insa te rugam sa nu-l faci
> public.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://cursuri.cs.pub.ro/pipermail/so2/attachments/20180315/ca3087bf/attachment.html>