Re: [PATCH v3] Doc/memory-barriers: Add Korean translation

From: SeongJae Park
Date: Fri Jun 24 2016 - 01:42:33 EST


Hello, Byungchul,


I guess the review is ongoing yet and maybe it requires more days. Can you let
me know your estimated time for the review if it doesn't bother you?


Thanks,
SeongJae Park

On Fri, Jun 17, 2016 at 3:24 PM, Minchan Kim <minchan@xxxxxxxxxx> wrote:
> On Wed, Jun 15, 2016 at 03:47:34PM +0900, SeongJae Park wrote:
>> 2016-06-09 5:45 GMT+09:00 SeongJae Park <sj38.park@xxxxxxxxx>:
>> > 2016-06-09 2:24 GMT+09:00 Paul E. McKenney <paulmck@xxxxxxxxxxxxxxxxxx>:
>> >> On Wed, Jun 08, 2016 at 05:58:41PM +0900, SeongJae Park wrote:
>> >>> This commit adds Korean version of memory-barriers.txt document. The
>> >>> header is refered to HOWTO Korean version.
>> >>>
>> >>> The translator, SeongJae Park, is interested in parallel programming and
>> >>> translating[1] a book[2] about the topic.
>> >>>
>> >>> The translation has started from Feb, 2016 and using a github public
>> >>> repository[3] to maintain the work. The work is following[4] updates to
>> >>> the original document as well.
>> >>>
>> >>> Because the translator has knowledge about the topic and already
>> >>> following up the upstream changes, one would sure that this translation
>> >>> will keep reasonable quality and freshness.
>> >>>
>> >>> [1] https://git.kernel.org/cgit/linux/kernel/git/paulmck/perfbook.git/commit/FAQ.txt?id=edbfcdee0460
>> >>> [2] https://www.kernel.org/pub/linux/kernel/people/paulmck/perfbook/perfbook.html
>> >>> [3] https://github.com/sjp38/linux.doc_trans_membarrier
>> >>> [4] https://github.com/sjp38/linux.doc_trans_membarrier/commit/06bd12d390f164fd253b861fc3aa8006d6d19ed9
>> >>>
>> >>> Signed-off-by: SeongJae Park <sj38.park@xxxxxxxxx>
>> >>> Acked-by: David Howells <dhowells@xxxxxxxxxx>
>> >>> Signed-off-by: Parl E. McKenney <paulmck@xxxxxxxxxxxxxxxxxx>
>> >>> Acked-by: Minchan Kim <minchan@xxxxxxxxxx>
>> >>
>> >> I cannot judge this, so I must defer to Minchan. That said, the diffstat
>> >> since your version of April 18 is as follows:
>> >>
>> >> memory-barriers.txt | 303 ++++++++++++++++++++++++++++++----------------------
>> >> 1 file changed, 175 insertions(+), 128 deletions(-)
>> >>
>> >> I cannot tell which mainline version these patches correspond to, but the
>> >> English version has this diffstat since v4.5:
>> >>
>> >> memory-barriers.txt | 293 +++++++++++++++++++++++++++++++++++++++++-----------
>> >> 1 file changed, 234 insertions(+), 59 deletions(-)
>> >>
>> >> So your level of change thus far seems plausible.
>> >
>> > I agree that Minchan deserves to judge that since he is the Korean translations
>> > maintainer.
>>
>> Minchan, may I ask your opinion about this patch?
>
> Sorry for late response.
>
> I think it's really worth. That's why I support it. However, until now,
> I didn't review it in detail. Sorry about that and stuck with urgent
> works so I asked colleague byungchul who scheduler guy, he has an
> interest so He will review this patch soon.
>
> Hope it helps you.
>
> Thanks.
>