Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759312AbYGRWXk (ORCPT ); Fri, 18 Jul 2008 18:23:40 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751808AbYGRWXc (ORCPT ); Fri, 18 Jul 2008 18:23:32 -0400 Received: from agminet01.oracle.com ([141.146.126.228]:17421 "EHLO agminet01.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751289AbYGRWXb (ORCPT ); Fri, 18 Jul 2008 18:23:31 -0400 Date: Fri, 18 Jul 2008 15:22:14 -0700 From: Joel Becker To: Louis Rilling Cc: Joel.Becker@localdomain, Linus Torvalds , Mark Fasheh , linux-kernel@vger.kernel.org, ocfs2-devel@oss.oracle.com, Andrew Morton Subject: Re: [Ocfs2-devel] [git patches] Ocfs2 and Configfs updates for 2.6.27 Message-ID: <20080718222214.GB659@ca-server1.us.oracle.com> Mail-Followup-To: Louis Rilling , Joel.Becker@localdomain, Linus Torvalds , Mark Fasheh , linux-kernel@vger.kernel.org, ocfs2-devel@oss.oracle.com, Andrew Morton References: <20080717175146.GH28014@wotan.suse.de> <20080717212038.GA8283@ca-server1.us.oracle.com> <20080717214512.GB8283@ca-server1.us.oracle.com> <20080717223018.GC8283@ca-server1.us.oracle.com> <20080718084803.GB3887@localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20080718084803.GB3887@localdomain> X-Burt-Line: Trees are cool. X-Red-Smith: Ninety feet between bases is perhaps as close as man has ever come to perfection. User-Agent: Mutt/1.5.16 (2007-06-11) X-Brightmail-Tracker: AAAAAQAAAAI= X-Brightmail-Tracker: AAAAAQAAAAI= X-Whitelist: TRUE X-Whitelist: TRUE Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1381 Lines: 38 On Fri, Jul 18, 2008 at 10:48:03AM +0200, Louis Rilling wrote: > > Linus, > > Pull this to get the revert and the PTR_ERR() version: > > > > git://oss.oracle.com/git/jlbec/linux-2.6.git configfs-fixup-ptr-error > > > > It's tested with the same testing I did before - ocfs2 and netconsole. > > While this new version looks ok, it seems that it still allows client > subsystems to return NULL in ->make_group()/->make_item(), in which case > configfs_mkdir() turns the result into -ENOMEM. Maybe this is intended to > smoothly update out-of-tree subsystems? I'd suggest to schedule a removal > of this backward compatibility, and display an error log to encourage > fixing the subsystems that still return NULL. It is precisely to keep out-of-tree modules from crashing. There's no easy way to make them know it changed otherwise :-) It could certainly BUG or WARN before making the fixup. Joel -- "I'm drifting and drifting Just like a ship out on the sea. Cause I ain't got nobody, baby, In this world to care for me." Joel Becker Principal Software Developer Oracle E-mail: joel.becker@oracle.com Phone: (650) 506-8127 -- 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/