On May. 26, 2010, 4:15 +0300, Tao Guo <guotao@xxxxxxxxxxxx> wrote: > On Tue, May 25, 2010 at 11:26 PM, Tao Guo <guotao@xxxxxxxxxxxx> wrote: >> On Tue, May 25, 2010 at 10:40 PM, Benny Halevy <bhalevy@xxxxxxxxxxx> wrote: >>> Boaz Harrosh wrote: >>>> From: Tao Guo <guotao@xxxxxxxxxxxx> >>>> >>>> So in blocklayoutdriver, we can use GFP_KERNEL to do memory >>>> allocation in bl_setup_layoutcommit(). >>>> >>>> The state protected by the lo_lock here is clear now, which is the >>>> layout_commit's position and state. .i.e write_begin/end_pos, >>>> nfsi->lo_cred, and the call to pnfs_get_layout_stateid. >>> >>> Yeah, this looks cleaner. >> Yes, thanks. >>> Tao, can you please test and ack this patch before I merge it? >>> >>> Thanks! >>> >>> Benny > I have tested it, it is OK. > > Tao >> Great, Thanks! I merged this patch for both the 2.6.34 pnfs-all-latest tree and pnfs-all-2.6.33. Benny -- 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