Re: [patch 03/13] ia64: swiotlb bug fixes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tuesday 06 February 2007 04:33, Luck, Tony wrote:
> On Mon, Feb 05, 2007 at 04:20:00PM -0800, akpm@xxxxxxxxxxxxxxxxxxxx wrote:
> > From: "Jan Beulich" <jbeulich@xxxxxxxxxx>
> > 
> > This patch fixes
> > - marking I-cache clean of pages DMAed to now only done for IA64
> > - broken multiple inclusion in include/asm-x86_64/swiotlb.h
> > - missing call to mark_clean in swiotlb_sync_sg()
> > - a (perhaps only theoretical) issue in swiotlb_dma_supported() when
> >   io_tlb_end is exactly at the end of memory
> > 
> > Signed-off-by: Jan Beulich <jbeulich@xxxxxxxxxx>
> > Cc: "Luck, Tony" <tony.luck@xxxxxxxxx>
> > Cc: Andi Kleen <ak@xxxxxxx>
> > Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
> 
> Andi,
> 
> Are you OK with these changes (and the ones in parts 4, 5, 6 that
> also touch swiotlb)?  They build cleanly on x86_64, but I haven't
> tested them there.

If they build they are ok for me. x86-64 ignores the struct page
bits anyways because everything is coherent, i just never bothered
to ifdef that because they're harmless.

(I suppose an alternative would be to move dma_mark_clean into arch/ia64
and provide a weak dummy default in swiotlb.c) 

-Andi
-
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel]     [Sparc Linux]     [DCCP]     [Linux ARM]     [Yosemite News]     [Linux SCSI]     [Linux x86_64]     [Linux for Ham Radio]

  Powered by Linux