Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/sys

On Mon, Jul 02, 2012 at 11:13:51PM +0100, Mindaugas Rasiukevicius wrote:
> "Manuel Bouyer" <> wrote:
> > Module Name:        src
> > Committed By:       bouyer
> > Date:               Mon Jul  2 18:15:48 UTC 2012
> > 
> > ...
> > 
> > Log Message:
> > Add sata Port MultiPlier (PMP) support to the ata bus layer,
> > as described in
> >
> > PMP support in integrated to the atabus layer.
> Nice piece of development!
> Few minor points.  Does atabus_alloc_drives() need to use M_NOWAIT and
> malloc(9), instead of kmem(9)?

At first glance I'd say it should always be called from the atabus thread,
so it should be OK

> Also, any reason why you use tsleep(9)
> instead of kpause(9) (in satapmp_subr.c and elsewhere)?  We are trying
> to get rid of the former.

kpause(9) could be used here. But I prefer to keep sys/dev/ata consistent
and convert all the code to kpause at the same time. It could have been
done here, I just didn't think about it.
Next opportunity will be with NCQ support, which I hope to look at soon.
This also would be the time to make ata(4) MP-safe.

Manuel Bouyer <>
     NetBSD: 26 ans d'experience feront toujours la difference

Home | Main Index | Thread Index | Old Index