2014-07-20 22:13 GMT+04:00 Shirish Pargaonkar <shirishpargaonkar@xxxxxxxxx>: > Pavel, > > That is correct, this patch does not change that functionality. I was > thinking do we > go through the loop again to attempt to acquire handle (only to > perhaps receive EBADF) > or bail out of cifs_writepages() by marking rest of the pages (till > end) with error flags. Understand. Probably we don't need to go through the loop since find_writable_file() has it's own loop inside (it goes to the start of the search if cifs_reopen_file() fails). So, the existing logic seems right. -- Best regards, Pavel Shilovsky. -- To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
