Date: Thu, 8 Sep 2005 06:25:07 -0500
From: Robin Holt <holt@sgi.com>
To: Robin Holt <holt@sgi.com>
Cc: Paul Jackson <pj@sgi.com>, KUROSAWA Takahiro <kurosawa@valinux.co.jp>,
       linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/5][BUG] SUBCPUSETS: fix for cpusets minor problem
Message-ID: <20050908112507.GB19987@lnx-holt.americas.sgi.com>
References: <20050908054053.35DAD70031@sv1.valinux.co.jp> <20050908111731.GA19987@lnx-holt.americas.sgi.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20050908111731.GA19987@lnx-holt.americas.sgi.com>
User-Agent: Mutt/1.4.2.1i
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1206
Lines: 32

On Thu, Sep 08, 2005 at 06:17:31AM -0500, Robin Holt wrote:
> On Thu, Sep 08, 2005 at 02:40:53PM +0900, KUROSAWA Takahiro wrote:
> > This patch fixes minor problem that the CPUSETS have when files
> > in the cpuset filesystem are read after lseek()-ed beyond the EOF.
> > 
> > Signed-off-by: KUROSAWA Takahiro <kurosawa@valinux.co.jp>
> > 
> > --- from-0001/kernel/cpuset.c
> > +++ to-work/kernel/cpuset.c	2005-09-05 20:26:18.075772762 +0900
> > @@ -984,6 +984,10 @@ static ssize_t cpuset_common_file_read(s
> >  	*s++ = '\n';
> >  	*s = '\0';
> >  
> > +	/* Do nothing if *ppos is at the eof or beyond the eof. */
> > +	if (s - page <= *ppos)
> > +		return 0;
> > +
> >  	start = page + *ppos;
> >  	n = s - start;
> >  	retval = n - copy_to_user(buf, start, min(n, nbytes));
> 
> 
> Paul,  I think this should go in regardless of the other subcpuset
> changes being proposed.  What do you think?


Oops, didn't see you had already sent it along.  Sorry, Robin
-
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/