From: Peter Zijlstra on
On Thu, 2010-04-22 at 15:50 +0800, Lin Ming wrote:

> This patch series is to remove hw_perf_group_sched_in() first.
>
> patch 1: core, add group scheduling transactional APIs
> patch 2: x86, implement group scheduling transactional APIs
> patch 3: powerpc, implement group scheduling transactional APIs
> patch 4: sparc, implement group scheduling transactional APIs
>
> Thanks for any comment.

Don't RISC architectures deal better with word sized variables than with
u8? I thought it curious you used u8 for the flags thing instead of int
or long, but I'm not quite sure it really matters one way or the other.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: Lin Ming on
On Thu, 2010-04-22 at 16:29 +0800, Peter Zijlstra wrote:
> On Thu, 2010-04-22 at 15:50 +0800, Lin Ming wrote:
>
> > This patch series is to remove hw_perf_group_sched_in() first.
> >
> > patch 1: core, add group scheduling transactional APIs
> > patch 2: x86, implement group scheduling transactional APIs
> > patch 3: powerpc, implement group scheduling transactional APIs
> > patch 4: sparc, implement group scheduling transactional APIs
> >
> > Thanks for any comment.
>
> Don't RISC architectures deal better with word sized variables than with
> u8? I thought it curious you used u8 for the flags thing instead of int
> or long, but I'm not quite sure it really matters one way or the other.

Will change it to "unsigned int".

Thanks.

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