aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/block
diff options
context:
space:
mode:
authorAndreas Gruenbacher <agruen@linbit.com>2014-09-11 08:29:05 -0400
committerJens Axboe <axboe@fb.com>2014-09-11 10:41:29 -0400
commit11f8b2b69d32d43a6d9b45c60c1fee48ab91f440 (patch)
tree8a1e1f1a1e09f452eb7dd54d896e9fd7a4b1e885 /drivers/block
parent018a17bdc8658ad448497c84d4ba21b6985820ec (diff)
drbd: Use better variable names
Rename local variable 'ds' to 'disk_state' or 'data_size'. 'dgs' to 'digest_size' Signed-off-by: Philipp Reisner <philipp.reisner@linbit.com> Signed-off-by: Lars Ellenberg <lars.ellenberg@linbit.com> Signed-off-by: Jens Axboe <axboe@fb.com>
Diffstat (limited to 'drivers/block')
-rw-r--r--drivers/block/drbd/drbd_int.h6
-rw-r--r--drivers/block/drbd/drbd_main.c28
-rw-r--r--drivers/block/drbd/drbd_receiver.c46
-rw-r--r--drivers/block/drbd/drbd_state.c18
4 files changed, 49 insertions, 49 deletions
diff --git a/drivers/block/drbd/drbd_int.h b/drivers/block/drbd/drbd_int.h
index 1a000016ccdf..9e1288e4148c 100644
--- a/drivers/block/drbd/drbd_int.h
+++ b/drivers/block/drbd/drbd_int.h
@@ -2109,7 +2109,7 @@ static inline bool is_sync_state(enum drbd_conns connection_state)
2109 2109
2110static inline void put_ldev(struct drbd_device *device) 2110static inline void put_ldev(struct drbd_device *device)
2111{ 2111{
2112 enum drbd_disk_state ds = device->state.disk; 2112 enum drbd_disk_state disk_state = device->state.disk;
2113 /* We must check the state *before* the atomic_dec becomes visible, 2113 /* We must check the state *before* the atomic_dec becomes visible,
2114 * or we have a theoretical race where someone hitting zero, 2114 * or we have a theoretical race where someone hitting zero,
2115 * while state still D_FAILED, will then see D_DISKLESS in the 2115 * while state still D_FAILED, will then see D_DISKLESS in the
@@ -2122,10 +2122,10 @@ static inline void put_ldev(struct drbd_device *device)
2122 __release(local); 2122 __release(local);
2123 D_ASSERT(device, i >= 0); 2123 D_ASSERT(device, i >= 0);
2124 if (i == 0) { 2124 if (i == 0) {
2125 if (ds == D_DISKLESS) 2125 if (disk_state == D_DISKLESS)
2126 /* even internal references gone, safe to destroy */ 2126 /* even internal references gone, safe to destroy */
2127 drbd_device_post_work(device, DESTROY_DISK); 2127 drbd_device_post_work(device, DESTROY_DISK);
2128 if (ds == D_FAILED) 2128 if (disk_state == D_FAILED)
2129 /* all application IO references gone. */ 2129 /* all application IO references gone. */
2130 if (!test_and_set_bit(GOING_DISKLESS, &device->flags)) 2130 if (!test_and_set_bit(GOING_DISKLESS, &device->flags))
2131 drbd_device_post_work(device, GO_DISKLESS); 2131 drbd_device_post_work(device, GO_DISKLESS);
diff --git a/drivers/block/drbd/drbd_main.c b/drivers/block/drbd/drbd_main.c
index 9b465bb68487..973c185c9cfe 100644
--- a/drivers/block/drbd/drbd_main.c
+++ b/drivers/block/drbd/drbd_main.c
@@ -1622,13 +1622,13 @@ int drbd_send_dblock(struct drbd_peer_device *peer_device, struct drbd_request *
1622 struct drbd_socket *sock; 1622 struct drbd_socket *sock;
1623 struct p_data *p; 1623 struct p_data *p;
1624 unsigned int dp_flags = 0; 1624 unsigned int dp_flags = 0;
1625 int dgs; 1625 int digest_size;
1626 int err; 1626 int err;
1627 1627
1628 sock = &peer_device->connection->data; 1628 sock = &peer_device->connection->data;
1629 p = drbd_prepare_command(peer_device, sock); 1629 p = drbd_prepare_command(peer_device, sock);
1630 dgs = peer_device->connection->integrity_tfm ? 1630 digest_size = peer_device->connection->integrity_tfm ?
1631 crypto_hash_digestsize(peer_device->connection->integrity_tfm) : 0; 1631 crypto_hash_digestsize(peer_device->connection->integrity_tfm) : 0;
1632 1632
1633 if (!p) 1633 if (!p)
1634 return -EIO; 1634 return -EIO;
@@ -1659,9 +1659,9 @@ int drbd_send_dblock(struct drbd_peer_device *peer_device, struct drbd_request *
1659 1659
1660 /* our digest is still only over the payload. 1660 /* our digest is still only over the payload.
1661 * TRIM does not carry any payload. */ 1661 * TRIM does not carry any payload. */
1662 if (dgs) 1662 if (digest_size)
1663 drbd_csum_bio(peer_device->connection->integrity_tfm, req->master_bio, p + 1); 1663 drbd_csum_bio(peer_device->connection->integrity_tfm, req->master_bio, p + 1);
1664 err = __send_command(peer_device->connection, device->vnr, sock, P_DATA, sizeof(*p) + dgs, NULL, req->i.size); 1664 err = __send_command(peer_device->connection, device->vnr, sock, P_DATA, sizeof(*p) + digest_size, NULL, req->i.size);
1665 if (!err) { 1665 if (!err) {
1666 /* For protocol A, we have to memcpy the payload into 1666 /* For protocol A, we have to memcpy the payload into
1667 * socket buffers, as we may complete right away 1667 * socket buffers, as we may complete right away
@@ -1674,23 +1674,23 @@ int drbd_send_dblock(struct drbd_peer_device *peer_device, struct drbd_request *
1674 * out ok after sending on this side, but does not fit on the 1674 * out ok after sending on this side, but does not fit on the
1675 * receiving side, we sure have detected corruption elsewhere. 1675 * receiving side, we sure have detected corruption elsewhere.
1676 */ 1676 */
1677 if (!(req->rq_state & (RQ_EXP_RECEIVE_ACK | RQ_EXP_WRITE_ACK)) || dgs) 1677 if (!(req->rq_state & (RQ_EXP_RECEIVE_ACK | RQ_EXP_WRITE_ACK)) || digest_size)
1678 err = _drbd_send_bio(peer_device, req->master_bio); 1678 err = _drbd_send_bio(peer_device, req->master_bio);
1679 else 1679 else
1680 err = _drbd_send_zc_bio(peer_device, req->master_bio); 1680 err = _drbd_send_zc_bio(peer_device, req->master_bio);
1681 1681
1682 /* double check digest, sometimes buffers have been modified in flight. */ 1682 /* double check digest, sometimes buffers have been modified in flight. */
1683 if (dgs > 0 && dgs <= 64) { 1683 if (digest_size > 0 && digest_size <= 64) {
1684 /* 64 byte, 512 bit, is the largest digest size 1684 /* 64 byte, 512 bit, is the largest digest size
1685 * currently supported in kernel crypto. */ 1685 * currently supported in kernel crypto. */
1686 unsigned char digest[64]; 1686 unsigned char digest[64];
1687 drbd_csum_bio(peer_device->connection->integrity_tfm, req->master_bio, digest); 1687 drbd_csum_bio(peer_device->connection->integrity_tfm, req->master_bio, digest);
1688 if (memcmp(p + 1, digest, dgs)) { 1688 if (memcmp(p + 1, digest, digest_size)) {
1689 drbd_warn(device, 1689 drbd_warn(device,
1690 "Digest mismatch, buffer modified by upper layers during write: %llus +%u\n", 1690 "Digest mismatch, buffer modified by upper layers during write: %llus +%u\n",
1691 (unsigned long long)req->i.sector, req->i.size); 1691 (unsigned long long)req->i.sector, req->i.size);
1692 } 1692 }
1693 } /* else if (dgs > 64) { 1693 } /* else if (digest_size > 64) {
1694 ... Be noisy about digest too large ... 1694 ... Be noisy about digest too large ...
1695 } */ 1695 } */
1696 } 1696 }
@@ -1711,13 +1711,13 @@ int drbd_send_block(struct drbd_peer_device *peer_device, enum drbd_packet cmd,
1711 struct drbd_socket *sock; 1711 struct drbd_socket *sock;
1712 struct p_data *p; 1712 struct p_data *p;
1713 int err; 1713 int err;
1714 int dgs; 1714 int digest_size;
1715 1715
1716 sock = &peer_device->connection->data; 1716 sock = &peer_device->connection->data;
1717 p = drbd_prepare_command(peer_device, sock); 1717 p = drbd_prepare_command(peer_device, sock);
1718 1718
1719 dgs = peer_device->connection->integrity_tfm ? 1719 digest_size = peer_device->connection->integrity_tfm ?
1720 crypto_hash_digestsize(peer_device->connection->integrity_tfm) : 0; 1720 crypto_hash_digestsize(peer_device->connection->integrity_tfm) : 0;
1721 1721
1722 if (!p) 1722 if (!p)
1723 return -EIO; 1723 return -EIO;
@@ -1725,9 +1725,9 @@ int drbd_send_block(struct drbd_peer_device *peer_device, enum drbd_packet cmd,
1725 p->block_id = peer_req->block_id; 1725 p->block_id = peer_req->block_id;
1726 p->seq_num = 0; /* unused */ 1726 p->seq_num = 0; /* unused */
1727 p->dp_flags = 0; 1727 p->dp_flags = 0;
1728 if (dgs) 1728 if (digest_size)
1729 drbd_csum_ee(peer_device->connection->integrity_tfm, peer_req, p + 1); 1729 drbd_csum_ee(peer_device->connection->integrity_tfm, peer_req, p + 1);
1730 err = __send_command(peer_device->connection, device->vnr, sock, cmd, sizeof(*p) + dgs, NULL, peer_req->i.size); 1730 err = __send_command(peer_device->connection, device->vnr, sock, cmd, sizeof(*p) + digest_size, NULL, peer_req->i.size);
1731 if (!err) 1731 if (!err)
1732 err = _drbd_send_zc_ee(peer_device, peer_req); 1732 err = _drbd_send_zc_ee(peer_device, peer_req);
1733 mutex_unlock(&sock->mutex); /* locked by drbd_prepare_command() */ 1733 mutex_unlock(&sock->mutex); /* locked by drbd_prepare_command() */
diff --git a/drivers/block/drbd/drbd_receiver.c b/drivers/block/drbd/drbd_receiver.c
index 9342b8da73ab..3ae769e671e8 100644
--- a/drivers/block/drbd/drbd_receiver.c
+++ b/drivers/block/drbd/drbd_receiver.c
@@ -1371,9 +1371,9 @@ int drbd_submit_peer_request(struct drbd_device *device,
1371 struct bio *bio; 1371 struct bio *bio;
1372 struct page *page = peer_req->pages; 1372 struct page *page = peer_req->pages;
1373 sector_t sector = peer_req->i.sector; 1373 sector_t sector = peer_req->i.sector;
1374 unsigned ds = peer_req->i.size; 1374 unsigned data_size = peer_req->i.size;
1375 unsigned n_bios = 0; 1375 unsigned n_bios = 0;
1376 unsigned nr_pages = (ds + PAGE_SIZE -1) >> PAGE_SHIFT; 1376 unsigned nr_pages = (data_size + PAGE_SIZE -1) >> PAGE_SHIFT;
1377 int err = -ENOMEM; 1377 int err = -ENOMEM;
1378 1378
1379 if (peer_req->flags & EE_IS_TRIM_USE_ZEROOUT) { 1379 if (peer_req->flags & EE_IS_TRIM_USE_ZEROOUT) {
@@ -1388,7 +1388,7 @@ int drbd_submit_peer_request(struct drbd_device *device,
1388 list_add_tail(&peer_req->w.list, &device->active_ee); 1388 list_add_tail(&peer_req->w.list, &device->active_ee);
1389 spin_unlock_irq(&device->resource->req_lock); 1389 spin_unlock_irq(&device->resource->req_lock);
1390 if (blkdev_issue_zeroout(device->ldev->backing_bdev, 1390 if (blkdev_issue_zeroout(device->ldev->backing_bdev,
1391 sector, ds >> 9, GFP_NOIO)) 1391 sector, data_size >> 9, GFP_NOIO))
1392 peer_req->flags |= EE_WAS_ERROR; 1392 peer_req->flags |= EE_WAS_ERROR;
1393 drbd_endio_write_sec_final(peer_req); 1393 drbd_endio_write_sec_final(peer_req);
1394 return 0; 1394 return 0;
@@ -1426,12 +1426,12 @@ next_bio: