netdev - Re: [PATCH 3/12] benet: net_object i/f functions, Makefile

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-next>] [day] [month] [year] [list]

Date:	Thu, 19 Jun 2008 02:59:19 -0700
From:	"Subbu Seetharaman" <subbus@...verengines.com>
To:	"Christoph Hellwig" <hch@...radead.org>
Cc:	"Sam Ravnborg" <sam@...nborg.org>, jeff@...zik.org,
	netdev@...r.kernel.org, sathyap@...verengines.com
Subject: Re: [PATCH 3/12] benet: net_object i/f functions, Makefile

Christoph,

The primary reason for separating the lower layer
code from rest of the driver is to share it
across two drivers for the two functions
of the device.  The secondary objective of
keeping this code OS neutral through 
abstractions has been conceded based
on the review comments from the last few 
rounds.

Thanks.

Subbu

----- Original Message -----
From: Christoph Hellwig [mailto:hch@...radead.org]
To: Subbu Seetharaman [mailto:subbus@...verengines.com]
Cc: Sam Ravnborg [mailto:sam@...nborg.org], jeff@...zik.org, netdev@...r.kernel.org, sathyap@...verengines.com
Sent: Thu, 19 Jun 2008 02:11:35 -0700
Subject: Re: [PATCH 3/12] benet: net_object i/f functions, Makefile

On Wed, Jun 18, 2008 at 11:27:01PM -0700, Subbu Seetharaman wrote:
> We could not get KBUILD_EXTRA_SYMBOLS to work.
> This  is what we did :
> 
> beclib's Module.symvers has entries like this :
> 
> 0x00000000      be_eq_destroy   /home/subbu/kod39-sperla/drivers/message/beclib/beclib  EXPORT_SYMBOL 
> 
> benet uses exports from beclib and we have the following line in benet's Makefile: 
> KBUILD_EXTRA_SYMBOLS = $(src)/../../message/beclib/Module.symvers 
> 
> When we build benet we still get warnings like this
> for symbols defined and exported from beclib :
> WARNING: "be_eq_destroy" [/home/subbu/kod39-sperla/drivers/net/benet/benet.ko] undefined! 
> 

The righ answer is of course to kill all this beclib junk and write
a proper linux driver.  We're made it pretty clear a few times in the
past that these abstraction layers don't go in.

___________________________________________________________________________________
This message, together with any attachment(s), contains confidential and proprietary information of
ServerEngines Corporation and is intended only for the designated recipient(s) named above. Any unauthorized
review, printing, retention, copying, disclosure or distribution is strictly prohibited.  If you are not the
intended recipient of this message, please immediately advise the sender by reply email message and
delete all copies of this message and any attachment(s). Thank you.

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html