Re: [PATCH 00/10] mtip32xx: various fixes

2016-02-23 Thread Asai Thambi SP

On 2/23/2016 8:14 AM, Jens Axboe wrote:
> On Mon, Feb 22 2016, Asai Thambi SP wrote:
>> Hi Jens,
>>
>> This patchset includes various fixes for mtip32xx driver, tested
>> on kernel 4.5-rc3
>>
>> Selvan Mani (10):
>>   mtip32xx: Abort secure erase when drive is mounted
>>   mtip32xx: Fix broken service thread handling
>>   mtip32xx: Remove unwanted code from taskfile error handler
>>   mtip32xx: Print exact time when an internal command is interrupted
>>   mtip32xx: Avoid issuing standby immediate cmd during FTL rebuild
>>   mtip32xx: Fix for rmmod crash when drive is in FTL rebuild
>>   mtip32xx: Handle safe removal during IO
>>   mtip32xx: Handle FTL rebuild failure state during device
>> initialization
>>   mtip32xx: Add timeout handler
>>   mtip32xx: Cleanup queued requests after surprise removal
> 
> All your patches are mangled, very white space damaged (tabs are 4
> spaces, etc). As mentioned, I don't think the first patch should go in,
> the rest look fine.
> 

I had my email client configured correctly, not sure how the setting changed. I 
will fix the formatting and resend the patchset. 


Re: [PATCH 00/10] mtip32xx: various fixes

2016-02-23 Thread Jens Axboe
On Mon, Feb 22 2016, Asai Thambi SP wrote:
> Hi Jens,
> 
> This patchset includes various fixes for mtip32xx driver, tested
> on kernel 4.5-rc3
> 
> Selvan Mani (10):
>   mtip32xx: Abort secure erase when drive is mounted
>   mtip32xx: Fix broken service thread handling
>   mtip32xx: Remove unwanted code from taskfile error handler
>   mtip32xx: Print exact time when an internal command is interrupted
>   mtip32xx: Avoid issuing standby immediate cmd during FTL rebuild
>   mtip32xx: Fix for rmmod crash when drive is in FTL rebuild
>   mtip32xx: Handle safe removal during IO
>   mtip32xx: Handle FTL rebuild failure state during device
> initialization
>   mtip32xx: Add timeout handler
>   mtip32xx: Cleanup queued requests after surprise removal

All your patches are mangled, very white space damaged (tabs are 4
spaces, etc). As mentioned, I don't think the first patch should go in,
the rest look fine.

-- 
Jens Axboe



[PATCH 00/10] mtip32xx: various fixes

2016-02-22 Thread Asai Thambi SP
Hi Jens,

This patchset includes various fixes for mtip32xx driver, tested
on kernel 4.5-rc3

Selvan Mani (10):
  mtip32xx: Abort secure erase when drive is mounted
  mtip32xx: Fix broken service thread handling
  mtip32xx: Remove unwanted code from taskfile error handler
  mtip32xx: Print exact time when an internal command is interrupted
  mtip32xx: Avoid issuing standby immediate cmd during FTL rebuild
  mtip32xx: Fix for rmmod crash when drive is in FTL rebuild
  mtip32xx: Handle safe removal during IO
  mtip32xx: Handle FTL rebuild failure state during device
initialization
  mtip32xx: Add timeout handler
  mtip32xx: Cleanup queued requests after surprise removal

 drivers/block/mtip32xx/mtip32xx.c |  380 +
 drivers/block/mtip32xx/mtip32xx.h |   13 ++-
 2 files changed, 314 insertions(+), 79 deletions(-)