Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759496AbXKASsS (ORCPT ); Thu, 1 Nov 2007 14:48:18 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752271AbXKASsA (ORCPT ); Thu, 1 Nov 2007 14:48:00 -0400 Received: from srv5.dvmed.net ([207.36.208.214]:55602 "EHLO mail.dvmed.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752461AbXKASr7 (ORCPT ); Thu, 1 Nov 2007 14:47:59 -0400 Message-ID: <472A1F5B.9060401@garzik.org> Date: Thu, 01 Nov 2007 14:47:55 -0400 From: Jeff Garzik User-Agent: Thunderbird 2.0.0.5 (X11/20070727) MIME-Version: 1.0 To: "Kok, Auke" CC: David Miller , netdev@vger.kernel.org, davej@redhat.com, ajax@redhat.com, linux-kernel@vger.kernel.org Subject: Re: [PATCH] e1000, e1000e valid-addr fixes References: <471E99E8.6030404@garzik.org> <20071023.180744.115914004.davem@davemloft.net> <471EABEE.8030900@garzik.org> <20071023.192313.115635750.davem@davemloft.net> <472A1526.8070208@intel.com> In-Reply-To: <472A1526.8070208@intel.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Score: -4.4 (----) X-Spam-Report: SpamAssassin version 3.1.9 on srv5.dvmed.net summary: Content analysis details: (-4.4 points, 5.0 required) Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1321 Lines: 48 Kok, Auke wrote: > David Miller wrote: >> From: Jeff Garzik >> Date: Tue, 23 Oct 2007 22:20:30 -0400 >> >>> David Miller wrote: >>>> From: Jeff Garzik >>>> Date: Tue, 23 Oct 2007 21:03:36 -0400 >>>> >>>>> I'm wondering if there is a way to avoid adding >>>>> >>>>> if (!is_valid_ether_addr(dev->dev_addr)) >>>>> return -EINVAL; >>>>> >>>>> to every ethernet driver's ->open() hook. >>>> The first idea I get is: >>>> >>>> 1) Create netdev->validate_dev_addr(). >>>> >>>> 2) If it exists, invoke it before ->open(), abort >>>> and return if any errors signaled. >>>> >>>> etherdev init hooks up a function that does the above >>>> check, which allows us to avoid editing every ethernet >>>> driver >>>> >>>> What do you think? >>> Seems sane to me. Something like this (attached)? >> Looks great: >> >> Acked-by: David S. Miller > > I like it. > > Should I start sending patches to remove the checks from e1000/e1000e/ixgb/ixgbe > already (to David, I assume?)? Send the patches to me like normal... Jeff - 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/