On 09/21/2011 02:27 PM, Benny Halevy wrote: >> Unless we do following: >> 1. preallocate memory for extent state convertion >> 2. use nfsiod/rpciod to handle bl_write_cleanup >> 3. for pnfs error case, create a kthread to recollapse and resend to MDS >> I don't quite understand. How do you use nfs state manager to do other tasks? > > You need to keep a list of things to do hanging off of the nfs client structure > and set a bit in cl_state telling the state manager it has work to do > and wake it up. It then needs to go over the list of, say nfs_inodes > and call into the layout driver to handle the errors. > > Benny Good god, Is it not already too complicated? The LD is out of the picture. You all seemed to agree that the LD has reported an io_done on the nfsiod/rpciod, and in the error case Generic layer needs to do it's coalescing on some other thread. So your description above is not correct, the LD is out of the picture. It all looks too complicated for me. A pnfs workqueue for both 2 and 3 above is very good. Specially since the workqueue also shares global pool threads, No? I like it that there is a preallocated thread for the error-case, think about it. Thanks Boaz -- 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
