Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S262270AbTFIXVL (ORCPT ); Mon, 9 Jun 2003 19:21:11 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S262271AbTFIXVL (ORCPT ); Mon, 9 Jun 2003 19:21:11 -0400 Received: from smtpzilla1.xs4all.nl ([194.109.127.137]:29970 "EHLO smtpzilla1.xs4all.nl") by vger.kernel.org with ESMTP id S262270AbTFIXVK (ORCPT ); Mon, 9 Jun 2003 19:21:10 -0400 Date: Tue, 10 Jun 2003 01:34:19 +0200 (CEST) From: Roman Zippel X-X-Sender: roman@serv To: "David S. Miller" cc: wa@almesberger.net, , Subject: Re: [PATCH][ATM] use rtnl_{lock,unlock} during device operations (take 2) In-Reply-To: <20030609.161435.104053652.davem@redhat.com> Message-ID: References: <20030609.160013.74730356.davem@redhat.com> <20030609.161435.104053652.davem@redhat.com> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 787 Lines: 26 Hi, On Mon, 9 Jun 2003, David S. Miller wrote: > > netdev->dead = 1; > > netdev->op_this = NULL; > > netdev->op_that = NULL; > > netdev->op_whatever = NULL; > > synchronize_kernel(); > > That assumes of course that the functions don't sleep. > (RCU isn't really the answer to everything.) > > They hold references to the object, it doesn't matter if > they sleep. That's not the point. You also have to wait for the already running operations to finish, before you can allow the module to unload. bye, Roman - 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/