On Tue, Dec 13, 2022 at 6:17 PM Zhi Li <lznuaa@xxxxxxxxx> wrote: > On Tue, Dec 13, 2022 at 6:08 PM Bjorn Helgaas <helgaas@xxxxxxxxxx> wrote: >> On Tue, Feb 22, 2022 at 10:23:54AM -0600, Frank Li wrote: >> >> > + * +--------------------------------------------------+ Base >> > + * | | >> > + * | | >> > + * | | >> > + * | Common Control Register | >> > + * | | >> > + * | | >> > + * | | >> > + * +-----------------------+--------------------------+ Base+span_offset >> > + * | | | >> > + * | Peer Span Space | Span Space | >> > + * | | | >> > + * | | | >> > + * +-----------------------+--------------------------+ Base+span_offset >> > + * | | | +span_count * 4 >> > + * | | | >> > + * | Span Space | Peer Span Space | >> > + * | | | >> > + * +-----------------------+--------------------------+ >> >> Are these comments supposed to say *spad*, i.e., scratchpad space, >> instead of "span", to correspond with spad_offset and spad_count >> below? > > Strange, I received some of your comments on the very old patches. What's strange about it? I went to the trouble to look up the patch that introduced the thing I'm asking about. I sent the email a few minutes ago. The question still applies to the current tree. Please use plain text email on the Linux mailing lists. Bjorn