Re: svn commit: r364799 - in head: share/man/man9 sys/crypto/ccp sys/dev/cxgbe/crypto sys/dev/sec sys/kern sys/opencrypto

[Brandon Bergren]

On Tue, Aug 25, 2020, at 9:37 PM, Alan Somers wrote:
> Author: asomers
> Date: Wed Aug 26 02:37:42 2020
> New Revision: 364799
> URL: https://svnweb.freebsd.org/changeset/base/364799
> 
> Modified: head/sys/dev/sec/sec.c
> ==============================================================================
> --- head/sys/dev/sec/sec.c	Wed Aug 26 02:13:27 2020	(r364798)
> +++ head/sys/dev/sec/sec.c	Wed Aug 26 02:37:42 2020	(r364799)
> @@ -851,6 +851,9 @@ sec_desc_map_dma(struct sec_softc *sc, struct sec_dma_
>  	case CRYPTO_BUF_MBUF:
>  		size = m_length(crp->crp_buf.cb_mbuf, NULL);
>  		break;
> +	case CRYPTO_BUF_VMPAGE:
> +		size = PAGE_SIZE - cb->cb_vm_page_offset;
> +		break;
>  	default:
>  		return (EINVAL);
>  	}

Uh, where is cb coming from? Shouldn't this be using crp->crp_buf.cb_vm_page_offset? This is causing a build failure on powerpc and powerpcspe. I don't see why other platforms aren't also erroring out here.