Date: Thu, 21 Nov 2013 10:34:16 +0100 From: Grzegorz Bernacki <gjb@semihalf.com> To: John-Mark Gurney <jmg@funkthat.com> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, Grzegorz Bernacki <gber@freebsd.org>, src-committers@freebsd.org Subject: Re: svn commit: r258387 - head/sys/dev/nand Message-ID: <CALWW6u4vymE_5%2BBRY3%2BTkB4fP2cVB7F5-HJbfu4U%2BTpcb8%2BP3g@mail.gmail.com> In-Reply-To: <20131120213119.GL2279@funkthat.com> References: <201311201110.rAKBAOMY061212@svn.freebsd.org> <20131120213119.GL2279@funkthat.com>
next in thread | previous in thread | raw e-mail | index | archive | help
Hi, Thanks for information. Fix has been committed. regards, grzesiek 2013/11/20 John-Mark Gurney <jmg@funkthat.com> > Grzegorz Bernacki wrote this message on Wed, Nov 20, 2013 at 11:10 +0000: > > Author: gber > > Date: Wed Nov 20 11:10:23 2013 > > New Revision: 258387 > > URL: http://svnweb.freebsd.org/changeset/base/258387 > > > > Log: > > Split raw reading/programming into smaller chunks to avoid allocating > too > > big chunk of kernel memory. Validate size of data. Add error handling > to > > avoid calling copyout() when data has not been read correctly. > > > > Reviewed by: zbb > > Reported by: x90c <geinblues@gmail.com> > > MFC after: 2 days > > > > Modified: > > head/sys/dev/nand/nand_cdev.c > > head/sys/dev/nand/nand_geom.c > > Looks like this change errors w/ gcc: > > http://tinderbox.freebsd.org/tinderbox-head-noclang-build-HEAD-armv6-arm.brief > > -- > John-Mark Gurney Voice: +1 415 225 5579 > > "All that I will do, has been done, All that I have, has not." >
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CALWW6u4vymE_5%2BBRY3%2BTkB4fP2cVB7F5-HJbfu4U%2BTpcb8%2BP3g>