On 2011-06-09 14:13, Trond Myklebust wrote: > On Thu, 2011-06-09 at 10:51 -0700, Boaz Harrosh wrote: > >> 1. We will then need to go through the ld in .pg_init > > No. I'll fix nfs_pageio_init_read() and friends by replacing > pnfs_get_read_ops() by something more appropriate. > OK, I just push the pnfs-block stuff out so you can see what I did over your previous version so I'll rebase over the new version once you send it. >> 2. What happens in the none pnfs-IO error case pg_bsize will need to be >> saved and restored to MDS value. > > Yes, but if you are falling back to read/write-through-mds, then you > need to re-run the coalescing _anyway_, since the total length of the > request needs to fit in an rsize/wsize sized request. > Right. So we will need a place holder to keep a separate value for minimum OSD's pg_bsize vs. the MDS's (like what we used to have for DS's [rw]size in the past) Benny >> 3. At least in objects there is no such constant limit, it all depends on >> the layout, start and end. I thought the all point of .pg_test was >> exactly for avoiding a constant pg_bsize. (This is what we had before) >> 4. All "the tests make no sense..." should be moved to the no-pnfs case >> please point these you found out, we'll need to fix them. >> >> Please understand that for none-files layouts pg_bsize is when IO goes >> through MDS only. > > As I said, I don't see how fallback to MDS can work correctly today for > the objects case for arbitrary values of rsize/wsize. > > Cheers > Trond -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html