linux-kernel - Re: [PATCH 0 of 9] swiotlb: use phys_addr

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]

Date:	Sat, 27 Dec 2008 17:56:22 +0100
From:	Ingo Molnar <mingo@...e.hu>
To:	FUJITA Tomonori <fujita.tomonori@....ntt.co.jp>
Cc:	jeremy@...p.org, tony.luck@...el.com, linux-kernel@...r.kernel.org,
	xen-devel@...ts.xensource.com, x86@...nel.org,
	ian.campbell@...rix.com, beckyb@...nel.crashing.org
Subject: Re: [PATCH 0 of 9] swiotlb: use phys_addr_t for pages


* FUJITA Tomonori <fujita.tomonori@....ntt.co.jp> wrote:

> On Sat, 27 Dec 2008 11:48:23 +0100
> Ingo Molnar <mingo@...e.hu> wrote:
> 
> > Also, this patch will interact with ia64 materially:
> > 
> >   Subject: [PATCH 9 of 9] ia64/x86/swiotlb: use enum dma_data_direciton in dma_ops
> > 
> > So we'll need a bit more confidence in the testing status of this queue, 
> > and we probably want to wait until Tony merged the pending .29 ia64 
> > changes, and also get an Ack from Tony for these bits:
> > 
> >  arch/ia64/dig/dig_vtd_iommu.c       |    9 +++--
> >  arch/ia64/hp/common/hwsw_iommu.c    |   22 ++++++++------
> >  arch/ia64/hp/common/sba_iommu.c     |   12 ++++---
> >  arch/ia64/include/asm/dma-mapping.h |   29 +++++++++---------
> >  arch/ia64/include/asm/swiotlb.h     |   26 ++++++++++------
> >  arch/ia64/kernel/machvec.c          |    6 ++-
> >  arch/ia64/sn/kernel/io_common.c     |    3 +-
> >  arch/ia64/sn/pci/pci_dma.c          |   21 ++++++++-----
> >  arch/ia64/sn/pci/pcibr/pcibr_dma.c  |    3 +-
> >  arch/ia64/sn/pci/tioca_provider.c   |    3 +-
> > 
> > (Tony Cc:-ed)
> 
> BTW, the above patch breaks IA64 build.
> 
> It's fixable but I think that it would be better to just drop this huge 
> patch and add some cast in powerpc code. the types of SWIOTBL function 
> arguments don't need to mach to DAM_API.txt though it might be nice if 
> so. There are other type mismatches. If we really want to do this 
> cleanups, we should fix up all the mismatches.

hm, we shouldnt do ugly typecasts - lets fix all the places. As Jeremy 
indicated, this is work in progress.

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