Date: Fri, 5 Feb 2021 17:12:50 +0000 (UTC) From: Justin Hibbits <chmeeedalf@gmail.com> To: Andriy Gapon <avg@FreeBSD.org> Cc: Alfredo Dal'Ava Junior <alfredo@FreeBSD.org>, src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org Subject: Re: git: 6bf9668d9d02 - stable/13 - mrsas: unbreak i386 build Message-ID: <5066e1e2-e4ce-48c3-8015-74810e7b727c@gmail.com> In-Reply-To: <9f35a234-4c6f-7c86-523f-824dc52656aa@FreeBSD.org> References: <202102051652.115GqYVS049749@gitrepo.freebsd.org> <9f35a234-4c6f-7c86-523f-824dc52656aa@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Feb 5, 2021 10:56:13 Andriy Gapon <avg@FreeBSD.org>: > On 05/02/2021 18:52, Alfredo Dal'Ava Junior wrote: >> The branch stable/13 has been updated by alfredo: >> >> URL: https://cgit.FreeBSD.org/src/commit/?id=3D6bf9668d9d02bdf3e0dafc4ba= 1010ef6a9b12247 >> >> commit 6bf9668d9d02bdf3e0dafc4ba1010ef6a9b12247 >> Author:=C2=A0=C2=A0=C2=A0=C2=A0 Alfredo Dal'Ava Junior <alfredo@FreeBSD.= org> >> AuthorDate: 2021-02-05 18:43:48 +0000 >> Commit:=C2=A0=C2=A0=C2=A0=C2=A0 Alfredo Dal'Ava Junior <alfredo@FreeBSD.= org> >> CommitDate: 2021-02-05 19:50:21 +0000 >> >> =C2=A0=C2=A0=C2=A0 mrsas: unbreak i386 build >> =C2=A0=C2=A0=C2=A0 >> =C2=A0=C2=A0=C2=A0 Fix build regression introduced by >> =C2=A0=C2=A0=C2=A0 e34a057ca6ebdf8e30ec8b0dc21d18eb450bf36a >> =C2=A0=C2=A0=C2=A0 >> =C2=A0=C2=A0=C2=A0 Reviewed by:=C2=A0=C2=A0=C2=A0 jhibbits > > Was it?=C2=A0 I do not see an approval in the review request. Yes,=C2=A0 it was reviewed by me,=C2=A0 since he took exactly what I sugges= ted, and we discussed over IRC. > >> =C2=A0=C2=A0=C2=A0 Sponsored by:=C2=A0=C2=A0 Eldorado Research Institute= (eldorado.org.br) >> =C2=A0=C2=A0=C2=A0 Differential Revision:=C2=A0 https://reviews.freebsd.= org/D28494 >> =C2=A0=C2=A0=C2=A0 >> =C2=A0=C2=A0=C2=A0 (cherry picked from commit 59fffbcf46ba6369420e655ff9= 173571410c643d) > > Why the insta-MFC _again_? > Especially while there is still a discussion in the review request. Agree here, but not sure the best resolution for the breakage in this case. > >> sys/dev/mrsas/mrsas.c | 4 +--- >> 1 file changed, 1 insertion(+), 3 deletions(-) >> >> diff --git a/sys/dev/mrsas/mrsas.c b/sys/dev/mrsas/mrsas.c >> index f4c34e237fc4..01173387c8d4 100644 >> --- a/sys/dev/mrsas/mrsas.c >> +++ b/sys/dev/mrsas/mrsas.c >> @@ -2782,9 +2782,7 @@ mrsas_ioc_init(struct mrsas_softc *sc) >> =C2=A0 init_frame->queue_info_new_phys_addr_lo =3D htole32(phys_addr); >> =C2=A0 init_frame->data_xfer_len =3D htole32(sizeof(Mpi2IOCInitRequest_t= )); >> >> - req_desc.addr.u.low =3D htole32((bus_addr_t)sc->ioc_init_phys_mem & 0x= FFFFFFFF); >> - req_desc.addr.u.high =3D htole32((bus_addr_t)sc->ioc_init_phys_mem >> = 32); >> - >> + req_desc.addr.Words =3D htole64((bus_addr_t)sc->ioc_init_phys_mem); >> =C2=A0 req_desc.MFAIo.RequestFlags =3D >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 (MRSAS_REQ_DESCRIPT_FLAGS_MFA << MRSAS_RE= Q_DESCRIPT_FLAGS_TYPE_SHIFT); >> >> > > > -- > Andriy Gapon - Justin
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?5066e1e2-e4ce-48c3-8015-74810e7b727c>