Commit 8f291ebf authored by Harald Freudenberger's avatar Harald Freudenberger Committed by Vasily Gorbik

s390/zcrypt: enable card/domain autoselect on ep11 cprbs

For EP11 CPRBs there was only to choose between specify
one or more ep11 targets or not give a target at all. Without
any target the zcrypt code assumed AUTOSELECT. For EP11 this
ended up in choosing any EP11 APQN with regards to the weight.

However, CCA CPRBs can have a more fine granular target
addressing. The caller can give 0xFFFF as AUTOSELECT for
the card and/or the domain. So it's possible to address
any card but domain given or any domain but card given.

This patch now introduces the very same for EP11 CPRB handling.
An EP11 target entry now may contain 0xFFFF as card and/or
domain value with the meaning of ANY card or domain. So
now the same behavior as with CCA CPRBs becomes possible:
Address any card with given domain or address any domain within
given card.

For convenience the zcrypt.h header file now has two new
defines AUTOSEL_AP and AUTOSEL_DOM covering the 0xFFFF
value to address card any and domain any.
Signed-off-by: default avatarHarald Freudenberger <freude@linux.ibm.com>
Signed-off-by: default avatarVasily Gorbik <gor@linux.ibm.com>
parent 7f820d05
...@@ -237,7 +237,9 @@ struct zcrypt_device_matrix_ext { ...@@ -237,7 +237,9 @@ struct zcrypt_device_matrix_ext {
struct zcrypt_device_status_ext device[MAX_ZDEV_ENTRIES_EXT]; struct zcrypt_device_status_ext device[MAX_ZDEV_ENTRIES_EXT];
}; };
#define AUTOSELECT 0xFFFFFFFF #define AUTOSELECT 0xFFFFFFFF
#define AUTOSEL_AP ((__u16) 0xFFFF)
#define AUTOSEL_DOM ((__u16) 0xFFFF)
#define ZCRYPT_IOCTL_MAGIC 'z' #define ZCRYPT_IOCTL_MAGIC 'z'
......
...@@ -849,7 +849,7 @@ static long _zcrypt_send_cprb(struct ap_perms *perms, ...@@ -849,7 +849,7 @@ static long _zcrypt_send_cprb(struct ap_perms *perms,
/* check if device is online and eligible */ /* check if device is online and eligible */
if (!zq->online || if (!zq->online ||
!zq->ops->send_cprb || !zq->ops->send_cprb ||
(tdom != (unsigned short) AUTOSELECT && (tdom != AUTOSEL_DOM &&
tdom != AP_QID_QUEUE(zq->queue->qid))) tdom != AP_QID_QUEUE(zq->queue->qid)))
continue; continue;
/* check if device node has admission for this queue */ /* check if device node has admission for this queue */
...@@ -874,7 +874,7 @@ static long _zcrypt_send_cprb(struct ap_perms *perms, ...@@ -874,7 +874,7 @@ static long _zcrypt_send_cprb(struct ap_perms *perms,
/* in case of auto select, provide the correct domain */ /* in case of auto select, provide the correct domain */
qid = pref_zq->queue->qid; qid = pref_zq->queue->qid;
if (*domain == (unsigned short) AUTOSELECT) if (*domain == AUTOSEL_DOM)
*domain = AP_QID_QUEUE(qid); *domain = AP_QID_QUEUE(qid);
rc = pref_zq->ops->send_cprb(pref_zq, xcRB, &ap_msg); rc = pref_zq->ops->send_cprb(pref_zq, xcRB, &ap_msg);
...@@ -901,7 +901,7 @@ static bool is_desired_ep11_card(unsigned int dev_id, ...@@ -901,7 +901,7 @@ static bool is_desired_ep11_card(unsigned int dev_id,
struct ep11_target_dev *targets) struct ep11_target_dev *targets)
{ {
while (target_num-- > 0) { while (target_num-- > 0) {
if (dev_id == targets->ap_id) if (targets->ap_id == dev_id || targets->ap_id == AUTOSEL_AP)
return true; return true;
targets++; targets++;
} }
...@@ -912,8 +912,11 @@ static bool is_desired_ep11_queue(unsigned int dev_qid, ...@@ -912,8 +912,11 @@ static bool is_desired_ep11_queue(unsigned int dev_qid,
unsigned short target_num, unsigned short target_num,
struct ep11_target_dev *targets) struct ep11_target_dev *targets)
{ {
int card = AP_QID_CARD(dev_qid), dom = AP_QID_QUEUE(dev_qid);
while (target_num-- > 0) { while (target_num-- > 0) {
if (AP_MKQID(targets->ap_id, targets->dom_id) == dev_qid) if ((targets->ap_id == card || targets->ap_id == AUTOSEL_AP) &&
(targets->dom_id == dom || targets->dom_id == AUTOSEL_DOM))
return true; return true;
targets++; targets++;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment