From owner-svn-src-all@FreeBSD.ORG Thu Nov 21 09:34:18 2013 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id ED6BC5E7 for ; Thu, 21 Nov 2013 09:34:17 +0000 (UTC) Received: from mail-ie0-f178.google.com (mail-ie0-f178.google.com [209.85.223.178]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id AC4982531 for ; Thu, 21 Nov 2013 09:34:17 +0000 (UTC) Received: by mail-ie0-f178.google.com with SMTP id lx4so7769380iec.9 for ; Thu, 21 Nov 2013 01:34:16 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:date :message-id:subject:from:to:cc:content-type; bh=Rc85Rp5NDFJxVLL89ngogi9QZmgSy2m9Eo2b37EnRDU=; b=LMLUfvmtlRkQ8SUYbgyquhqMS8YJVtpUYWPgFzpx2HRyEzPMBiYiv52JW7v96zQ3NU tWjZC1cSWNHpOGdhyiOlht7nGCoQpVfU6VICrSz6wSCylgSl6Owh3yvlS4iLxQOApSTk aWON3JTtHaJumk6zUARrL3se3Vz7YItSuuesWzf2WPabxYcSBSkfXc7rbqREvIZvYg/s 4leZh7vOhmKTt5HJBQLKwSHUq2iD4jBDbEst7MVdnp51BpGTUUL+omU2qq0bkE6OXSej bD86ASog+qF+NFK5PlaTNYBazlcrXH7Ph1JASm3xCNrfjrn9LWJN+1kjxw3XdT0OW5Gt 5Bjg== X-Gm-Message-State: ALoCoQkImeaUOKtTzx9/qFZnRpyumBEdXHJuw9mQRTymC52h4C4OMmTNN/oQAW/r0/hOKo5B2SVD MIME-Version: 1.0 X-Received: by 10.50.79.198 with SMTP id l6mr4514556igx.31.1385026456858; Thu, 21 Nov 2013 01:34:16 -0800 (PST) Received: by 10.50.209.5 with HTTP; Thu, 21 Nov 2013 01:34:16 -0800 (PST) In-Reply-To: <20131120213119.GL2279@funkthat.com> References: <201311201110.rAKBAOMY061212@svn.freebsd.org> <20131120213119.GL2279@funkthat.com> Date: Thu, 21 Nov 2013 10:34:16 +0100 Message-ID: Subject: Re: svn commit: r258387 - head/sys/dev/nand From: Grzegorz Bernacki To: John-Mark Gurney Content-Type: text/plain; charset=ISO-8859-1 X-Content-Filtered-By: Mailman/MimeDel 2.1.16 Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, Grzegorz Bernacki , src-committers@freebsd.org X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.16 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 21 Nov 2013 09:34:18 -0000 Hi, Thanks for information. Fix has been committed. regards, grzesiek 2013/11/20 John-Mark Gurney > 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 > > 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." >