Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758623AbXFSKE3 (ORCPT ); Tue, 19 Jun 2007 06:04:29 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1757740AbXFSKEM (ORCPT ); Tue, 19 Jun 2007 06:04:12 -0400 Received: from TYO202.gate.nec.co.jp ([202.32.8.206]:56548 "EHLO tyo202.gate.nec.co.jp" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757353AbXFSKEK (ORCPT ); Tue, 19 Jun 2007 06:04:10 -0400 Message-ID: <4677AA06.4090600@bx.jp.nec.com> Date: Tue, 19 Jun 2007 19:03:50 +0900 From: Keiichi KII User-Agent: Thunderbird 2.0.0.0 (Windows/20070326) MIME-Version: 1.0 To: Satyam Sharma CC: Matt Mackall , Andrew Morton , David Miller , linux-kernel@vger.kernel.org, netdev@vger.kernel.org Subject: Re: [RFC][PATCH -mm take5 4/7] using symlink for the net_device References: <466FC455.5060001@bx.jp.nec.com> <466FC6ED.9060208@bx.jp.nec.com> In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1037 Lines: 32 Hello Satyam, > Sorry, but we're not covering from the error condition fully here. Note > that later you merge the temporary modify_target_list entirely back > into the target_list ... which would still contain these erroneous > nodes. A full cleanup (kobject_unregister the entry, and then list_del > from modify_target_list) is required here, before continuing. I will fix this. If the error occurs, I think so that we need to cleanup completely. >> + strcpy(nt->np.dev_name, dev->name); > > ... you'll have move this up. > Why? I don't have opposition about moving this up, but I'm misplacing the abobe code? or it isn't appropriate about coding style? Thanks -- Keiichi KII NEC Corporation OSS Platform Development Division E-mail: k-keiichi@bx.jp.nec.com - 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/