NetBSD-Bugs archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: kern/60189: physio() is unnecessarily brittle

The following reply was made to PR kern/60189; it has been noted by GNATS.

From: Jason Thorpe <thorpej%me.com@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: kern-bug-people%netbsd.org@localhost,
 gnats-admin%netbsd.org@localhost,
 netbsd-bugs%netbsd.org@localhost
Subject: Re: kern/60189: physio() is unnecessarily brittle
Date: Mon, 13 Apr 2026 09:10:03 +0100

 --Apple-Mail=_B4888766-84DC-46B8-A4A5-5D24B7C3A209
 Content-Transfer-Encoding: quoted-printable
 Content-Type: text/plain;
 	charset=utf-8
 
 
 
 > On Apr 13, 2026, at 5:36=E2=80=AFAM, Jason Thorpe <thorpej%me.com@localhost> =
 wrote:
 >=20
 > Alternative solution: a form of uvm_fault_wire() that takes invokes a =
 callback for each page that gets wired, passing the PA of that page for =
 collection to be used later by vmapbuf().
 
 
 =E2=80=A6and thinking about it more, this would be preferable, because =
 then it could be used easily by bus_dmamap_load_uio() (not that anything =
 really uses that API=E2=80=A6)
 
 -- thorpej
 
 
 --Apple-Mail=_B4888766-84DC-46B8-A4A5-5D24B7C3A209
 Content-Transfer-Encoding: quoted-printable
 Content-Type: text/html;
 	charset=utf-8
 
 <html aria-label=3D"message body"><head><meta http-equiv=3D"content-type" =
 content=3D"text/html; charset=3Dutf-8"></head><body =
 style=3D"overflow-wrap: break-word; -webkit-nbsp-mode: space; =
 line-break: after-white-space;"><br =
 id=3D"lineBreakAtBeginningOfMessage"><div><br><blockquote =
 type=3D"cite"><div>On Apr 13, 2026, at 5:36=E2=80=AFAM, Jason Thorpe =
 &lt;thorpej%me.com@localhost&gt; wrote:</div><br =
 class=3D"Apple-interchange-newline"><div><span style=3D"caret-color: =
 rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: =
 normal; font-variant-caps: normal; font-weight: 400; letter-spacing: =
 normal; orphans: 2; text-align: start; text-indent: 0px; text-transform: =
 none; white-space: normal; widows: 2; word-spacing: 0px; =
 -webkit-text-stroke-width: 0px; text-decoration: none; float: none; =
 display: inline !important;">Alternative solution: a form of =
 uvm_fault_wire() that takes invokes a callback for each page that gets =
 wired, passing the PA of that page for collection to be used later by =
 vmapbuf().</span><br style=3D"caret-color: rgb(0, 0, 0); font-family: =
 Helvetica; font-size: 12px; font-style: normal; font-variant-caps: =
 normal; font-weight: 400; letter-spacing: normal; orphans: 2; =
 text-align: start; text-indent: 0px; text-transform: none; white-space: =
 normal; widows: 2; word-spacing: 0px; -webkit-text-stroke-width: 0px; =
 text-decoration-line: none; text-decoration-thickness: auto; =
 text-decoration-style: =
 solid;"></div></blockquote></div><div><br></div><div>=E2=80=A6and =
 thinking about it more, this would be preferable, because then it could =
 be used easily by bus_dmamap_load_uio() (not that anything really uses =
 that API=E2=80=A6)</div><br><div>
 <span class=3D"Apple-style-span" style=3D"border-collapse: separate; =
 color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; =
 font-style: normal; font-variant: normal; font-weight: normal; =
 letter-spacing: normal; line-height: normal; orphans: 2; text-align: =
 auto; text-indent: 0px; text-transform: none; white-space: normal; =
 widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; =
 -webkit-border-vertical-spacing: 0px; =
 -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: =
 auto; -webkit-text-stroke-width: 0px; "><div style=3D"word-wrap: =
 break-word; -webkit-nbsp-mode: space; -webkit-line-break: =
 after-white-space; ">-- thorpej</div></span>
 
 </div>
 <br></body></html>=
 
 --Apple-Mail=_B4888766-84DC-46B8-A4A5-5D24B7C3A209--
Home | Main Index | Thread Index | Old Index