On Thu, Feb 13, 2025 at 02:12:42PM -0400, Jason Gunthorpe wrote: > On Thu, Feb 13, 2025 at 07:55:17PM +0200, Leon Romanovsky wrote: > > On Thu, Feb 13, 2025 at 01:40:43PM -0400, Jason Gunthorpe wrote: > > > On Thu, Feb 13, 2025 at 07:35:10PM +0200, Leon Romanovsky wrote: > > > > > > > Initially curr_base is 0xFF.....FF and curr_len is 0. > > > > > > curr base can't be so unaligned can it? > > > > It is only for first iteration where it is compared with > > sg_dma_address(), immediately after that it is overwritten. > > But this is all working with inherently page aligned stuff, cur_base + > len1 + len2 + len3 + len_n should be page aligned for interior segments.. This is unknown to static code analyze tools. I'm not concerned about logical change, but about possible static code analyze failures. Thanks
