RE: Problem on a kernel driver(SuSE, SMP)

Alan Cox (alan@lxorguk.ukuu.org.uk)
05 Sep 2002 00:44:33 +0100


On Wed, 2002-09-04 at 18:56, Libershteyn, Vladimir wrote:

> //
> // sleep until data is ready
> //
> down_interruptible(&a->sem[enumerator]);

Suppose its interrupted. You dont check that and handle it..

> board_address = ((unsigned long *)((unsigned char *)a->vaddr + OutputQueueFilled));
> length = *board_address;

You can't poke around in memory directly either. Yes it works on x86 but
unless you use ioremap combined with readl and friends it wont work they
way you expect on ia64, x86-64, ...

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/