linux-kernel - Re: Re: [PATCH] staging: gdm724: adding kernel endianness header

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 for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <14d39953976.ac8f06c0202743.2448506682461818605@kerneldev.net>
Date:	Sat, 09 May 2015 11:55:17 -0500
From:	Jaime Arrocha <jarr@...neldev.net>
To:	Dan Carpenter <dan.carpenter@...cle.com>
Cc:	<gregkh@...uxfoundation.org>, <devel@...verdev.osuosl.org>,
	<deanahn@...semi.com>, <jonathankim@...semi.com>,
	<linux-kernel@...r.kernel.org>, <jarrocha@...ux.com>
Subject: Re: Re: [PATCH] staging: gdm724: adding kernel endianness header



---- On Sat, 09 May 2015 05:59:40 -0500 Dan Carpenter <dan.carpenter@...cle.com> wrote ---- 

 > On Fri, May 08, 2015 at 06:29:24PM -0500, Jaime Arrocha wrote: 
 > > >From TODO list: remove test for host endian 
 > > Included header to gather information about host endianness. 
 > > Please let me know if the code addition requires corrections 
 > > to meet standards. 
 > >  
 >  
 > No.  This isn't really an improvement.  We need to get rid of ->host_ed 
 > entirely.  Which is actually easy.  Basically change gdm_cpu_to_dev16() 
 > and friends to do so something: 
 >  
 > u16 gdm_cpu_to_dev16(struct gdm_endian *ed, u16 x) 
 > { 
 >     if (ed->dev_ed == ENDIANNESS_LITTLE) 
 >         return cpu_to_le16(x); 
 >     else 
 >         return cpu_to_be16(x); 
 > } 
 >  
 > regards, 
 > dan carpenter 
 >  
 > 

You are right, I will dive a little further to find out the use for host_ed and eliminate it. I did
read an email from a while ago where one author mentioned that he needed to keep this function.

Also, the kernel already has functions to swap numbers for endianness. Will you rather use those?

Thanks for your feedback,
Jaime

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