> On Mon, Jul 31, 2023 at 09:54:59PM +0900, Jinyoung Choi wrote: > > The bio_integrity_add_page() returns the set length if the execution > > result is successful. Otherwise, return 0. > > > > Unnecessary if statement was removed. And when the result value was less > > than the set value, it was changed to failed. > > Maybe word this as > > bio_integrity_add_page() returns the add length if successful, else 0, > just as bio_add_page. Simply check return value checking in > bio_integrity_prep to not deal with a > 0 but < len case that can't > happen. > > Otherwise looks good: > > Reviewed-by: Christoph Hellwig <hch@xxxxxx> Hi, Christoph. Thank you for your review. I will update comment soon! Best Regards, Jinyoung.
