>> … >>> +++ b/fs/exfat/dir.c >> … >>> +static int exfat_readdir(struct inode *inode, struct exfat_dir_entry >> *dir_entry) >>> +{ >> … >>> + if (!ep) { >>> + ret = -EIO; >>> + goto free_clu; >>> + } >> >> How do you think about to move a bit of common exception handling code >> (at similar places)? > Not sure it is good. The software development opinions can vary also for this change pattern according to different design goals. Is such a transformation just another possibility to reduce duplicate source code a bit? Regards, Markus