diff options
| author | Andreas Gruenbacher <agruen@linbit.com> | 2014-09-11 08:29:05 -0400 |
|---|---|---|
| committer | Jens Axboe <axboe@fb.com> | 2014-09-11 10:41:29 -0400 |
| commit | 11f8b2b69d32d43a6d9b45c60c1fee48ab91f440 (patch) | |
| tree | 8a1e1f1a1e09f452eb7dd54d896e9fd7a4b1e885 /drivers/block | |
| parent | 018a17bdc8658ad448497c84d4ba21b6985820ec (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.h | 6 | ||||
| -rw-r--r-- | drivers/block/drbd/drbd_main.c | 28 | ||||
| -rw-r--r-- | drivers/block/drbd/drbd_receiver.c | 46 | ||||
| -rw-r--r-- | drivers/block/drbd/drbd_state.c | 18 |
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 | ||
| 2110 | static inline void put_ldev(struct drbd_device *device) | 2110 | static 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: | |||
