spin_lock is used in wrong way

[anton-bondarev]

functions 'spin_lock' and 'spin_unlock' in net_pack_manager.c are used with wrong interface

    unsigned long sp;
    spin_lock(sp);

must be

    unsigned long sp;
    spin_lock(&sp);

[anton-bondarev]

Closed in f8a9ae4

[abusalimov]

No, the first case is true.
spin_xxx are defined as macros, you do not need referencing the lock variable to pass it. Furthermore, using plain spin_lock/spin_unlock does not make sense in uniprocessor systems, particularly in embox, these macros are defined as no-op
(src/linux-compat/include/linux/spinlock.h).
To protect data in interrupt context you should use spin_lock_irqsave(lock, flags) and spin_unlock_irqrestore(lock, flags).

Reopened, status set to Accepted

[anton-bondarev]

Closed in 847a75c