| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <CAKaHn9KxRrBsn4b9fSO1eDzM3XdV2GzfwVX+cGw9uS_eKg75dw@mail.gmail.com>
Date: Mon, 5 Jul 2021 16:35:19 +0800
From: Art Nikpal <email2tema@...il.com>
To: 陈华才 <chenhuacai@...ngson.cn>
Cc: Bjorn Helgaas <helgaas@...nel.org>,
Neil Armstrong <narmstrong@...libre.com>,
Yue Wang <yue.wang@...ogic.com>,
Kevin Hilman <khilman@...libre.com>,
Lorenzo Pieralisi <lorenzo.pieralisi@....com>,
Rob Herring <robh@...nel.org>,
Krzysztof Wilczynski <kw@...ux.com>,
Jerome Brunet <jbrunet@...libre.com>,
Christian Hewitt <christianshewitt@...il.com>,
Martin Blumenstingl <martin.blumenstingl@...glemail.com>,
PCI <linux-pci@...r.kernel.org>,
linux-arm-kernel <linux-arm-kernel@...ts.infradead.org>,
"open list:ARM/Amlogic Meson..." <linux-amlogic@...ts.infradead.org>,
linux-kernel@...r.kernel.org, Artem Lapkin <art@...das.com>,
Nick Xie <nick@...das.com>, Gouwa Wang <gouwa@...das.com>,
chenhuacai@...il.com
Subject: Re: Re: [PATCH 0/4] PCI: replace dublicated MRRS limit quirks
> Does that means keystone and Loongson has the same MRRS problem? And what should I do now?
Look like yes ! and amlogic has the same problem.
I think somebody need to rewrite it all to one common quirk for this problem.
If no one has any objection, I can try to remake it again.
On Fri, Jul 2, 2021 at 9:15 AM 陈华才 <chenhuacai@...ngson.cn> wrote:
>
> Hi, Bjorn,
>
> > -----原始邮件-----
> > 发件人: "Bjorn Helgaas" <helgaas@...nel.org>
> > 发送时间: 2021-07-01 23:46:34 (星期四)
> > 收件人: "Artem Lapkin" <email2tema@...il.com>
> > 抄送: narmstrong@...libre.com, yue.wang@...ogic.com, khilman@...libre.com, lorenzo.pieralisi@....com, robh@...nel.org, kw@...ux.com, jbrunet@...libre.com, christianshewitt@...il.com, martin.blumenstingl@...glemail.com, linux-pci@...r.kernel.org, linux-arm-kernel@...ts.infradead.org, linux-amlogic@...ts.infradead.org, linux-kernel@...r.kernel.org, art@...das.com, nick@...das.com, gouwa@...das.com, "Huacai Chen" <chenhuacai@...ngson.cn>
> > 主题: Re: [PATCH 0/4] PCI: replace dublicated MRRS limit quirks
> >
> > [+cc Huacai]
> >
> > On Sat, Jun 19, 2021 at 02:39:48PM +0800, Artem Lapkin wrote:
> > > Replace dublicated MRRS limit quirks by mrrs_limit_quirk from core
> > > * drivers/pci/controller/dwc/pci-keystone.c
> > > * drivers/pci/controller/pci-loongson.c
> >
> > s/dublicated/duplicated/ (several occurrences)
> >
> > Capitalize subject lines.
> >
> > Use "git log --online" to learn conventions and follow them.
> >
> > Add "()" after function names.
> >
> > Capitalize acronyms appropriately (NVMe, MRRS, PCI, etc).
> >
> > End sentences with periods.
> >
> > A "move" patch must include both the removal and the addition and make
> > no changes to the code itself.
> >
> > Amlogic appears without explanation in 2/4. Must be separate patch to
> > address only that specific issue. Should reference published erratum
> > if possible. "Solves some issue" is not a compelling justification.
> >
> > The tree must be consistent and functionally the same or improved
> > after every patch.
> >
> > Add to pci_ids.h only if symbol used more than one place.
> >
> > See
> > https://lore.kernel.org/r/20210701074458.1809532-3-chenhuacai@loongson.cn,
> > which looks similar. Combine efforts if possible and cc Huacai so
> > you're both aware of overlapping work.
> >
> > More hints in case they're useful:
> > https://lore.kernel.org/linux-pci/20171026223701.GA25649@bhelgaas-glaptop.roam.corp.google.com/
> >
> > > Both ks_pcie_quirk loongson_mrrs_quirk was rewritten without any
> > > functionality changes by one mrrs_limit_quirk
> Does that means keystone and Loongson has the same MRRS problem? And what should I do now?
>
> Huacai
> > >
> > > Added DesignWare PCI controller which need same quirk for
> > > * drivers/pci/controller/dwc/pci-meson.c (PCI_DEVICE_ID_SYNOPSYS_HAPSUSB3)
> > >
> > > This quirk can solve some issue for Khadas VIM3/VIM3L(Amlogic)
> > > with HDMI scrambled picture and nvme devices at intensive writing...
> > >
> > > come from:
> > > * https://lore.kernel.org/linux-pci/20210618063821.1383357-1-art@khadas.com/
> > >
> > > Artem Lapkin (4):
> > > PCI: move Keystone and Loongson device IDs to pci_ids
> > > PCI: core: quirks: add mrrs_limit_quirk
> > > PCI: keystone move mrrs quirk to core
> > > PCI: loongson move mrrs quirk to core
> > >
> > > --
> > > 2.25.1
> > >
>
>
> </chenhuacai@...ngson.cn></email2tema@...il.com></helgaas@...nel.org>
Powered by blists - more mailing lists