On Mon, 12 Sep 2016 15:11:46 -0700 Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> wrote: > > @@ -409,7 +408,7 @@ static int afs_write_back_from_locked_page(struct afs_writeback *wb, > > case -ENOMEDIUM: > > case -ENXIO: > > afs_kill_pages(wb->vnode, true, first, last); > > - set_bit(AS_EIO, &wb->vnode->vfs_inode.i_mapping->flags); > > + mapping_set_error(wb->vnode->vfs_inode.i_mapping, -ENXIO); > > This one is a functional change: mapping_set_error() will rewrite > -ENXIO into -EIO. Doesn't seem at all important though. hm, OK, it's not a functional change - the code was already doing s/ENXIO/EIO/. Let's make it look more truthful? --- a/fs/afs/write.c~fs-use-mapping_set_error-instead-of-opencoded-set_bit-fix +++ a/fs/afs/write.c @@ -408,7 +408,7 @@ no_more: case -ENOMEDIUM: case -ENXIO: afs_kill_pages(wb->vnode, true, first, last); - mapping_set_error(wb->vnode->vfs_inode.i_mapping, -ENXIO); + mapping_set_error(wb->vnode->vfs_inode.i_mapping, -EIO); break; case -EACCES: case -EPERM: _ -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>