From patchwork Wed Mar 25 13:57:59 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Subrata Modak X-Patchwork-Id: 25086 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.176.167]) by ozlabs.org (Postfix) with ESMTP id C2DE6DDD1C for ; Thu, 26 Mar 2009 00:58:14 +1100 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758092AbZCYN6K (ORCPT ); Wed, 25 Mar 2009 09:58:10 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1756873AbZCYN6J (ORCPT ); Wed, 25 Mar 2009 09:58:09 -0400 Received: from e8.ny.us.ibm.com ([32.97.182.138]:46168 "EHLO e8.ny.us.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752901AbZCYN6H (ORCPT ); Wed, 25 Mar 2009 09:58:07 -0400 Received: from d01relay04.pok.ibm.com (d01relay04.pok.ibm.com [9.56.227.236]) by e8.ny.us.ibm.com (8.13.1/8.13.1) with ESMTP id n2PDnpBJ001530; Wed, 25 Mar 2009 09:49:51 -0400 Received: from d01av03.pok.ibm.com (d01av03.pok.ibm.com [9.56.224.217]) by d01relay04.pok.ibm.com (8.13.8/8.13.8/NCO v9.2) with ESMTP id n2PDw4L8186922; Wed, 25 Mar 2009 09:58:04 -0400 Received: from d01av03.pok.ibm.com (loopback [127.0.0.1]) by d01av03.pok.ibm.com (8.12.11.20060308/8.13.3) with ESMTP id n2PDw3n9010226; Wed, 25 Mar 2009 09:58:04 -0400 Received: from subratamodak.linux.ibm.com (subratamodak.in.ibm.com [9.124.158.88]) by d01av03.pok.ibm.com (8.12.11.20060308/8.12.11) with ESMTP id n2PDvxYK010019; Wed, 25 Mar 2009 09:58:00 -0400 From: Subrata Modak To: "David S. Miller" Cc: Sachin Sant , Subrata Modak , netdev , linux-next@vger.kernel.org, Stephen Rothwell , LKML Date: Wed, 25 Mar 2009 19:27:59 +0530 Message-Id: <20090325135759.4397.53055.sendpatchset@subratamodak.linux.ibm.com> Subject: [PATCH] Re: Next March 25: net/netfilter/xt_LED build failure. Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org Hi David, >Today's next randconfig build on x86 failed with following error. > >net/netfilter/xt_LED.c:40: error: field netfilter_led_trigger has incomplete type >net/netfilter/xt_LED.c: In function led_timeout_callback: >net/netfilter/xt_LED.c:78: warning: unused variable ledinternal >net/netfilter/xt_LED.c: In function led_tg_check: >net/netfilter/xt_LED.c:102: error: implicit declaration of function led_trigger_register >net/netfilter/xt_LED.c: In function led_tg_destroy: >net/netfilter/xt_LED.c:135: error: implicit declaration of function led_trigger_unregister > >CONFIG_LED_TRIGGER is not set. > >complete .config attached. The following patch will solve the build problem reported by Sachin. But would leave the following warnings. I hope we can ignore them: net/netfilter/xt_LED.c: In function ‘led_timeout_callback’: net/netfilter/xt_LED.c:78: warning: unused variable ‘ledinternal’ net/netfilter/xt_LED.c: In function ‘led_tg_check’: net/netfilter/xt_LED.c:87: warning: unused variable ‘err’ Signed-Off-By: Subrata Modak --- -- Regards-- Subrata -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html --- a/linux-2.6.29/include/linux/leds.h 2009-03-25 18:38:07.000000000 +0530 +++ b/linux-2.6.29/include/linux/leds.h 2009-03-25 18:24:55.000000000 +0530 @@ -73,9 +73,6 @@ extern void led_classdev_resume(struct l /* * LED Triggers */ -#ifdef CONFIG_LEDS_TRIGGERS - -#define TRIG_NAME_MAX 50 struct led_trigger { /* Trigger Properties */ @@ -91,6 +88,10 @@ struct led_trigger { struct list_head next_trig; }; +#ifdef CONFIG_LEDS_TRIGGERS + +#define TRIG_NAME_MAX 50 + /* Registration functions for complex triggers */ extern int led_trigger_register(struct led_trigger *trigger); extern void led_trigger_unregister(struct led_trigger *trigger); --- a/linux-2.6.29/net/netfilter/xt_LED.c 2009-03-25 18:38:08.000000000 +0530 +++ b/linux-2.6.29/net/netfilter/xt_LED.c 2009-03-25 18:26:55.000000000 +0530 @@ -99,6 +99,7 @@ static bool led_tg_check(const struct xt ledinternal->netfilter_led_trigger.name = ledinfo->id; +#ifdef CONFIG_LEDS_TRIGGERS err = led_trigger_register(&ledinternal->netfilter_led_trigger); if (err) { printk(KERN_CRIT KBUILD_MODNAME @@ -108,7 +109,7 @@ static bool led_tg_check(const struct xt ": Trigger name is already in use.\n"); goto exit_alloc; } - +#endif /* See if we need to set up a timer */ if (ledinfo->delay > 0) setup_timer(&ledinternal->timer, led_timeout_callback, @@ -118,8 +119,10 @@ static bool led_tg_check(const struct xt return true; +#ifdef CONFIG_LEDS_TRIGGERS exit_alloc: kfree(ledinternal); +#endif return false; } @@ -132,7 +135,9 @@ static void led_tg_destroy(const struct if (ledinfo->delay > 0) del_timer_sync(&ledinternal->timer); +#ifdef CONFIG_LEDS_TRIGGERS led_trigger_unregister(&ledinternal->netfilter_led_trigger); +#endif kfree(ledinternal); }