Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757380Ab0KOOhp (ORCPT ); Mon, 15 Nov 2010 09:37:45 -0500 Received: from mail-ew0-f46.google.com ([209.85.215.46]:55826 "EHLO mail-ew0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753446Ab0KOOho (ORCPT ); Mon, 15 Nov 2010 09:37:44 -0500 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=sender:date:from:to:subject:message-id:references:mime-version :content-type:content-disposition:content-transfer-encoding :in-reply-to:user-agent; b=Y+5WbJikkqu+cO+XUvN4dRXAzuTsn3EYQhGP/RDa2sY0ajL6OhfduosyyLlrNVUeis 6oD4SJZlySjGgR+ykHPM4njBLMfr0b2x4HB+kwnbAHbP3MSiwfo7W6nNAfaOR0w9+vrH lGN9mkdU3xzakrEI7cap2VWVcOQOepKXmNsVo= Date: Mon, 15 Nov 2010 17:37:37 +0300 From: Vasiliy Kulikov To: Dan Carpenter , kernel-janitors@vger.kernel.org, David Woodhouse , linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] fs: jffs2: fix error value sign Message-ID: <20101115143735.GA4548@albatros> References: <1289765320-6071-1-git-send-email-segoon@openwall.com> <20101114202905.GA21614@bicker> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20101114202905.GA21614@bicker> User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2148 Lines: 53 On Sun, Nov 14, 2010 at 23:29 +0300, Dan Carpenter wrote: > On Sun, Nov 14, 2010 at 11:08:39PM +0300, Vasiliy Kulikov wrote: > > do_verify_xattr_datum(), do_load_xattr_datum(), load_xattr_datum() > > and verify_xattr_ref() should return negative value on error. > > Sometimes they return EIO that is positive. Change this to -EIO. > > > > Signed-off-by: Vasiliy Kulikov > > --- > > This driver is not compilable, so the patch is not tested. > > > > It compiles. Really? x86_64, linux-next: make -C /home/vasya/linux fs/jffs2/xattr.o make: Вход в каталог `/home/vasya/dev/linux-next' CHK include/linux/version.h CHK include/generated/utsrelease.h CALL scripts/checksyscalls.sh CC fs/jffs2/xattr.o fs/jffs2/xattr.c: In function ‘unload_xattr_datum’: fs/jffs2/xattr.c:87: error: ‘struct jffs2_sb_info’ has no member named ‘xdatum_mem_usage’ fs/jffs2/xattr.c: In function ‘reclaim_xattr_datum’: fs/jffs2/xattr.c:105: error: ‘struct jffs2_sb_info’ has no member named ‘xdatum_mem_threshold’ fs/jffs2/xattr.c:105: error: ‘struct jffs2_sb_info’ has no member named ‘xdatum_mem_usage’ fs/jffs2/xattr.c:108: error: ‘struct jffs2_sb_info’ has no member named ‘xdatum_mem_usage’ fs/jffs2/xattr.c:109: error: ‘struct jffs2_sb_info’ has no member named ‘xdatum_mem_usage’ fs/jffs2/xattr.c:110: error: ‘XATTRINDEX_HASHSIZE’ undeclared (first use in this function) fs/jffs2/xattr.c:110: error: (Each undeclared identifier is reported only once fs/jffs2/xattr.c:110: error: for each function it appears in.) For both defconfig and allyesconfig. > regards, > dan carpenter > > > -- > To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Vasiliy -- 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/