PERFORCE change 1200830 for review
John-Mark Gurney
jmg at FreeBSD.org
Fri Sep 26 12:38:16 UTC 2014
http://p4web.freebsd.org/@@1200830?ac=10
Change 1200830 by jmg at jmg_carbon2 on 2014/09/26 12:38:07
remove CRYPTO_F_REL flag... One driver implemented it, but didn't
do anything different, only one consumer set it, but expected the
same behavior... both NetBSD and OpenBSD have similar behaviors...
The normal behavior IS that the data is in the same location...
Affected files ...
.. //depot/projects/opencrypto/share/man/man9/crypto.9#5 edit
.. //depot/projects/opencrypto/sys/geom/eli/g_eli_crypto.c#3 edit
.. //depot/projects/opencrypto/sys/geom/eli/g_eli_integrity.c#3 edit
.. //depot/projects/opencrypto/sys/geom/eli/g_eli_privacy.c#3 edit
.. //depot/projects/opencrypto/sys/mips/rmi/dev/sec/rmisec.c#2 edit
.. //depot/projects/opencrypto/sys/opencrypto/cryptodev.h#10 edit
Differences ...
==== //depot/projects/opencrypto/share/man/man9/crypto.9#5 (text+ko) ====
@@ -304,8 +304,6 @@
is an
.Vt uio
structure.
-.It Dv CRYPTO_F_REL
-Must return data in the same place.
.It Dv CRYPTO_F_BATCH
Batch operation if possible.
.It Dv CRYPTO_F_CBIMM
==== //depot/projects/opencrypto/sys/geom/eli/g_eli_crypto.c#3 (text+ko) ====
@@ -101,7 +101,7 @@
crp->crp_opaque = NULL;
crp->crp_callback = g_eli_crypto_done;
crp->crp_buf = (void *)data;
- crp->crp_flags = CRYPTO_F_CBIFSYNC | CRYPTO_F_REL;
+ crp->crp_flags = CRYPTO_F_CBIFSYNC;
crp->crp_desc = crd;
error = crypto_dispatch(crp);
==== //depot/projects/opencrypto/sys/geom/eli/g_eli_integrity.c#3 (text+ko) ====
@@ -479,7 +479,7 @@
crp->crp_opaque = (void *)bp;
crp->crp_buf = (void *)data;
data += encr_secsize;
- crp->crp_flags = CRYPTO_F_CBIFSYNC | CRYPTO_F_REL;
+ crp->crp_flags = CRYPTO_F_CBIFSYNC;
if (g_eli_batch)
crp->crp_flags |= CRYPTO_F_BATCH;
if (bp->bio_cmd == BIO_WRITE) {
==== //depot/projects/opencrypto/sys/geom/eli/g_eli_privacy.c#3 (text+ko) ====
@@ -286,7 +286,7 @@
crp->crp_callback = g_eli_crypto_write_done;
else /* if (bp->bio_cmd == BIO_READ) */
crp->crp_callback = g_eli_crypto_read_done;
- crp->crp_flags = CRYPTO_F_CBIFSYNC | CRYPTO_F_REL;
+ crp->crp_flags = CRYPTO_F_CBIFSYNC;
if (g_eli_batch)
crp->crp_flags |= CRYPTO_F_BATCH;
crp->crp_desc = crd;
==== //depot/projects/opencrypto/sys/mips/rmi/dev/sec/rmisec.c#2 (text+ko) ====
@@ -423,13 +423,8 @@
cmd->op.source_buf = (uint64_t) (unsigned long)crp->crp_buf;
cmd->op.source_buf_size = crp->crp_ilen;
- if (crp->crp_flags & CRYPTO_F_REL) {
- cmd->op.dest_buf = (uint64_t) (unsigned long)crp->crp_buf;
- cmd->op.dest_buf_size = crp->crp_ilen;
- } else {
- cmd->op.dest_buf = (uint64_t) (unsigned long)crp->crp_buf;
- cmd->op.dest_buf_size = crp->crp_ilen;
- }
+ cmd->op.dest_buf = (uint64_t) (unsigned long)crp->crp_buf;
+ cmd->op.dest_buf_size = crp->crp_ilen;
cmd->op.num_packets = 1;
cmd->op.num_fragments = 1;
==== //depot/projects/opencrypto/sys/opencrypto/cryptodev.h#10 (text+ko) ====
@@ -368,7 +368,6 @@
#define CRYPTO_F_IMBUF 0x0001 /* Input/output are mbuf chains */
#define CRYPTO_F_IOV 0x0002 /* Input/output are uio */
-#define CRYPTO_F_REL 0x0004 /* Must return data in same place */
#define CRYPTO_F_BATCH 0x0008 /* Batch op if possible */
#define CRYPTO_F_CBIMM 0x0010 /* Do callback immediately */
#define CRYPTO_F_DONE 0x0020 /* Operation completed */
More information about the p4-projects
mailing list