Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1240140 > unrolled thread
| Started by | Michael Ellerman <mpe@ellerman.id.au> |
|---|---|
| First post | 2015-10-06 05:40 +0200 |
| Last post | 2015-10-11 16:30 +0200 |
| Articles | 10 — 3 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [PATCH 4/5 v2] pseries/iommu: implement DDW-aware dma_get_page_shift Michael Ellerman <mpe@ellerman.id.au> - 2015-10-06 05:40 +0200
Re: [kbuild-all] [PATCH 4/5 v2] pseries/iommu: implement DDW-aware dma_get_page_shift Fengguang Wu <lkp@intel.com> - 2015-10-07 16:00 +0200
Re: [kbuild-all] [PATCH 4/5 v2] pseries/iommu: implement DDW-aware dma_get_page_shift Michael Ellerman <mpe@ellerman.id.au> - 2015-10-08 02:20 +0200
Re: [kbuild-all] [PATCH 4/5 v2] pseries/iommu: implement DDW-aware dma_get_page_shift Fengguang Wu <lkp@intel.com> - 2015-10-08 03:10 +0200
Re: [kbuild-all] [PATCH 4/5 v2] pseries/iommu: implement DDW-aware dma_get_page_shift Fengguang Wu <lkp@intel.com> - 2015-10-08 03:20 +0200
Re: [kbuild-all] [PATCH 4/5 v2] pseries/iommu: implement DDW-aware dma_get_page_shift Michael Ellerman <mpe@ellerman.id.au> - 2015-10-08 06:10 +0200
Re: [kbuild-all] [PATCH 4/5 v2] pseries/iommu: implement DDW-aware dma_get_page_shift Fengguang Wu <lkp@intel.com> - 2015-10-11 16:30 +0200
Re: [kbuild-all] [PATCH 4/5 v2] pseries/iommu: implement DDW-aware dma_get_page_shift Michael Ellerman <mpe@ellerman.id.au> - 2015-10-12 05:00 +0200
Re: [kbuild-all] [PATCH 4/5 v2] pseries/iommu: implement DDW-aware dma_get_page_shift Christoph Hellwig <hch@infradead.org> - 2015-10-08 09:50 +0200
testing email patches Fengguang Wu <lkp@intel.com> - 2015-10-11 16:30 +0200
| From | Michael Ellerman <mpe@ellerman.id.au> |
|---|---|
| Date | 2015-10-06 05:40 +0200 |
| Subject | Re: [PATCH 4/5 v2] pseries/iommu: implement DDW-aware dma_get_page_shift |
| Message-ID | <qgrgC-2u2-3@gated-at.bofh.it> |
On Sat, 2015-10-03 at 04:33 +0800, kbuild test robot wrote: > Hi Nishanth, > > [auto build test results on v4.3-rc3 -- if it's inappropriate base, please ignore] > > config: powerpc-defconfig (attached as .config) > reproduce: > wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross > chmod +x ~/bin/make.cross > # save the attached .config to linux build tree > make.cross ARCH=powerpc > > All error/warnings (new ones prefixed by >>): > > arch/powerpc/platforms/pseries/iommu.c: In function 'iommu_init_early_pSeries': > >> arch/powerpc/platforms/pseries/iommu.c:1433:9: error: 'struct machdep_calls' has no member named 'dma_get_page_shift' > ppc_md.dma_get_page_shift = dma_get_page_shift_pSeriesLP; It was added in patch 3/5, so I think this error is bogus. Unless there's a typo I'm missing? cheers -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [next] | [standalone]
| From | Fengguang Wu <lkp@intel.com> |
|---|---|
| Date | 2015-10-07 16:00 +0200 |
| Subject | Re: [kbuild-all] [PATCH 4/5 v2] pseries/iommu: implement DDW-aware dma_get_page_shift |
| Message-ID | <qgXqc-6Da-49@gated-at.bofh.it> |
| In reply to | #1240140 |
On Tue, Oct 06, 2015 at 02:39:06PM +1100, Michael Ellerman wrote: > On Sat, 2015-10-03 at 04:33 +0800, kbuild test robot wrote: > > Hi Nishanth, > > > > [auto build test results on v4.3-rc3 -- if it's inappropriate base, please ignore] > > > > config: powerpc-defconfig (attached as .config) > > reproduce: > > wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross > > chmod +x ~/bin/make.cross > > # save the attached .config to linux build tree > > make.cross ARCH=powerpc > > > > All error/warnings (new ones prefixed by >>): > > > > arch/powerpc/platforms/pseries/iommu.c: In function 'iommu_init_early_pSeries': > > >> arch/powerpc/platforms/pseries/iommu.c:1433:9: error: 'struct machdep_calls' has no member named 'dma_get_page_shift' > > ppc_md.dma_get_page_shift = dma_get_page_shift_pSeriesLP; > > It was added in patch 3/5, so I think this error is bogus. Unless there's a > typo I'm missing? Yes sorry, the patchset was not detected correctly in your case, ending up the patches being tested as individual ones. I'll fixup the robot. Thanks, Fengguang -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Michael Ellerman <mpe@ellerman.id.au> |
|---|---|
| Date | 2015-10-08 02:20 +0200 |
| Subject | Re: [kbuild-all] [PATCH 4/5 v2] pseries/iommu: implement DDW-aware dma_get_page_shift |
| Message-ID | <qh76a-3Sm-25@gated-at.bofh.it> |
| In reply to | #1241532 |
On Wed, 2015-10-07 at 21:56 +0800, Fengguang Wu wrote: > On Tue, Oct 06, 2015 at 02:39:06PM +1100, Michael Ellerman wrote: > > On Sat, 2015-10-03 at 04:33 +0800, kbuild test robot wrote: > > > Hi Nishanth, > > > > > > [auto build test results on v4.3-rc3 -- if it's inappropriate base, please ignore] > > > > > > config: powerpc-defconfig (attached as .config) > > > reproduce: > > > wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross > > > chmod +x ~/bin/make.cross > > > # save the attached .config to linux build tree > > > make.cross ARCH=powerpc > > > > > > All error/warnings (new ones prefixed by >>): > > > > > > arch/powerpc/platforms/pseries/iommu.c: In function 'iommu_init_early_pSeries': > > > >> arch/powerpc/platforms/pseries/iommu.c:1433:9: error: 'struct machdep_calls' has no member named 'dma_get_page_shift' > > > ppc_md.dma_get_page_shift = dma_get_page_shift_pSeriesLP; > > > > It was added in patch 3/5, so I think this error is bogus. Unless there's a > > typo I'm missing? > > Yes sorry, the patchset was not detected correctly in your case, > ending up the patches being tested as individual ones. > > I'll fixup the robot. Thanks. How did the robot decide to build this series in the first place? Does it build everything sent to one of the lists on CC? cheers -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Fengguang Wu <lkp@intel.com> |
|---|---|
| Date | 2015-10-08 03:10 +0200 |
| Subject | Re: [kbuild-all] [PATCH 4/5 v2] pseries/iommu: implement DDW-aware dma_get_page_shift |
| Message-ID | <qh7Sy-51D-5@gated-at.bofh.it> |
| In reply to | #1241884 |
On Thu, Oct 08, 2015 at 11:11:59AM +1100, Michael Ellerman wrote: > On Wed, 2015-10-07 at 21:56 +0800, Fengguang Wu wrote: > > On Tue, Oct 06, 2015 at 02:39:06PM +1100, Michael Ellerman wrote: > > > On Sat, 2015-10-03 at 04:33 +0800, kbuild test robot wrote: > > > > Hi Nishanth, > > > > > > > > [auto build test results on v4.3-rc3 -- if it's inappropriate base, please ignore] > > > > > > > > config: powerpc-defconfig (attached as .config) > > > > reproduce: > > > > wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross > > > > chmod +x ~/bin/make.cross > > > > # save the attached .config to linux build tree > > > > make.cross ARCH=powerpc > > > > > > > > All error/warnings (new ones prefixed by >>): > > > > > > > > arch/powerpc/platforms/pseries/iommu.c: In function 'iommu_init_early_pSeries': > > > > >> arch/powerpc/platforms/pseries/iommu.c:1433:9: error: 'struct machdep_calls' has no member named 'dma_get_page_shift' > > > > ppc_md.dma_get_page_shift = dma_get_page_shift_pSeriesLP; > > > > > > It was added in patch 3/5, so I think this error is bogus. Unless there's a > > > typo I'm missing? > > > > Yes sorry, the patchset was not detected correctly in your case, > > ending up the patches being tested as individual ones. > > > > I'll fixup the robot. > > Thanks. > > How did the robot decide to build this series in the first place? Does it build > everything sent to one of the lists on CC? Yes, currently the following mailing lists are subscribed. Patches sent to them will be tested if they can be git-am to RC or linux-next kernels: kvm linux-acpi linux-bluetooth linux-btrfs linux-embedded linux-ext4 linux-fsdevel linux-media linux-mmc linux-nfs linux-omap linux-pci linux-pm linux-raid linux-rdma linux-scsi linux-usb linux-wireless netdev netfilter-devel linux-mm driverdev-devel intel-gfx Thanks, Fengguang -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Fengguang Wu <lkp@intel.com> |
|---|---|
| Date | 2015-10-08 03:20 +0200 |
| Subject | Re: [kbuild-all] [PATCH 4/5 v2] pseries/iommu: implement DDW-aware dma_get_page_shift |
| Message-ID | <qh82e-5cJ-3@gated-at.bofh.it> |
| In reply to | #1241916 |
On Thu, Oct 08, 2015 at 09:06:03AM +0800, Fengguang Wu wrote: > On Thu, Oct 08, 2015 at 11:11:59AM +1100, Michael Ellerman wrote: > > On Wed, 2015-10-07 at 21:56 +0800, Fengguang Wu wrote: > > > On Tue, Oct 06, 2015 at 02:39:06PM +1100, Michael Ellerman wrote: > > > > On Sat, 2015-10-03 at 04:33 +0800, kbuild test robot wrote: > > > > > Hi Nishanth, > > > > > > > > > > [auto build test results on v4.3-rc3 -- if it's inappropriate base, please ignore] > > > > > > > > > > config: powerpc-defconfig (attached as .config) > > > > > reproduce: > > > > > wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross > > > > > chmod +x ~/bin/make.cross > > > > > # save the attached .config to linux build tree > > > > > make.cross ARCH=powerpc > > > > > > > > > > All error/warnings (new ones prefixed by >>): > > > > > > > > > > arch/powerpc/platforms/pseries/iommu.c: In function 'iommu_init_early_pSeries': > > > > > >> arch/powerpc/platforms/pseries/iommu.c:1433:9: error: 'struct machdep_calls' has no member named 'dma_get_page_shift' > > > > > ppc_md.dma_get_page_shift = dma_get_page_shift_pSeriesLP; > > > > > > > > It was added in patch 3/5, so I think this error is bogus. Unless there's a > > > > typo I'm missing? > > > > > > Yes sorry, the patchset was not detected correctly in your case, > > > ending up the patches being tested as individual ones. > > > > > > I'll fixup the robot. > > > > Thanks. > > > > How did the robot decide to build this series in the first place? Does it build > > everything sent to one of the lists on CC? > > Yes, currently the following mailing lists are subscribed. Patches > sent to them will be tested if they can be git-am to RC or linux-next > kernels: > > kvm > linux-acpi > linux-bluetooth > linux-btrfs > linux-embedded > linux-ext4 > linux-fsdevel > linux-media > linux-mmc > linux-nfs > linux-omap > linux-pci > linux-pm > linux-raid > linux-rdma > linux-scsi > linux-usb > linux-wireless > netdev > netfilter-devel > linux-mm > driverdev-devel > intel-gfx And of course linux-kernel. More lists could be added in future. Thanks, Fengguang -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Michael Ellerman <mpe@ellerman.id.au> |
|---|---|
| Date | 2015-10-08 06:10 +0200 |
| Subject | Re: [kbuild-all] [PATCH 4/5 v2] pseries/iommu: implement DDW-aware dma_get_page_shift |
| Message-ID | <qhaGJ-F6-3@gated-at.bofh.it> |
| In reply to | #1241921 |
On Thu, 2015-10-08 at 09:16 +0800, Fengguang Wu wrote: > On Thu, Oct 08, 2015 at 09:06:03AM +0800, Fengguang Wu wrote: > > On Thu, Oct 08, 2015 at 11:11:59AM +1100, Michael Ellerman wrote: > > > On Wed, 2015-10-07 at 21:56 +0800, Fengguang Wu wrote: > > > > On Tue, Oct 06, 2015 at 02:39:06PM +1100, Michael Ellerman wrote: > > > > > On Sat, 2015-10-03 at 04:33 +0800, kbuild test robot wrote: > > > > > > Hi Nishanth, > > > > > > > > > > > > [auto build test results on v4.3-rc3 -- if it's inappropriate base, please ignore] > > > > > > > > > > > > config: powerpc-defconfig (attached as .config) > > > > > > reproduce: > > > > > > wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross > > > > > > chmod +x ~/bin/make.cross > > > > > > # save the attached .config to linux build tree > > > > > > make.cross ARCH=powerpc > > > > > > > > > > > > All error/warnings (new ones prefixed by >>): > > > > > > > > > > > > arch/powerpc/platforms/pseries/iommu.c: In function 'iommu_init_early_pSeries': > > > > > > >> arch/powerpc/platforms/pseries/iommu.c:1433:9: error: 'struct machdep_calls' has no member named 'dma_get_page_shift' > > > > > > ppc_md.dma_get_page_shift = dma_get_page_shift_pSeriesLP; > > > > > > > > > > It was added in patch 3/5, so I think this error is bogus. Unless there's a > > > > > typo I'm missing? > > > > > > > > Yes sorry, the patchset was not detected correctly in your case, > > > > ending up the patches being tested as individual ones. > > > > > > > > I'll fixup the robot. > > > > > > Thanks. > > > > > > How did the robot decide to build this series in the first place? Does it build > > > everything sent to one of the lists on CC? > > > > Yes, currently the following mailing lists are subscribed. Patches > > sent to them will be tested if they can be git-am to RC or linux-next > > kernels: > > > > kvm > > linux-acpi > > linux-bluetooth > > linux-btrfs > > linux-embedded > > linux-ext4 > > linux-fsdevel > > linux-media > > linux-mmc > > linux-nfs > > linux-omap > > linux-pci > > linux-pm > > linux-raid > > linux-rdma > > linux-scsi > > linux-usb > > linux-wireless > > netdev > > netfilter-devel > > linux-mm > > driverdev-devel > > intel-gfx > > And of course linux-kernel. More lists could be added in future. So do you mind adding linuxppc-dev@lists.ozlabs.org ? :) It's pretty low traffic compared to lkml. cheers -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Fengguang Wu <lkp@intel.com> |
|---|---|
| Date | 2015-10-11 16:30 +0200 |
| Subject | Re: [kbuild-all] [PATCH 4/5 v2] pseries/iommu: implement DDW-aware dma_get_page_shift |
| Message-ID | <qipNo-2B9-13@gated-at.bofh.it> |
| In reply to | #1241956 |
Hi Michael, > > > > How did the robot decide to build this series in the first place? Does it build > > > > everything sent to one of the lists on CC? > > > > > > Yes, currently the following mailing lists are subscribed. Patches > > > sent to them will be tested if they can be git-am to RC or linux-next > > > kernels: > > > > > > kvm > > > linux-acpi > > > linux-bluetooth > > > linux-btrfs > > > linux-embedded > > > linux-ext4 > > > linux-fsdevel > > > linux-media > > > linux-mmc > > > linux-nfs > > > linux-omap > > > linux-pci > > > linux-pm > > > linux-raid > > > linux-rdma > > > linux-scsi > > > linux-usb > > > linux-wireless > > > netdev > > > netfilter-devel > > > linux-mm > > > driverdev-devel > > > intel-gfx > > > > And of course linux-kernel. More lists could be added in future. > > So do you mind adding linuxppc-dev@lists.ozlabs.org ? :) > > It's pretty low traffic compared to lkml. Sure, I'm subscribing to it. :) Thanks, Fengguang -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Michael Ellerman <mpe@ellerman.id.au> |
|---|---|
| Date | 2015-10-12 05:00 +0200 |
| Subject | Re: [kbuild-all] [PATCH 4/5 v2] pseries/iommu: implement DDW-aware dma_get_page_shift |
| Message-ID | <qiBvc-2F4-5@gated-at.bofh.it> |
| In reply to | #1244141 |
On Sun, 2015-10-11 at 22:22 +0800, Fengguang Wu wrote: > > > > So do you mind adding linuxppc-dev@lists.ozlabs.org ? :) > > > > It's pretty low traffic compared to lkml. > > Sure, I'm subscribing to it. :) Thanks. cheers -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Christoph Hellwig <hch@infradead.org> |
|---|---|
| Date | 2015-10-08 09:50 +0200 |
| Subject | Re: [kbuild-all] [PATCH 4/5 v2] pseries/iommu: implement DDW-aware dma_get_page_shift |
| Message-ID | <qhe7E-5rB-3@gated-at.bofh.it> |
| In reply to | #1241921 |
Hi Fengguang, I think this proactive testing does a little more harm than good in it's current form. While offering testing for patches that aren't in git trees and or by people that don't even have a git tree that the build bots known about does seem useful, blindly doing it for every patch against something that most likely isn't the right base seems counter intertuitive. We'll probaby need some annotation in the O/n mail that asks for a test and sets a base tree to actually make it useful. With those few tweaks it should be really useful! Maybe we should have a discussion about this at kernel summit? On Thu, Oct 08, 2015 at 09:16:09AM +0800, Fengguang Wu wrote: > And of course linux-kernel. More lists could be added in future. > > Thanks, > Fengguang ---end quoted text--- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Fengguang Wu <lkp@intel.com> |
|---|---|
| Date | 2015-10-11 16:30 +0200 |
| Subject | testing email patches |
| Message-ID | <qipNo-2B9-15@gated-at.bofh.it> |
| In reply to | #1242022 |
Hi Christoph,
On Thu, Oct 08, 2015 at 12:46:16AM -0700, Christoph Hellwig wrote:
> Hi Fengguang,
>
> I think this proactive testing does a little more harm than good in
> it's current form. While offering testing for patches that aren't in
> git trees and or by people that don't even have a git tree that the
> build bots known about does seem useful, blindly doing it for every
> patch against something that most likely isn't the right base seems
> counter intertuitive. We'll probaby need some annotation in the O/n
> mail that asks for a test and sets a base tree to actually make it
> useful. With those few tweaks it should be really useful!
>
> Maybe we should have a discussion about this at kernel summit?
Yes that may be a good topic for gathering feedbacks and ideas for
improving this useful but messy testing feature.
The best option could be to define a way to annotate the base tree's
branch/commit of a patchset and possibly automate the annotation in
the git/quilt email clients. Currently git does generate
index c933675..a2aa0cd
for each diff files, however I find it far from enough -- there are
~50000 files in the kernel tree, knowing hash numbers for the typical
1-100 files a patchset may touch is pretty helpless in narrowing down
the search space of possible base trees.
It may also be useful to specify whether or not a patchset needs such
kind of testing. That'd be an easier task -- the robot can detect some
magic words in the email and take action accordingly. And it can auto
skip testing in some cases -- for example, Peter Zijlstra offered a
good point to skip testing patches without "Signed-off-by:" in a "Re:"
email. The patches already tested as git tree commits in the 500+ git
trees the 0day robot monitors will be auto skipped, too.
Also we could test against both mainline and linux-next -- an
excellent suggestion from Dan Carpenter, which is just implemented,
thanks!
Not knowing the exact base tree leads to inherent messiness, and the
possible solutions/workarounds are
1) annotations to tell the base tree (best option)
2) annotations & heuristics to skip tests
3) guess the suitable tree the maintainer would apply patches to
4) test on mainline & linux-next to filter out unstable build errors
(3) is kind of dirty work and enlightenments from subsystem
maintainers are highly welcome -- typically the suggestions can be
offered when you find a bug report to be unsuitable.
As for now I'm trying to teach 0day robot to apply patches to the
matching subsystem's "for-next" branch based on information in the
MAINTAINERS file.
Thanks,
Fengguang
> On Thu, Oct 08, 2015 at 09:16:09AM +0800, Fengguang Wu wrote:
> > And of course linux-kernel. More lists could be added in future.
> >
> > Thanks,
> > Fengguang
> ---end quoted text---
> _______________________________________________
> kbuild-all mailing list
> kbuild-all@lists.01.org
> https://lists.01.org/mailman/listinfo/kbuild-all
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web