[PATCH] microblaze: Remove custom lowlevellock.h.

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[PATCH] microblaze: Remove custom lowlevellock.h.

Torvald Riegel-4
microblaze seems to use the same implementation of low-level locks as
the generic Linux lowlevellock.h.  Removing the custom low-level lock
implementation will make further refactoring easier, for example adding
proper error checking to futex operations.

Not tested at all.  David, could you review and test, please?

Thanks!

2014-12-17  Torvald Riegel  <[hidden email]>

        * sysdeps/unix/sysv/linux/microblaze/lowlevellock.h: Remove file.


lll-microblaze.patch (16K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] microblaze: Remove custom lowlevellock.h.

Torvald Riegel-4
Ping.

On Wed, 2014-12-17 at 23:49 +0100, Torvald Riegel wrote:

> microblaze seems to use the same implementation of low-level locks as
> the generic Linux lowlevellock.h.  Removing the custom low-level lock
> implementation will make further refactoring easier, for example adding
> proper error checking to futex operations.
>
> Not tested at all.  David, could you review and test, please?
>
> Thanks!
>
> 2014-12-17  Torvald Riegel  <[hidden email]>
>
> * sysdeps/unix/sysv/linux/microblaze/lowlevellock.h: Remove file.
>



Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] microblaze: Remove custom lowlevellock.h.

David Holsgrove
On 8 January 2015 at 01:45, Torvald Riegel <[hidden email]> wrote:

> Ping.
>
> On Wed, 2014-12-17 at 23:49 +0100, Torvald Riegel wrote:
>> microblaze seems to use the same implementation of low-level locks as
>> the generic Linux lowlevellock.h.  Removing the custom low-level lock
>> implementation will make further refactoring easier, for example adding
>> proper error checking to futex operations.
>>
>> Not tested at all.  David, could you review and test, please?
>>
>> Thanks!
>>
>> 2014-12-17  Torvald Riegel  <[hidden email]>
>>
>>       * sysdeps/unix/sysv/linux/microblaze/lowlevellock.h: Remove file.
>>
>
Committed for you also, thanks Torvald.

David