Re: git: e133103ab6b3 - main - mrsas(4) fix build on targets that don't define PTRIN.

On Fri, Apr 28, 2023 at 11:38:46PM +0300, Dmitry Chagin wrote:
| On Fri, Apr 28, 2023 at 08:15:52PM +0000, Doug Ambrisko wrote:
| > The branch main has been updated by ambrisko:
| > 
| > URL: https://cgit.FreeBSD.org/src/commit/?id=e133103ab6b35d96d06f24825cff6caa298f595a
| > 
| > commit e133103ab6b35d96d06f24825cff6caa298f595a
| > Author:     Doug Ambrisko <ambrisko@FreeBSD.org>
| > AuthorDate: 2023-04-28 20:12:32 +0000
| > Commit:     Doug Ambrisko <ambrisko@FreeBSD.org>
| > CommitDate: 2023-04-28 20:15:43 +0000
| > 
| >     mrsas(4) fix build on targets that don't define PTRIN.
| 
| just
| #include <sys/abi_compat.h>

I wasn't sure and wanted to get the builds fixed and didn't see that
in sys/dev/mfi/* just the define ... but just defining it failed on
amd64!

Thanks for the tip.  I'm testing a build of kernels with that now.

Doug A.
 
| > ---
| >  sys/dev/mrsas/mrsas_ioctl.c | 4 ++++
| >  1 file changed, 4 insertions(+)
| > 
| > diff --git a/sys/dev/mrsas/mrsas_ioctl.c b/sys/dev/mrsas/mrsas_ioctl.c
| > index b8d88c164e81..7f1d9a91ba0f 100644
| > --- a/sys/dev/mrsas/mrsas_ioctl.c
| > +++ b/sys/dev/mrsas/mrsas_ioctl.c
| > @@ -55,6 +55,10 @@ struct mrsas_passthru_cmd {
| >  	int complete;
| >  };
| >  
| > +#ifndef PTRIN
| > +#define PTRIN(p) ((void *)(uintptr_t)(p))
| > +#endif
| > +
| >  /*
| >   * Function prototypes
| >   */