[edk2] [PATCH 0/4] SataSiI3132Dxe fixes

Daniil Egranov posted 4 patches 6 years, 5 months ago
Failed in applying to current master (apply log)
EmbeddedPkg/Drivers/SataSiI3132Dxe/SataSiI3132.c   | 301 ++++++++++++++++-----
EmbeddedPkg/Drivers/SataSiI3132Dxe/SataSiI3132.h   |  17 ++
.../Drivers/SataSiI3132Dxe/SiI3132AtaPassThru.c    |   4 +-
3 files changed, 252 insertions(+), 70 deletions(-)
[edk2] [PATCH 0/4] SataSiI3132Dxe fixes
Posted by Daniil Egranov 6 years, 5 months ago
This set of patches fixes an issue with 64-bit DMA and implements
the missing exit boot event and driver stop functionality including
memory/protocols cleanup procedure.

Daniil Egranov (4):
  Drivers/SataSiI3132Dxe: Fixed PCI IO read and write operations
  Drivers/SataSiI3132Dxe: Allow 64-bit DMA transfer
  Drivers/SataSiI3132Dxe: Enable multi-controller support
  Drivers/SataSiI3132Dxe: Fixed startup and shutdown procedures

 EmbeddedPkg/Drivers/SataSiI3132Dxe/SataSiI3132.c   | 301 ++++++++++++++++-----
 EmbeddedPkg/Drivers/SataSiI3132Dxe/SataSiI3132.h   |  17 ++
 .../Drivers/SataSiI3132Dxe/SiI3132AtaPassThru.c    |   4 +-
 3 files changed, 252 insertions(+), 70 deletions(-)

-- 
2.11.0

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Re: [edk2] [PATCH 0/4] SataSiI3132Dxe fixes
Posted by Ard Biesheuvel 6 years, 5 months ago
On 27 October 2017 at 06:33, Daniil Egranov <daniil.egranov@arm.com> wrote:
> This set of patches fixes an issue with 64-bit DMA and implements
> the missing exit boot event and driver stop functionality including
> memory/protocols cleanup procedure.
>
> Daniil Egranov (4):
>   Drivers/SataSiI3132Dxe: Fixed PCI IO read and write operations
>   Drivers/SataSiI3132Dxe: Allow 64-bit DMA transfer
>   Drivers/SataSiI3132Dxe: Enable multi-controller support
>   Drivers/SataSiI3132Dxe: Fixed startup and shutdown procedures
>
>  EmbeddedPkg/Drivers/SataSiI3132Dxe/SataSiI3132.c   | 301 ++++++++++++++++-----
>  EmbeddedPkg/Drivers/SataSiI3132Dxe/SataSiI3132.h   |  17 ++
>  .../Drivers/SataSiI3132Dxe/SiI3132AtaPassThru.c    |   4 +-
>  3 files changed, 252 insertions(+), 70 deletions(-)
>

Hi Daniil,

Thanks for taking the time to fix this driver.

I will go ahead and push the first two patches, given that they are
self-contained and obvious bug fixes.

The remaining patches, please split them up, and please align more
closely with what other upstream PCI drivers do.

Regards,
Ard.
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Re: [edk2] [PATCH 0/4] SataSiI3132Dxe fixes
Posted by Jeremy Linton 6 years, 5 months ago
Hi,

On 10/27/2017 12:33 AM, Daniil Egranov wrote:
> This set of patches fixes an issue with 64-bit DMA and implements
> the missing exit boot event and driver stop functionality including
> memory/protocols cleanup procedure.
> 
> Daniil Egranov (4):
>    Drivers/SataSiI3132Dxe: Fixed PCI IO read and write operations
>    Drivers/SataSiI3132Dxe: Allow 64-bit DMA transfer
>    Drivers/SataSiI3132Dxe: Enable multi-controller support
>    Drivers/SataSiI3132Dxe: Fixed startup and shutdown procedures
> 
>   EmbeddedPkg/Drivers/SataSiI3132Dxe/SataSiI3132.c   | 301 ++++++++++++++++-----
>   EmbeddedPkg/Drivers/SataSiI3132Dxe/SataSiI3132.h   |  17 ++
>   .../Drivers/SataSiI3132Dxe/SiI3132AtaPassThru.c    |   4 +-
>   3 files changed, 252 insertions(+), 70 deletions(-)

This is generally good, but there remain quite a number of "errors" in 
the command submission path as well as the completely unnecessary 4k IO 
alignment requirement which has been known to break older grubs/etc. A 
few of those "errors" were fixed in this patch set (1) as well, so might 
be worthwhile if you are looking at this driver to integrate those fixes 
as well.

(1) https://lists.01.org/pipermail/edk2-devel/2017-March/008277.html

Thanks,
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Re: [edk2] [PATCH 0/4] SataSiI3132Dxe fixes
Posted by Ard Biesheuvel 6 years, 5 months ago
On 27 October 2017 at 17:57, Jeremy Linton <jeremy.linton@arm.com> wrote:
> Hi,
>
> On 10/27/2017 12:33 AM, Daniil Egranov wrote:
>>
>> This set of patches fixes an issue with 64-bit DMA and implements
>> the missing exit boot event and driver stop functionality including
>> memory/protocols cleanup procedure.
>>
>> Daniil Egranov (4):
>>    Drivers/SataSiI3132Dxe: Fixed PCI IO read and write operations
>>    Drivers/SataSiI3132Dxe: Allow 64-bit DMA transfer
>>    Drivers/SataSiI3132Dxe: Enable multi-controller support
>>    Drivers/SataSiI3132Dxe: Fixed startup and shutdown procedures
>>
>>   EmbeddedPkg/Drivers/SataSiI3132Dxe/SataSiI3132.c   | 301
>> ++++++++++++++++-----
>>   EmbeddedPkg/Drivers/SataSiI3132Dxe/SataSiI3132.h   |  17 ++
>>   .../Drivers/SataSiI3132Dxe/SiI3132AtaPassThru.c    |   4 +-
>>   3 files changed, 252 insertions(+), 70 deletions(-)
>
>
> This is generally good, but there remain quite a number of "errors" in the
> command submission path as well as the completely unnecessary 4k IO
> alignment requirement which has been known to break older grubs/etc. A few
> of those "errors" were fixed in this patch set (1) as well, so might be
> worthwhile if you are looking at this driver to integrate those fixes as
> well.
>

Yes, please. And apologies for forgetting about thise patches.

> (1) https://lists.01.org/pipermail/edk2-devel/2017-March/008277.html
>
> Thanks,
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel