linux-kernel - RE: [PATCH] scatterlist: define SG chain for arm architecture

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: <496565EC904933469F292DDA3F1663E602F46545F2@dlee06.ent.ti.com>
Date:	Fri, 1 Oct 2010 13:54:37 -0500
From:	"Guzman Lugo, Fernando" <fernando.lugo@...com>
To:	Russell King - ARM Linux <linux@....linux.org.uk>,
	FUJITA Tomonori <fujita.tomonori@....ntt.co.jp>
CC:	"linux-arm-kernel@...ts.infradead.org" 
	<linux-arm-kernel@...ts.infradead.org>,
	"Hiroshi.DOYU@...ia.com" <Hiroshi.DOYU@...ia.com>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH] scatterlist: define SG chain for arm architecture

> -----Original Message-----
> From: Russell King - ARM Linux [mailto:linux@....linux.org.uk] 
> Sent: Friday, October 01, 2010 11:40 AM
> To: Guzman Lugo, Fernando
> Cc: linux-arm-kernel@...ts.infradead.org; 
> Hiroshi.DOYU@...ia.com; linux-kernel@...r.kernel.org
> Subject: Re: [PATCH] scatterlist: define SG chain for arm architecture
> 
> On Fri, Oct 01, 2010 at 11:28:41AM -0500, Fernando Guzman Lugo wrote:
> > Define SG chain for ARM architecture in order to work with big 
> > buffers. This patch is needed in order to iovmm users can 
> pass a big 
> > buffer to map it and ARM architecture has not issues using SG chain.
> > 
> > This change was suggested by FUJITA Tomonori.
> 
> Have you audited all places where SG lists are used in the 
> ARM architecture to ensure that SG chain support is safe?  
> (That goes for ARM generic code and ARM specific drivers.)

Unfortunately I only can test on omap platform. I am not so
Familiar with scatter gather code and what is the restriction
For one page size of SG list (maybe they can comment). However
For SG chaining seems there is no restriction with the exception
Of using the with function to assigned to page and iterate each
Element of the list and it has sometime when SG chaining was 
Introduce all the code should be use this new functions.

I am agree with you about make sure nothing is broken with the change.
Maybe some people of other arm platforms can comment if they have
Issues with the patch and test the change and if there is no complain
Then merge the change. If there are issues maybe I can send the patch
Just for omap platform. However, looks like there is no issues enabling
It to all arm, but again I am not so familiar with SG, so I will let
The experts to comment.

Thanks a lot,
Fernando.

> 
> If the answer is no, this patch is unacceptable until such a 
> review has been carried out.
> --
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/