Thread (29 messages) 29 messages, 4 authors, 2020-07-28

Re: [PATCH 01/10] block: introduce blk_is_valid_logical_block_size

From: Christoph Hellwig <hch@lst.de>
Date: 2020-07-21 15:13:21
Also in: linux-mmc, linux-scsi, lkml, virtualization

+/**
+ * blk_check_logical_block_size - check if logical block size is supported
+ * by the kernel
+ * @size:  the logical block size, in bytes
+ *
+ * Description:
+ *   This function checks if the block layers supports given block size
+ **/
+bool blk_is_valid_logical_block_size(unsigned int size)
+{
+	return size >= SECTOR_SIZE && size <= PAGE_SIZE && !is_power_of_2(size);
Shouldn't this be a ... && is_power_of_2(size)?
 	if (q->limits.io_min < q->limits.physical_block_size)
 		q->limits.io_min = q->limits.physical_block_size;
+
 }
This adds a pointless empty line.
+extern bool blk_is_valid_logical_block_size(unsigned int size);
No need for externs on function declarations.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help