linux-kernel - Re: [lustre-devel] [PATCH v3 5/8] staging:lustre: separate kernel and user land defines in the LNet headers

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: <D19B6F47.F5921%andreas.dilger@intel.com>
Date:	Tue, 9 Jun 2015 00:33:50 +0000
From:	"Dilger, Andreas" <andreas.dilger@...el.com>
To:	Dan Carpenter <dan.carpenter@...cle.com>,
	James Simmons <jsimmons@...radead.org>
CC:	"devel@...verdev.osuosl.org" <devel@...verdev.osuosl.org>,
	"Greg Kroah-Hartman" <gregkh@...uxfoundation.org>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	"Drokin, Oleg" <oleg.drokin@...el.com>,
	"lustre-devel@...ts.lustre.org" <lustre-devel@...ts.lustre.org>
Subject: Re: [lustre-devel] [PATCH v3 5/8] staging:lustre: separate kernel
 and user land defines in the LNet headers

On 2015/06/05, 3:02 AM, "Dan Carpenter" <dan.carpenter@...cle.com> wrote:

>On Wed, Jun 03, 2015 at 04:43:24PM -0400, James Simmons wrote:
>> Currently the lnet headers used by user land contain various internal
>> LNet structures that are only used by kernel space. Move the user land
>> structures to headers used by user land. The kernel structures are
>> relocated to headers that are never exposed to user land.
>> 
>> Signed-off-by: James Simmons <jsimmons@...radead.org>
>> ---
>> diff --git a/drivers/staging/lustre/lnet/lnet/acceptor.c
>>b/drivers/staging/lustre/lnet/lnet/acceptor.c
>> index 1dc7c8a..4928f5c 100644
>> --- a/drivers/staging/lustre/lnet/lnet/acceptor.c
>> +++ b/drivers/staging/lustre/lnet/lnet/acceptor.c
>> @@ -243,8 +243,6 @@ lnet_accept(struct socket *sock, __u32 magic)
>>  
>>  		if (magic == le32_to_cpu(LNET_PROTO_TCP_MAGIC))
>>  			str = "'old' socknal/tcpnal";
>> -		else if (lnet_accept_magic(magic, LNET_PROTO_RA_MAGIC))
>> -			str = "'old' ranal";
>>  		else
>>  			str = "unrecognised";
>>  
>
>Presumably this was done intentionally.  We deleted LNET_PROTO_RA_MAGIC.
>The changelog was not very clear why.

The "Rapid Array" network interface is an obsolete network once used by
Cray hardware but hasn't been supported for about 10 years and was (mostly)
deleted from the tree already.  This was just a left-over I guess.

Cheers, Andreas
-- 
Andreas Dilger

Lustre Software Architect
Intel High Performance Data Division


--
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/