Re: cifs: bugfix for unreclaimed writeback pages in cifs_writev_requeue()

2013-03-04 Thread Jeff Layton
On Mon, 4 Mar 2013 15:04:49 + Anton Altaparmakov wrote: > Hi, > > The below commit that is present in 3.9-rc1 is buggy. It releases the page > at which point it may no longer exist and then it unlocks it afterwards. > Even if you are somehow getting away with it I think it is an > explo

Re: cifs: bugfix for unreclaimed writeback pages in cifs_writev_requeue()

2013-03-04 Thread Anton Altaparmakov
Hi, The below commit that is present in 3.9-rc1 is buggy. It releases the page at which point it may no longer exist and then it unlocks it afterwards. Even if you are somehow getting away with it I think it is an explosion/memory corruption waiting to happen... Best regards, Anton