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

Re: kern/55781: more rump fixes

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

From: Christos Zoulas <christos%zoulas.com@localhost>
To: Ruslan Nikolaev <nruslan_devel%yahoo.com@localhost>
Cc: gnats-bugs%netbsd.org@localhost,
 kern-bug-people%netbsd.org@localhost,
 gnats-admin%netbsd.org@localhost,
 netbsd-bugs%netbsd.org@localhost
Subject: Re: kern/55781: more rump fixes
Date: Wed, 4 Nov 2020 10:43:01 -0500

 --Apple-Mail=_3FDF9937-27B4-4D07-9FD2-749B157D74D8
 Content-Type: multipart/alternative;
 	boundary="Apple-Mail=_AAFF1261-194E-4F40-A767-FA40CD80CF3F"
 
 
 --Apple-Mail=_AAFF1261-194E-4F40-A767-FA40CD80CF3F
 Content-Transfer-Encoding: quoted-printable
 Content-Type: text/plain;
 	charset=us-ascii
 
 
 
 > On Nov 3, 2020, at 8:00 PM, Ruslan Nikolaev <nruslan_devel%yahoo.com@localhost> =
 wrote:
 >=20
 > Thanks for the feedback! I'll see if I can fix it. In the meantime, I =
 also posted one more SMP-related patch: kern/55781
 >=20
 > We also have rump (glue) files for new drivers such as ixgbe, nvme, =
 etc -- we will post them as well.
 
 How does 55781 work? doesn't "rump_inited" prevent the function to be =
 entered from other cpus?
 
 =
 https://github.com/ssrg-vt/src-netbsd/blob/168bf207298443ea437f83cd5594d89=
 07dd0be97/sys/rump/librump/rumpkern/rump.c#L238 =
 <https://github.com/ssrg-vt/src-netbsd/blob/168bf207298443ea437f83cd5594d8=
 907dd0be97/sys/rump/librump/rumpkern/rump.c#L238>
 
 christos
 
 --Apple-Mail=_AAFF1261-194E-4F40-A767-FA40CD80CF3F
 Content-Transfer-Encoding: quoted-printable
 Content-Type: text/html;
 	charset=us-ascii
 
 <html><head><meta http-equiv=3D"Content-Type" content=3D"text/html; =
 charset=3Dus-ascii"></head><body style=3D"word-wrap: break-word; =
 -webkit-nbsp-mode: space; line-break: after-white-space;" class=3D""><br =
 class=3D""><div><br class=3D""><blockquote type=3D"cite" class=3D""><div =
 class=3D"">On Nov 3, 2020, at 8:00 PM, Ruslan Nikolaev &lt;<a =
 href=3D"mailto:nruslan_devel%yahoo.com@localhost"; =
 class=3D"">nruslan_devel%yahoo.com@localhost</a>&gt; wrote:</div><br =
 class=3D"Apple-interchange-newline"><div class=3D""><span =
 style=3D"caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: =
 12px; font-style: normal; font-variant-caps: normal; font-weight: =
 normal; letter-spacing: normal; text-align: start; text-indent: 0px; =
 text-transform: none; white-space: normal; word-spacing: 0px; =
 -webkit-text-stroke-width: 0px; text-decoration: none; float: none; =
 display: inline !important;" class=3D"">Thanks for the feedback! I'll =
 see if I can fix it. In the meantime, I also posted one more SMP-related =
 patch: kern/55781</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: normal; letter-spacing: normal; =
 text-align: start; text-indent: 0px; text-transform: none; white-space: =
 normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; =
 text-decoration: none;" class=3D""><br style=3D"caret-color: rgb(0, 0, =
 0); font-family: Helvetica; font-size: 12px; font-style: normal; =
 font-variant-caps: normal; font-weight: normal; letter-spacing: normal; =
 text-align: start; text-indent: 0px; text-transform: none; white-space: =
 normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; =
 text-decoration: none;" class=3D""><span style=3D"caret-color: rgb(0, 0, =
 0); font-family: Helvetica; font-size: 12px; font-style: normal; =
 font-variant-caps: normal; font-weight: normal; letter-spacing: normal; =
 text-align: start; text-indent: 0px; text-transform: none; white-space: =
 normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; =
 text-decoration: none; float: none; display: inline !important;" =
 class=3D"">We also have rump (glue) files for new drivers such as ixgbe, =
 nvme, etc -- we will post them as well.</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: normal; letter-spacing: =
 normal; text-align: start; text-indent: 0px; text-transform: none; =
 white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; =
 text-decoration: none;" class=3D""></div></blockquote></div><br =
 class=3D""><div class=3D"">How does 55781 work? doesn't "rump_inited" =
 prevent the function to be entered from other cpus?</div><div =
 class=3D""><br class=3D""></div><div class=3D""><a =
 href=3D"https://github.com/ssrg-vt/src-netbsd/blob/168bf207298443ea437f83c=
 d5594d8907dd0be97/sys/rump/librump/rumpkern/rump.c#L238" =
 class=3D"">https://github.com/ssrg-vt/src-netbsd/blob/168bf207298443ea437f=
 83cd5594d8907dd0be97/sys/rump/librump/rumpkern/rump.c#L238</a></div><div =
 class=3D""><br class=3D""></div><div =
 class=3D"">christos</div></body></html>=
 
 --Apple-Mail=_AAFF1261-194E-4F40-A767-FA40CD80CF3F--
 
 --Apple-Mail=_3FDF9937-27B4-4D07-9FD2-749B157D74D8
 Content-Transfer-Encoding: 7bit
 Content-Disposition: attachment;
 	filename=signature.asc
 Content-Type: application/pgp-signature;
 	name=signature.asc
 Content-Description: Message signed with OpenPGP
 
 -----BEGIN PGP SIGNATURE-----
 Comment: GPGTools - http://gpgtools.org
 
 iF0EARECAB0WIQS+BJlbqPkO0MDBdsRxESqxbLM7OgUCX6LMBQAKCRBxESqxbLM7
 OmiIAKCm43T19qB6si4sSJrGNQci1wUf0QCfVHOsCxNcOHq4InWooDFOlaVO2cQ=
 =1VEz
 -----END PGP SIGNATURE-----
 
 --Apple-Mail=_3FDF9937-27B4-4D07-9FD2-749B157D74D8--
Home | Main Index | Thread Index | Old Index