Re: [PATCH v3 5/7] block: Implement support for zoned block devices

Martin,

On 9/28/16 12:54, Martin K. Petersen wrote:

quoted

quoted

quoted

quoted

quoted

"Damien" == Damien Le Moal [off-list ref] writes:

Purely cosmetic: Looks like whitespace is a bit funky (spaces instead of
tabs several places). Also a few typos.

+ * blkdev_report_zones - Get zones information
+ * @bdev:	Target block device
+ * @sector:	Sector from which to report zones
+ * @zones:      Array of zone structures where to return the zones information
+ * @nr_zones:   Number of zone structures in the zone array
+ * @gfp_mask:	Memory allocation flags (for bio_alloc)

Tabs and spaces: Together at last! ^^^^^

+	/*
+	 * Process the report resukt: skip the header and go through the

result

+	/* Check alignement (handle eventual smaller last zone) */

alignment

Thanks for all the comments. Should I send a fixed-up series or just
send correction patches later ?

Best regards.

-- 
Damien Le Moal, Ph.D.
Sr. Manager, System Software Group, HGST Research,
HGST, a Western Digital brand
Damien.LeMoal@hgst.com
(+81) 0466-98-3593 (ext. 513593)
1 kirihara-cho, Fujisawa, 
Kanagawa, 252-0888 Japan
www.hgst.com