Commit 0d48566d authored by Gerd Bayer's avatar Gerd Bayer Committed by Heiko Carstens

s390/pci: rename lock member in struct zpci_dev

Since this guards only the Function Measurement Block, rename from
generic lock to fmb_lock in preparation to introduce another lock
that guards the state member
Signed-off-by: default avatarGerd Bayer <gbayer@linux.ibm.com>
Reviewed-by: default avatarNiklas Schnelle <schnelle@linux.ibm.com>
Signed-off-by: default avatarHeiko Carstens <hca@linux.ibm.com>
parent 29f6fe17
...@@ -142,7 +142,6 @@ struct zpci_dev { ...@@ -142,7 +142,6 @@ struct zpci_dev {
u8 reserved : 2; u8 reserved : 2;
unsigned int devfn; /* DEVFN part of the RID*/ unsigned int devfn; /* DEVFN part of the RID*/
struct mutex lock;
u8 pfip[CLP_PFIP_NR_SEGMENTS]; /* pci function internal path */ u8 pfip[CLP_PFIP_NR_SEGMENTS]; /* pci function internal path */
u32 uid; /* user defined id */ u32 uid; /* user defined id */
u8 util_str[CLP_UTIL_STR_LEN]; /* utility string */ u8 util_str[CLP_UTIL_STR_LEN]; /* utility string */
...@@ -170,6 +169,7 @@ struct zpci_dev { ...@@ -170,6 +169,7 @@ struct zpci_dev {
u64 dma_mask; /* DMA address space mask */ u64 dma_mask; /* DMA address space mask */
/* Function measurement block */ /* Function measurement block */
struct mutex fmb_lock;
struct zpci_fmb *fmb; struct zpci_fmb *fmb;
u16 fmb_update; /* update interval */ u16 fmb_update; /* update interval */
u16 fmb_length; u16 fmb_length;
......
...@@ -806,7 +806,7 @@ struct zpci_dev *zpci_create_device(u32 fid, u32 fh, enum zpci_state state) ...@@ -806,7 +806,7 @@ struct zpci_dev *zpci_create_device(u32 fid, u32 fh, enum zpci_state state)
zdev->state = state; zdev->state = state;
kref_init(&zdev->kref); kref_init(&zdev->kref);
mutex_init(&zdev->lock); mutex_init(&zdev->fmb_lock);
mutex_init(&zdev->kzdev_lock); mutex_init(&zdev->kzdev_lock);
rc = zpci_init_iommu(zdev); rc = zpci_init_iommu(zdev);
......
...@@ -91,9 +91,9 @@ static int pci_perf_show(struct seq_file *m, void *v) ...@@ -91,9 +91,9 @@ static int pci_perf_show(struct seq_file *m, void *v)
if (!zdev) if (!zdev)
return 0; return 0;
mutex_lock(&zdev->lock); mutex_lock(&zdev->fmb_lock);
if (!zdev->fmb) { if (!zdev->fmb) {
mutex_unlock(&zdev->lock); mutex_unlock(&zdev->fmb_lock);
seq_puts(m, "FMB statistics disabled\n"); seq_puts(m, "FMB statistics disabled\n");
return 0; return 0;
} }
...@@ -130,7 +130,7 @@ static int pci_perf_show(struct seq_file *m, void *v) ...@@ -130,7 +130,7 @@ static int pci_perf_show(struct seq_file *m, void *v)
} }
pci_sw_counter_show(m); pci_sw_counter_show(m);
mutex_unlock(&zdev->lock); mutex_unlock(&zdev->fmb_lock);
return 0; return 0;
} }
...@@ -148,7 +148,7 @@ static ssize_t pci_perf_seq_write(struct file *file, const char __user *ubuf, ...@@ -148,7 +148,7 @@ static ssize_t pci_perf_seq_write(struct file *file, const char __user *ubuf,
if (rc) if (rc)
return rc; return rc;
mutex_lock(&zdev->lock); mutex_lock(&zdev->fmb_lock);
switch (val) { switch (val) {
case 0: case 0:
rc = zpci_fmb_disable_device(zdev); rc = zpci_fmb_disable_device(zdev);
...@@ -157,7 +157,7 @@ static ssize_t pci_perf_seq_write(struct file *file, const char __user *ubuf, ...@@ -157,7 +157,7 @@ static ssize_t pci_perf_seq_write(struct file *file, const char __user *ubuf,
rc = zpci_fmb_enable_device(zdev); rc = zpci_fmb_enable_device(zdev);
break; break;
} }
mutex_unlock(&zdev->lock); mutex_unlock(&zdev->fmb_lock);
return rc ? rc : count; return rc ? rc : count;
} }
......
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