[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <BCB48C05FCE8BC4D9E61E841ECBE6DB702B124@039-SN2MPN1-013.039d.mgd.msft.net>
Date: Wed, 15 Feb 2012 07:29:36 +0000
From: Liu Qiang-B32616 <B32616@...escale.com>
To: Li Yang-R58472 <r58472@...escale.com>
CC: "jgarzik@...ox.com" <jgarzik@...ox.com>,
"linux-ide@...r.kernel.org" <linux-ide@...r.kernel.org>,
"linuxppc-dev@...ts.ozlabs.org" <linuxppc-dev@...ts.ozlabs.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH V2 RESEND] fsl-sata: I/O load balancing
> -----Original Message-----
> From: linux-ide-owner@...r.kernel.org [mailto:linux-ide-
> owner@...r.kernel.org] On Behalf Of Li Yang
> Sent: Wednesday, February 15, 2012 3:22 PM
> To: Liu Qiang-B32616
> Cc: jgarzik@...ox.com; linux-ide@...r.kernel.org; linuxppc-
> dev@...ts.ozlabs.org; linux-kernel@...r.kernel.org
> Subject: Re: [PATCH V2 RESEND] fsl-sata: I/O load balancing
>
> On Wed, Feb 15, 2012 at 1:49 PM, Qiang Liu <qiang.liu@...escale.com>
> wrote:
>
> Hi Liu Qiang,
>
> The patch is fine except for the title and comment. It's too vague to
> say I/O load balancing. You need explicit description like "add
> interrupt coalescing support" as title.
>
> > Reduce interrupt signals through reset Interrupt Coalescing Control Reg.
> > Provide dynamic method to adjust interrupt signals and timer ticks by
> sysfs.
> > It is a tradeoff for different applications.
>
> How about:
>
> Adds support for interrupt coalescing feature to reduce interrupt events.
> Provides mechanism of adjusting coalescing count and timeout tick by
> sysfs on runtime, so that tradeoff of latency and CPU load can be made
> depending on applications.
>
Ok, I will change the title and comments more specifically.
> - Leo
> --
> To unsubscribe from this list: send the line "unsubscribe linux-ide" in
> the body of a message to majordomo@...r.kernel.org More majordomo info at
> http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists