On 05/04, Ronnie Sahlberg wrote:
+struct cached_dirents {
+ bool is_valid:1;
+ bool is_failed:1;
Does it make sense to have both? Do you expect a situation where is_valid && is_failed happens? From the patch, I don't see such case and the code could be adjusted to use !is_valid where appropriate. But let me know if I missed something. This is just a cosmetic nitpick, but other than that, Reviewed-by: Enzo Matsumiya <ematsumiya@xxxxxxx> Cheers, Enzo
