On Thu, Nov 13, 2008 at 8:15 AM, Ilya Yanok <yanok@xxxxxxxxxxx> wrote: > Using src_list argument of async_xor() as a storage for dma addresses > implies sizeof(dma_addr_t) <= sizeof(struct page *) restriction which is > not always true. > > Signed-off-by: Ilya Yanok <yanok@xxxxxxxxxxx> > --- I don't like the stack space implications of this change. Especially for large arrays we will be carrying two 'src_cnt' size arrays on the stack, one from MD and one from async_tx. However, I think the current scheme of overwriting input parameters is pretty ugly. So, I want to benchmark the performance implications of adding a GFP_NOIO allocation here, with the idea being that if the allocation fails we can still fallback to the synchronous code path. -- Dan -- To unsubscribe from this list: send the line "unsubscribe linux-raid" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
