<div dir="ltr"><p class="gmail-isSelectedEnd">Hello,</p><p class="gmail-isSelectedEnd">Thank you for the detailed review. I am reworking the series as an RFC for the EAL DMA API and PCI bus hook, with igb as the first user.</p><p class="gmail-isSelectedEnd">DMA properties now come from each device’s host bridge, and cache handling is in EAL. I will also revise RX buffer reuse, TX completion, and descriptor ownership, and add separate burst functions for non-coherent DMA.</p><p class="gmail-isSelectedEnd">I will also remove the extra guide and shorten the comments.</p><p>Best regards,<br>Rayhan</p></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Mon, Sep 21, 2026 at 11:42 AM Stephen Hemminger <<a href="mailto:stephen@networkplumber.org">stephen@networkplumber.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Mon, 21 Sep 2026 09:56:17 +0100<br>
Bruce Richardson <<a href="mailto:bruce.richardson@intel.com" target="_blank">bruce.richardson@intel.com</a>> wrote:<br>
<br>
> > <br>
> > Short observations:<br>
> >   1. Too much AI generated slop, extra docs, comments on everything.<br>
> >      New code should look like the surrounding code.<br>
> >      This looks like AI wasn't quite sure and left lots of docs for future self.  <br>
> <br>
> I really don't view this as a bad thing. A little too much detail in the<br>
> comments is better for the future than too little. Given these patches<br>
> touch areas outside the usual behaviour we expect from DPDK platforms,<br>
> having extra comments is definitely useful.<br>
> <br>
> <br>
> /Bruce<br>
<br>
There is a fine line between a couple of sentences and three paragraphs<br>
</blockquote></div>