linux-kernel - RE: [MeeGo-Dev][PATCH] Topcliff: Update PCH

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

Message-ID: <D5AB6E638E5A3E4B8F4406B113A5A19A28EA26B2@shsmsx501.ccr.corp.intel.com>
Date:	Thu, 12 Aug 2010 09:13:24 +0800
From:	"Wang, Qi" <qi.wang@...el.com>
To:	Greg KH <gregkh@...e.de>,
	Masayuki Ohtake <masa-korg@....okisemi.com>
CC:	"meego-dev@...go.com" <meego-dev@...go.com>,
	LKML <linux-kernel@...r.kernel.org>,
	"Wang, Yong Y" <yong.y.wang@...el.com>,
	"Khor, Andrew Chih Howe" <andrew.chih.howe.khor@...el.com>,
	"arjan@...ux.intel.com" <arjan@...ux.intel.com>
Subject: RE: [MeeGo-Dev][PATCH] Topcliff: Update PCH_IEEE1588 driver to
 2.6.35

> -----Original Message-----
> From: Greg KH [mailto:gregkh@...e.de]
> Sent: Wednesday, August 11, 2010 11:15 PM
> To: Masayuki Ohtake
> Cc: meego-dev@...go.com; LKML; Wang, Qi; Wang, Yong Y; Khor, Andrew
> Chih Howe; arjan@...ux.intel.com
> Subject: Re: [MeeGo-Dev][PATCH] Topcliff: Update PCH_IEEE1588 driver to
> 2.6.35
> 
> On Wed, Aug 11, 2010 at 04:29:25PM +0900, Masayuki Ohtake wrote:
> > Hi Greg,
> >
> > > Do they all have to be ioctls?  What exactly are they doing?
> > I think using ioctl is common for this patch.
> 
> "Common" is not ok, right?
> 
> > Do you think that using ioctl is NOT appropriate for this patch?
> 
> Yes.
> 
> > Let me know your intension in more detail.
> 
> Please express yours.  Why do you feel you need these to be ioctls?
> What exactly are they doing?  Where is the documentation for them all?
> 
> > > And are they 32/64bit safe?
> > Only 32bit support.
> 
> That's an obvious problem that needs to be resolved, right?
The Tunnelcreak processor (A kind of Atom-based processor) only supports 32b, so PCH drivers only support 32b.
They can make it 32/64 compatible, but it's difficult to test them, because there're no 64b Atom-based platform have those PCH devices.

Best Regards,
QI.
> 
> thanks,
> 
> greg k-h