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

[Jessica Clarke]

On 26 Aug 2020, at 20:16, Brandon Bergren <bdragon at imap.cc> wrote:
> 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.

Because it's PowerPC-specific:

sys/conf/files.powerpc:dev/sec/sec.c         optional    sec mpc85xx

Jess