Re: [PATCH] docs/zh_CN: Update zh_CN/kernel-hacking/hacking.rst to 5.19-rc1

From: Wu X.C.
Date: Tue Jun 07 2022 - 03:55:31 EST


On Tue, Jun 07, 2022 at 10:40:16AM +0800, teng sterling wrote:
> Wu XiangCheng <bobwxc@xxxxxxxx> 于2022年6月7日周二 01:20写道:
> >
> > * update to commit f35cf1a59e9a ("Documentation: kernel-hacking: minor
> > edits for style")
> >
> > * fix a homophone typo reported by Li Feng
> >
> > Reported-by: Li Feng <felixlee868@xxxxxxxxxx>
> > Signed-off-by: Wu XiangCheng <bobwxc@xxxxxxxx>
> Reviewed-by: Yanteng Si <siyanteng@xxxxxxxxxxx>

Thank you!

>
> BTW,
> b4 doesn't seem to work for this patch, but I manually downloaded the
> email for a build test and it was fine. :)

Unable to reproduce. Check the version of b4?

❯ b4 --version
0.8.0
❯ b4 am Yp41+eTjoPRa4hrl@xxxxxxxxxxx
Looking up https://lore.kernel.org/r/Yp41%2BeTjoPRa4hrl%40bobwxc.mipc
Grabbing thread from lore.kernel.org/all/Yp41%2BeTjoPRa4hrl%40bobwxc.mipc/t.mbox.gz
Analyzing 2 messages in the thread
Checking attestation on all messages, may take a moment...
---
✓ [PATCH] docs/zh_CN: Update zh_CN/kernel-hacking/hacking.rst to 5.19-rc1
---
✓ Signed: openpgp/bobwxc@xxxxxxxx
✓ Signed: DKIM/email.cn
---
Total patches: 1
---
NOTE: some trailers ignored due to from/email mismatches:
! Trailer: Reviewed-by: Yanteng Si <siyanteng@xxxxxxxxxxx>
Msg From: teng sterling <sterlingteng@xxxxxxxxx>
NOTE: Rerun with -S to apply them anyway
---
Link: https://lore.kernel.org/r/Yp41+eTjoPRa4hrl@xxxxxxxxxxx
Base: not specified
git am ./20220607_bobwxc_docs_zh_cn_update_zh_cn_kernel_hacking_hacking_rst_to_5_19_rc1.mbx

--
Thanks,
Wu

>
> log:
> $../b4/b4.sh am Yp41+eTjoPRa4hrl@xxxxxxxxxxx
> Looking up https://lore.kernel.org/r/Yp41%2BeTjoPRa4hrl%40bobwxc.mipc
> Grabbing thread from
> lore.kernel.org/all/Yp41%2BeTjoPRa4hrl%40bobwxc.mipc/t.mbox.gz
> Analyzing 1 messages in the thread
> Checking attestation on all messages, may take a moment...
> Traceback (most recent call last):
> File "/home/siyanteng/b4/b4/command.py", line 263, in <module>
> ......
>
> Thanks,
> Yanteng
> > ---
> > .../zh_CN/kernel-hacking/hacking.rst | 22 +++++++++----------
> > 1 file changed, 11 insertions(+), 11 deletions(-)
> >
> > diff --git a/Documentation/translations/zh_CN/kernel-hacking/hacking.rst b/Documentation/translations/zh_CN/kernel-hacking/hacking.rst
> > index f2bc154c5bcc..bda79646bb1e 100644
> > --- a/Documentation/translations/zh_CN/kernel-hacking/hacking.rst
> > +++ b/Documentation/translations/zh_CN/kernel-hacking/hacking.rst
> > @@ -81,7 +81,7 @@
> > 过硬件中断)的“软件中断”将运行( ``kernel/softirq.c`` )。
> >
> > 此处完成了许多真正的中断处理工作。在向SMP过渡的早期,只有“bottom halves下半
> > -部”(BHs)机制,无法利用多个CPU的优势。在从那些一团糟的就电脑切换过来后不久,
> > +部”(BHs)机制,无法利用多个CPU的优势。在从那些一团糟的旧电脑切换过来后不久,
> > 我们放弃了这个限制,转而使用“软中断”。
> >
> > ``include/linux/interrupt.h`` 列出了不同的软中断。定时器软中断是一个非常重要
> > @@ -95,8 +95,7 @@
> >
> > .. warning::
> >
> > - “tasklet”这个名字是误导性的:它们与“任务”无关,可能更多与当时
> > - 阿列克谢·库兹涅佐夫享用的糟糕伏特加有关。
> > + “tasklet”这个名字是误导性的:它们与“任务”无关。
> >
> > 你可以使用 :c:func:`in_softirq()` 宏( ``include/linux/preempt.h`` )来确认
> > 是否处于软中断(或子任务)中。
> > @@ -247,7 +246,7 @@ Provide mechanism not policy”。
> > 与 :c:func:`put_user()` 和 :c:func:`get_user()` 不同,它们返回未复制的
> > 数据量(即0仍然意味着成功)。
> >
> > -【是的,这个愚蠢的接口真心让我尴尬。火爆的口水仗大概每年都会发生。
> > +【是的,这个讨厌的接口真心让我尴尬。火爆的口水仗大概每年都会发生。
> > —— Rusty Russell】
> >
> > 这些函数可以隐式睡眠。它不应该在用户上下文之外调用(没有意义)、调用时禁用中断
> > @@ -538,9 +537,9 @@ Documentation/core-api/symbol-namespaces.rst 。
> >
> > Linus和其他开发人员有时会更改开发内核中的函数或结构体名称;这样做不仅是为了
> > 让每个人都保持警惕,还反映了一个重大的更改(例如,不能再在打开中断的情况下
> > -调用,或者执行额外的检查,或者不执行以前捕获的检查)。通常这会附带一个linux
> > -内核邮件列表中相当全面的注释;请搜索存档以查看。简单地对文件进行全局替换通常
> > -会让事情变得 **更糟** 。
> > +调用,或者执行额外的检查,或者不执行以前捕获的检查)。通常这会附带发送一个
> > +相当全面的注释到相应的内核邮件列表中;请搜索存档以查看。简单地对文件进行全局
> > +替换通常只会让事情变得 **更糟** 。
> >
> > 初始化结构体成员
> > ------------------
> > @@ -610,7 +609,7 @@ C++
> >
> > 为了让你的东西更正式、补丁更整洁,还有一些工作要做:
> >
> > -- 搞清楚你在谁的地界儿上干活。查看源文件的顶部、 ``MAINTAINERS`` 文件以及
> > +- 搞清楚你修改的代码属于谁。查看源文件的根目录、 ``MAINTAINERS`` 文件以及
> > ``CREDITS`` 文件的最后一部分。你应该和此人协调,确保你没有重新发明轮子,
> > 或者尝试一些已经被拒绝的东西。
> >
> > @@ -629,9 +628,10 @@ C++
> > “obj-$(CONFIG_xxx) += xxx.o”。语法记录在
> > Documentation/kbuild/makefiles.rst 。
> >
> > -- 如果你做了一些有意义的事情,那可以把自己放进 ``CREDITS`` ,通常不止一个
> > - 文件(无论如何你的名字都应该在源文件的顶部)。维护人员意味着您希望在对
> > - 子系统进行更改时得到询问,并了解缺陷;这意味着对某部分代码做出更多承诺。
> > +- 如果你认为自己做了一些有意义的事情,可以把自己放进 ``CREDITS`` ,通常不
> > + 止一个文件(无论如何你的名字都应该在源文件的顶部)。 ``MAINTAINERS``
> > + 意味着您希望在对子系统进行更改时得到询问,并了解缺陷;这意味着对某部分
> > + 代码做出更多承诺。
> >
> > - 最后,别忘记去阅读 Documentation/process/submitting-patches.rst ,
> > 也许还有 Documentation/process/submitting-drivers.rst 。
> > --
> > 2.30.2
> >