Prev: [RFC] Mount point suggestions for cgroup
Next: [PATCH v2 2/2] omap: RX-51: enable tsc2005
From: Jens Axboe on 4 Nov 2009 02:50
On Tue, Nov 03 2009, Vivek Goyal wrote:
> Hi All,
>
> This is V1 of the Block IO controller patches on top of 2.6.32-rc5.

Thanks for posting this Vivek. Can you rebase the patches on top of my
for-2.6.33 branch, there are a bigger-than-usual number of pending CFQ
patches in there so things won't apply directly.

If you do that, I'll pull these patches into testing branch and
hopefully merge it in soonish. This patchset looks a lot more mergeable,
thanks!

--
Jens Axboe

--
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: Vivek Goyal on 4 Nov 2009 08:50
On Wed, Nov 04, 2009 at 08:43:33AM +0100, Jens Axboe wrote:
> On Tue, Nov 03 2009, Vivek Goyal wrote:
> > Hi All,
> >
> > This is V1 of the Block IO controller patches on top of 2.6.32-rc5.
>
> Thanks for posting this Vivek. Can you rebase the patches on top of my
> for-2.6.33 branch, there are a bigger-than-usual number of pending CFQ
> patches in there so things won't apply directly.
>
> If you do that, I'll pull these patches into testing branch and
> hopefully merge it in soonish. This patchset looks a lot more mergeable,
> thanks!
>

Thanks Jens. Sure, will rebase the patches on top of "for-2.6.33" branch
and repost.

Thanks
Vivek
--
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: Jeff Moyer on 5 Nov 2009 06:40
Vivek Goyal <vgoyal(a)redhat.com> writes:

> On Wed, Nov 04, 2009 at 02:12:19PM -0500, Jeff Moyer wrote:
>> Vivek Goyal <vgoyal(a)redhat.com> writes:
>>
>> > Hi All,
>> >
>> > This is V1 of the Block IO controller patches on top of 2.6.32-rc5.
>> >
>> > A consolidated patch can be found here:
>> >
>> > http://people.redhat.com/vgoyal/io-controller/blkio-controller/blkio-controller-v1.patch
>>
>> Overall it looks good. Vivek, could you please run some benchmarks
>> against a vanilla kernel and then with your patch applied and cgroups
>> compiled in but not configured?
>>
>
> Sure I can. Do you have something specific in mind?

I don't, actually. iozone comes to mind as a simple test to setup and
run. Or you could run one or more of the fio sample job files. Really,
I just want to see if we're taking a huge performance hit so we can fix
that up before it's merged.

Thanks!
Jeff
--
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: Vivek Goyal on 5 Nov 2009 06:41
On Wed, Nov 04, 2009 at 02:27:10PM -0500, Jeff Moyer wrote:
> Vivek Goyal <vgoyal(a)redhat.com> writes:
>
> > On Wed, Nov 04, 2009 at 02:12:19PM -0500, Jeff Moyer wrote:
> >> Vivek Goyal <vgoyal(a)redhat.com> writes:
> >>
> >> > Hi All,
> >> >
> >> > This is V1 of the Block IO controller patches on top of 2.6.32-rc5.
> >> >
> >> > A consolidated patch can be found here:
> >> >
> >> > http://people.redhat.com/vgoyal/io-controller/blkio-controller/blkio-controller-v1.patch
> >>
> >> Overall it looks good. Vivek, could you please run some benchmarks
> >> against a vanilla kernel and then with your patch applied and cgroups
> >> compiled in but not configured?
> >>
> >
> > Sure I can. Do you have something specific in mind?
>
> I don't, actually. iozone comes to mind as a simple test to setup and
> run. Or you could run one or more of the fio sample job files. Really,
> I just want to see if we're taking a huge performance hit so we can fix
> that up before it's merged.

Sure, will run some tests to see if this patchset is introducing any huge
performance regressions somewhere.

Thanks
Vivek
--
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: Vivek Goyal on 5 Nov 2009 06:41
On Wed, Nov 04, 2009 at 02:12:19PM -0500, Jeff Moyer wrote:
> Vivek Goyal <vgoyal(a)redhat.com> writes:
>
> > Hi All,
> >
> > This is V1 of the Block IO controller patches on top of 2.6.32-rc5.
> >
> > A consolidated patch can be found here:
> >
> > http://people.redhat.com/vgoyal/io-controller/blkio-controller/blkio-controller-v1.patch
>
> Overall it looks good. Vivek, could you please run some benchmarks
> against a vanilla kernel and then with your patch applied and cgroups
> compiled in but not configured?
>

Sure I can. Do you have something specific in mind?

Thanks
Vivek
--
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/
 |  Next  |  Last
Pages: 1 2
Prev: [RFC] Mount point suggestions for cgroup
Next: [PATCH v2 2/2] omap: RX-51: enable tsc2005