On Tuesday January 23, [EMAIL PROTECTED] wrote: > On Tue, 23 Jan 2007 11:26:52 +1100 > NeilBrown <[EMAIL PROTECTED]> wrote: > > > + for (j = 0; j < vcnt ; j++) > > + > > memcpy(page_address(sbio->bi_io_vec[j].bv_page), > > + > > page_address(pbio->bi_io_vec[j].bv_page), > > + PAGE_SIZE); > > I trust these BIOs are known to only contain suitably-allocated, MD-private > pages? Because if these pages can be user pages then this change is > spectacularly buggy ;)
Your trust is well placed. This is in the 'resync' path, were all buffers are allocated in lowmem and are full pages and so-forth, so this is perfectly safe. NeilBrown - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
