lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <a24804730804271851j61ded9c2tf1c15b6da845050c@mail.gmail.com>
Date:	Sun, 27 Apr 2008 18:51:35 -0700
From:	"Chris Peterson" <cpeterso@...terso.com>
To:	"Jiri Slaby" <jirislaby@...il.com>
Cc:	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] fix warning: remove unused static function in drivers/char/ip2/i2ellis.c

Yes, I did compile it, but I now realize I had compiled the computone
driver to be linked statically, but iiEllisCleanup() is only called
when the driver is built as a module. What is the appropriate cleanup
code path for non-modular drivers?


chris


On Sun, Apr 27, 2008 at 12:42 AM, Jiri Slaby <jirislaby@...il.com> wrote:
> On 04/27/2008 09:09 AM, Chris Peterson wrote:
>
> > This patch against linux-2.6.25-git7.
> >
> > chris
> >
> >
> > Signed-off-by: Chris Peterson <cpeterso@...terso.com>
> > ---
> > diff -Naur linux-2.6.25-git7/drivers/char/ip2/i2ellis.c
> > linux-2.6.25-git7+patch/drivers/char/ip2/i2ellis.c
> > --- linux-2.6.25-git7/drivers/char/ip2/i2ellis.c        2008-04-25
> > 22:47:01.000000000 -0700
> > +++ linux-2.6.25-git7+patch/drivers/char/ip2/i2ellis.c  2008-04-25
> > 23:19:57.000000000 -0700
> > @@ -86,22 +86,6 @@
> >  }
> >
> >
> //******************************************************************************
> > -// Function:   iiEllisCleanup()
> > -// Parameters: None
> > -//
> > -// Returns:    Nothing
> > -//
> > -// Description:
> > -//
> > -// This routine performs any required cleanup of the iiEllis subsystem.
> > -//
> >
> -//******************************************************************************
> > -static void
> > -iiEllisCleanup(void)
> > -{
> > -}
> >
>
>  Well, have you tried to even build it? The .c file is *included* into
> i2main.c where this function is used.
>
>  NAK
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ