<html> <head> <meta http-equiv="Content-Type" content="text/html; charset=windows-1252"> </head> <body> You are referring to the wrong document (PXE 2.1 spec for 16-bit UNDIs/legacy BIOS). Appendix E of UEFI specification defines UNDI functions, behaviors and supported opcodes in UEFI. Thanks, Maciej <div class="moz-cite-prefix">On 26-Nov-20 17:20, Sivaraman Nainar wrote: </div> <blockquote type="cite" cite="mid:B4DE137BDB63634BAC03BD9DE765F19702B4B9D428@VENUS1.in.megatrends.com"> <meta http-equiv="Content-Type" content="text/html; charset=windows-1252"> <meta name="Generator" content="Microsoft Word 15 (filtered medium)">  <style> <div class="WordSection1"> Hello Maciej:<o:p></o:p> <o:p> </o:p> As per the PXE Specification: <o:p></o:p> The protocol driver calls UNDI repeatedly with the FuncFlag equal to PXENV_UNDI_ISR_IN_GET_NEXT to get all the buffers in a frame and also all the received<o:p></o:p> frames in the queue. On this call, UNDI must remember the previous buffer given to the protoco,l remove it from the receive queue and recycle it. In case of a multi-buffered frame, if the previous<o:p></o:p> buffer is not the last buffer in the frame it must return the next buffer in the frame in the parameter block. Otherwise it must return the first buffer in the next frame.<o:p></o:p> <o:p> </o:p> Assume the UNDI Driver does not properly Store that TxAddress in the GetStatus then the possibility of MNP Layer having issues when it treats the recycling based on the GetStatus.<o:p></o:p> 2. If a NIC has a pre-allocated region for Tx buffers and cannot dynamically swap them out, UNDI->Transmit() should copy the Tx buffer into appropriate region. UNDI driver should also store that Tx buffer somewhere within the driver, where it will be accessible by UNDI->GetStatus() function. <o:p></o:p> When this point 2 fails the Recycle support done in the SW Layer shall also fail. <o:p></o:p> <o:p> </o:p> Pleases correct my understanding if any . <o:p></o:p> <o:p> </o:p> -Siva<o:p></o:p> <div> <div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in"> From: Rabeda, Maciej [<a class="moz-txt-link-freetext" href="mailto:maciej.rabeda@linux.intel.com">mailto:maciej.rabeda@linux.intel.com</a>] Sent: Thursday, November 26, 2020 9:10 PM To: <a class="moz-txt-link-abbreviated" href="mailto:devel@edk2.groups.io">devel@edk2.groups.io</a>; Sivaraman Nainar Subject: Re: [edk2-devel] reg: PCD To Control MNP Buffer Recycle Support<o:p></o:p> </div> </div> <o:p> </o:p> Hi Nainar, What do you mean by "controllers do not support recycling of Tx buffers"? MNP provides a transmit buffer to SNP (and with EDK2's SNP also to UNDI) and at this point it is considered owned by SNP/UNDI. After UNDI/SNP successfully transmits the packet, it is expected to give it back via UNDI->GetStatus(). Adding network controller (NIC) and UNDI into the picture. 1. UNDI->Transmit() gets a single Tx buffer to put on the wire. 2. If a NIC has a pre-allocated region for Tx buffers and cannot dynamically swap them out, UNDI->Transmit() should copy the Tx buffer into appropriate region. UNDI driver should also store that Tx buffer somewhere within the driver, where it will be accessible by UNDI->GetStatus() function. 3. After successful transmit of Tx buffer, on next UNDI->GetStatus() with PXE_OPFLAGS_GET_TRANSMITTED_BUFFERS OpFlag, driver should put that Tx buffer address in PXE_CDB.DBAddr provided to UNDI->GetStatus() as a parameter. Summing the above - Tx buffer recycling is purely a SW feature. I do not understand your request nor find it valid. Thanks, Maciej<o:p></o:p> <div> On 24-Nov-20 15:30, Sivaraman Nainar wrote:<o:p></o:p> </div> <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt"> Hello all:<o:p></o:p> <o:p></o:p> MNPDxe supported with the Recycle buffer support from the below git commit,<o:p></o:p> <o:p></o:p> <a href="https://github.com/tianocore/edk2/commit/0507449955c5c629cec196b62986afbb91203ed9#diff-fb5b97ad38efea22f5ddd745f6e43ebdb509dc4a5aef81997ba53af5f918a47b" moz-do-not-send="true">https://github.com/tianocore/edk2/commit/0507449955c5c629cec196b62986afbb91203ed9#diff-fb5b97ad38efea22f5ddd745f6e43ebdb509dc4a5aef81997ba53af5f918a47b</a><o:p></o:p> <o:p></o:p> But many network controllers does not support the Recycling Tx Buffers which will cause the PXE Download failure or HTTP Boot failure. <o:p></o:p> <o:p></o:p> Can this feature can be controlled by a Dynamic PCD so that the same code can work for controller which does not support Tx Buffer Recycling.<o:p></o:p> <o:p></o:p> Thanks<o:p></o:p> Siva<o:p></o:p> <div> </div> </blockquote> <o:p> </o:p> </div> </blockquote> </body> </html> <div width="1" style="color:white;clear:both">_._,_._,_</div> <hr> Groups.io Links: You receive all messages sent to this group. <a target="_blank" href="https://edk2.groups.io/g/devel/message/68247">View/Reply Online (#68247)</a> | | <a target="_blank" href="https://groups.io/mt/78477824/1813853">Mute This Topic</a> | <a href="https://edk2.groups.io/g/devel/post">New Topic</a> <a href="https://edk2.groups.io/g/devel/editsub/1813853">Your Subscription</a> | <a href="mailto:devel+owner@edk2.groups.io">Contact Group Owner</a> | <a href="https://edk2.groups.io/g/devel/unsub">Unsubscribe</a> [edk2-devel-archive@redhat.com] <div width="1" style="color:white;clear:both">_._,_._,_</div>