Re: [PATCH v21 00/12] Add 32 bit VDSO time function support

2014-03-02 Thread H. Peter Anvin
On 03/02/2014 08:32 AM, Andy Lutomirski wrote:
> 
> OTOH, hpa may prefer incremental patches, now that this lives in tip.
> 

The branch currently in -tip is dead, so it is not an issue either way.

-hpa


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH v21 00/12] Add 32 bit VDSO time function support

2014-03-02 Thread Andy Lutomirski
On Sun, Mar 2, 2014 at 2:47 AM, Ingo Molnar  wrote:
>
> * Stefani Seibold  wrote:
>
>> This patch add the functions vdso_gettimeofday(), vdso_clock_gettime()
>> and vdso_time() to the 32 bit VDSO.
>
> What I'm missing from all the series is any trace of the significant
> review and debug work that Andy Lutomirski did for the series. Please
> add Acked-by or Reviewed-by tags to credit his contributions, as
> appropriate. (if Andy is fine with that.)
>

I'm okay w/ Reviewed-by, except for the patch for 32-bit vdso on
64-bit, which I haven't really reviewed (nor do I feel fully qualified
to review it -- it's a bit of a core change, and I'd prefer for
someone involved in that code to comment on it).

For the patches I wrote, "From" would be nice :)

OTOH, hpa may prefer incremental patches, now that this lives in tip.

--Andy
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH v21 00/12] Add 32 bit VDSO time function support

2014-03-02 Thread Ingo Molnar

* Stefani Seibold  wrote:

> This patch add the functions vdso_gettimeofday(), vdso_clock_gettime()
> and vdso_time() to the 32 bit VDSO.

What I'm missing from all the series is any trace of the significant 
review and debug work that Andy Lutomirski did for the series. Please 
add Acked-by or Reviewed-by tags to credit his contributions, as 
appropriate. (if Andy is fine with that.)

Thanks,

Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH v21 00/12] Add 32 bit VDSO time function support

2014-03-02 Thread Ingo Molnar

* Stefani Seibold stef...@seibold.net wrote:

 This patch add the functions vdso_gettimeofday(), vdso_clock_gettime()
 and vdso_time() to the 32 bit VDSO.

What I'm missing from all the series is any trace of the significant 
review and debug work that Andy Lutomirski did for the series. Please 
add Acked-by or Reviewed-by tags to credit his contributions, as 
appropriate. (if Andy is fine with that.)

Thanks,

Ingo
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH v21 00/12] Add 32 bit VDSO time function support

2014-03-02 Thread Andy Lutomirski
On Sun, Mar 2, 2014 at 2:47 AM, Ingo Molnar mi...@kernel.org wrote:

 * Stefani Seibold stef...@seibold.net wrote:

 This patch add the functions vdso_gettimeofday(), vdso_clock_gettime()
 and vdso_time() to the 32 bit VDSO.

 What I'm missing from all the series is any trace of the significant
 review and debug work that Andy Lutomirski did for the series. Please
 add Acked-by or Reviewed-by tags to credit his contributions, as
 appropriate. (if Andy is fine with that.)


I'm okay w/ Reviewed-by, except for the patch for 32-bit vdso on
64-bit, which I haven't really reviewed (nor do I feel fully qualified
to review it -- it's a bit of a core change, and I'd prefer for
someone involved in that code to comment on it).

For the patches I wrote, From would be nice :)

OTOH, hpa may prefer incremental patches, now that this lives in tip.

--Andy
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH v21 00/12] Add 32 bit VDSO time function support

2014-03-02 Thread H. Peter Anvin
On 03/02/2014 08:32 AM, Andy Lutomirski wrote:
 
 OTOH, hpa may prefer incremental patches, now that this lives in tip.
 

The branch currently in -tip is dead, so it is not an issue either way.

-hpa


--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/