Thread (85 messages) 85 messages, 6 authors, 2021-03-30

RE: [PATCH V3 07/25] smartpqi: update AIO Sub Page 0x02 support

From: <Don.Brace@microchip.com>
Date: 2021-01-11 20:55:07

-----Original Message-----
From: Martin Wilck [mailto:mwilck@suse.com] 
Subject: Re: [PATCH V3 07/25] smartpqi: update AIO Sub Page 0x02 support

On Thu, 2020-12-10 at 14:35 -0600, Don Brace wrote:
From: Kevin Barnett <redacted>

The specification for AIO Sub-Page (0x02) has changed slightly.
* bring the driver into conformance with the spec.


+static inline u32 pqi_aio_limit_to_bytes(__le16 *limit) {
+       u32 bytes;
+
+       bytes = get_unaligned_le16(limit);
+       if (bytes == 0)
+               bytes = ~0;
+       else
+               bytes *= 1024;
+
+       return bytes;
+}
Nice, but this function and it's callers belong into patch 06/25.

Don: 
      * Squashed smartpqi-update-AIO-Sub-Page-0x02-support
      * Moved formatting HUNK for pqi_scsi_dev_raid_map_data into
        smartpqi-refactor-aio-submission-code
      * Moved structure pqi_aio_r56_path_request formatting HUNKS into
        smartpqi-add-support-for-raid5-and-raid6-writes
      * Moved remaining formatting HUNKs into
        smartpqi-align-code-with-oob-driver

Thanks for all of your attention to detail,
Don
quoted hunk ↗ jump to hunk
+
 #pragma pack(1)


 static int pqi_get_advanced_raid_bypass_config(struct pqi_ctrl_info
*ctrl_info)
@@ -753,33 +766,28 @@ static int
pqi_get_advanced_raid_bypass_config(struct pqi_ctrl_info *ctrl_info)
                        BMIC_SENSE_FEATURE_IO_PAGE_AIO_SUBPAGE ||
                get_unaligned_le16(&buffer-
quoted
aio_subpage.header.page_length) <
                        MINIMUM_AIO_SUBPAGE_LENGTH) {
-               rc = -EINVAL;
This should be changed in 06/25.
                goto error;
        }
Regards
Martin


Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help