* [PATCH] powerpc/powernv: Print the M64 range information @ 2014-12-10 14:29 Wei Yang 2014-12-10 22:40 ` Gavin Shan 0 siblings, 1 reply; 6+ messages in thread From: Wei Yang @ 2014-12-10 14:29 UTC (permalink / raw) To: gwshan; +Cc: Wei Yang, linuxppc-dev The M64 range information is missed, which would be helpful in debug. This patch print the M64 range information in the same format as M32. Signed-off-by: Wei Yang <weiyang@linux.vnet.ibm.com> --- arch/powerpc/platforms/powernv/pci-ioda.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/arch/powerpc/platforms/powernv/pci-ioda.c b/arch/powerpc/platforms/powernv/pci-ioda.c index 21c8b4d..34f1a27 100644 --- a/arch/powerpc/platforms/powernv/pci-ioda.c +++ b/arch/powerpc/platforms/powernv/pci-ioda.c @@ -369,6 +369,9 @@ static void __init pnv_ioda_parse_m64_window(struct pnv_phb *phb) phb->ioda.m64_segsize = phb->ioda.m64_size / phb->ioda.total_pe; phb->ioda.m64_base = pci_addr; + printk(KERN_INFO " MEM64 0x%016llx..0x%016llx -> 0x%016llx\n", + res->start, res->end, pci_addr); + /* Use last M64 BAR to cover M64 window */ phb->ioda.m64_bar_idx = 15; phb->init_m64 = pnv_ioda2_init_m64; -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] powerpc/powernv: Print the M64 range information 2014-12-10 14:29 [PATCH] powerpc/powernv: Print the M64 range information Wei Yang @ 2014-12-10 22:40 ` Gavin Shan 2014-12-11 7:29 ` Wei Yang 0 siblings, 1 reply; 6+ messages in thread From: Gavin Shan @ 2014-12-10 22:40 UTC (permalink / raw) To: Wei Yang; +Cc: linuxppc-dev, gwshan On Wed, Dec 10, 2014 at 10:29:37PM +0800, Wei Yang wrote: >The M64 range information is missed, which would be helpful in debug. > >This patch print the M64 range information in the same format as M32. > >Signed-off-by: Wei Yang <weiyang@linux.vnet.ibm.com> >--- > arch/powerpc/platforms/powernv/pci-ioda.c | 3 +++ > 1 file changed, 3 insertions(+) > >diff --git a/arch/powerpc/platforms/powernv/pci-ioda.c b/arch/powerpc/platforms/powernv/pci-ioda.c >index 21c8b4d..34f1a27 100644 >--- a/arch/powerpc/platforms/powernv/pci-ioda.c >+++ b/arch/powerpc/platforms/powernv/pci-ioda.c >@@ -369,6 +369,9 @@ static void __init pnv_ioda_parse_m64_window(struct pnv_phb *phb) > phb->ioda.m64_segsize = phb->ioda.m64_size / phb->ioda.total_pe; > phb->ioda.m64_base = pci_addr; > >+ printk(KERN_INFO " MEM64 0x%016llx..0x%016llx -> 0x%016llx\n", >+ res->start, res->end, pci_addr); >+ Please output segment size if you think it's helpful as well. Thanks, Gavin > /* Use last M64 BAR to cover M64 window */ > phb->ioda.m64_bar_idx = 15; > phb->init_m64 = pnv_ioda2_init_m64; >-- >1.7.9.5 > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] powerpc/powernv: Print the M64 range information 2014-12-10 22:40 ` Gavin Shan @ 2014-12-11 7:29 ` Wei Yang 2014-12-11 21:58 ` Gavin Shan 0 siblings, 1 reply; 6+ messages in thread From: Wei Yang @ 2014-12-11 7:29 UTC (permalink / raw) To: Gavin Shan; +Cc: Wei Yang, linuxppc-dev On Thu, Dec 11, 2014 at 09:40:30AM +1100, Gavin Shan wrote: >On Wed, Dec 10, 2014 at 10:29:37PM +0800, Wei Yang wrote: >>The M64 range information is missed, which would be helpful in debug. >> >>This patch print the M64 range information in the same format as M32. >> >>Signed-off-by: Wei Yang <weiyang@linux.vnet.ibm.com> >>--- >> arch/powerpc/platforms/powernv/pci-ioda.c | 3 +++ >> 1 file changed, 3 insertions(+) >> >>diff --git a/arch/powerpc/platforms/powernv/pci-ioda.c b/arch/powerpc/platforms/powernv/pci-ioda.c >>index 21c8b4d..34f1a27 100644 >>--- a/arch/powerpc/platforms/powernv/pci-ioda.c >>+++ b/arch/powerpc/platforms/powernv/pci-ioda.c >>@@ -369,6 +369,9 @@ static void __init pnv_ioda_parse_m64_window(struct pnv_phb *phb) >> phb->ioda.m64_segsize = phb->ioda.m64_size / phb->ioda.total_pe; >> phb->ioda.m64_base = pci_addr; >> >>+ printk(KERN_INFO " MEM64 0x%016llx..0x%016llx -> 0x%016llx\n", >>+ res->start, res->end, pci_addr); >>+ > >Please output segment size if you think it's helpful as well. The M64 segment size is printed in pnv_pci_init_ioda_phb() line 2620. > >Thanks, >Gavin > >> /* Use last M64 BAR to cover M64 window */ >> phb->ioda.m64_bar_idx = 15; >> phb->init_m64 = pnv_ioda2_init_m64; >>-- >>1.7.9.5 >> -- Richard Yang Help you, Help me ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] powerpc/powernv: Print the M64 range information 2014-12-11 7:29 ` Wei Yang @ 2014-12-11 21:58 ` Gavin Shan 2014-12-12 4:28 ` Wei Yang 2014-12-12 4:39 ` [PATCH] powerpc/powernv: Print the M64 range information in bootup log Wei Yang 0 siblings, 2 replies; 6+ messages in thread From: Gavin Shan @ 2014-12-11 21:58 UTC (permalink / raw) To: Wei Yang; +Cc: linuxppc-dev, Gavin Shan On Thu, Dec 11, 2014 at 03:29:01PM +0800, Wei Yang wrote: >On Thu, Dec 11, 2014 at 09:40:30AM +1100, Gavin Shan wrote: >>On Wed, Dec 10, 2014 at 10:29:37PM +0800, Wei Yang wrote: >>>The M64 range information is missed, which would be helpful in debug. >>> >>>This patch print the M64 range information in the same format as M32. s/print/prints >>> >>>Signed-off-by: Wei Yang <weiyang@linux.vnet.ibm.com> >>>--- >>> arch/powerpc/platforms/powernv/pci-ioda.c | 3 +++ >>> 1 file changed, 3 insertions(+) >>> >>>diff --git a/arch/powerpc/platforms/powernv/pci-ioda.c b/arch/powerpc/platforms/powernv/pci-ioda.c >>>index 21c8b4d..34f1a27 100644 >>>--- a/arch/powerpc/platforms/powernv/pci-ioda.c >>>+++ b/arch/powerpc/platforms/powernv/pci-ioda.c >>>@@ -369,6 +369,9 @@ static void __init pnv_ioda_parse_m64_window(struct pnv_phb *phb) >>> phb->ioda.m64_segsize = phb->ioda.m64_size / phb->ioda.total_pe; >>> phb->ioda.m64_base = pci_addr; >>> >>>+ printk(KERN_INFO " MEM64 0x%016llx..0x%016llx -> 0x%016llx\n", >>>+ res->start, res->end, pci_addr); >>>+ >> >>Please output segment size if you think it's helpful as well. > >The M64 segment size is printed in pnv_pci_init_ioda_phb() line 2620. > Yes, it has been printed separately. So the change looks good to me after you replace "printk(KERN_INFO ...)" with pr_info. Could you please send updated version and include following line if you want: Reviewed-by: Gavin Shan <gwshan@linux.vnet.ibm.com> Thanks, Gavin ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] powerpc/powernv: Print the M64 range information 2014-12-11 21:58 ` Gavin Shan @ 2014-12-12 4:28 ` Wei Yang 2014-12-12 4:39 ` [PATCH] powerpc/powernv: Print the M64 range information in bootup log Wei Yang 1 sibling, 0 replies; 6+ messages in thread From: Wei Yang @ 2014-12-12 4:28 UTC (permalink / raw) To: Gavin Shan; +Cc: Wei Yang, linuxppc-dev On Fri, Dec 12, 2014 at 08:58:19AM +1100, Gavin Shan wrote: >On Thu, Dec 11, 2014 at 03:29:01PM +0800, Wei Yang wrote: >>On Thu, Dec 11, 2014 at 09:40:30AM +1100, Gavin Shan wrote: >>>On Wed, Dec 10, 2014 at 10:29:37PM +0800, Wei Yang wrote: >>>>The M64 range information is missed, which would be helpful in debug. >>>> >>>>This patch print the M64 range information in the same format as M32. > >s/print/prints > >>>> >>>>Signed-off-by: Wei Yang <weiyang@linux.vnet.ibm.com> >>>>--- >>>> arch/powerpc/platforms/powernv/pci-ioda.c | 3 +++ >>>> 1 file changed, 3 insertions(+) >>>> >>>>diff --git a/arch/powerpc/platforms/powernv/pci-ioda.c b/arch/powerpc/platforms/powernv/pci-ioda.c >>>>index 21c8b4d..34f1a27 100644 >>>>--- a/arch/powerpc/platforms/powernv/pci-ioda.c >>>>+++ b/arch/powerpc/platforms/powernv/pci-ioda.c >>>>@@ -369,6 +369,9 @@ static void __init pnv_ioda_parse_m64_window(struct pnv_phb *phb) >>>> phb->ioda.m64_segsize = phb->ioda.m64_size / phb->ioda.total_pe; >>>> phb->ioda.m64_base = pci_addr; >>>> >>>>+ printk(KERN_INFO " MEM64 0x%016llx..0x%016llx -> 0x%016llx\n", >>>>+ res->start, res->end, pci_addr); >>>>+ >>> >>>Please output segment size if you think it's helpful as well. >> >>The M64 segment size is printed in pnv_pci_init_ioda_phb() line 2620. >> > >Yes, it has been printed separately. So the change looks good to me >after you replace "printk(KERN_INFO ...)" with pr_info. Could you please >send updated version and include following line if you want: Sure, glad to. > >Reviewed-by: Gavin Shan <gwshan@linux.vnet.ibm.com> > >Thanks, >Gavin -- Richard Yang Help you, Help me ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] powerpc/powernv: Print the M64 range information in bootup log 2014-12-11 21:58 ` Gavin Shan 2014-12-12 4:28 ` Wei Yang @ 2014-12-12 4:39 ` Wei Yang 1 sibling, 0 replies; 6+ messages in thread From: Wei Yang @ 2014-12-12 4:39 UTC (permalink / raw) To: gwshan, linuxppc-dev; +Cc: Wei Yang The M64 range information is missed in dmesg, which would be helpful in debug. This patch prints the M64 range information in the same format as M32. Signed-off-by: Wei Yang <weiyang@linux.vnet.ibm.com> Reviewed-by: Gavin Shan <gwshan@linux.vnet.ibm.com> V1: * use pr_info() instead of printk(KERN_INFO) --- arch/powerpc/platforms/powernv/pci-ioda.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/arch/powerpc/platforms/powernv/pci-ioda.c b/arch/powerpc/platforms/powernv/pci-ioda.c index 21c8b4d..4fbdc7d 100644 --- a/arch/powerpc/platforms/powernv/pci-ioda.c +++ b/arch/powerpc/platforms/powernv/pci-ioda.c @@ -369,6 +369,9 @@ static void __init pnv_ioda_parse_m64_window(struct pnv_phb *phb) phb->ioda.m64_segsize = phb->ioda.m64_size / phb->ioda.total_pe; phb->ioda.m64_base = pci_addr; + pr_info(" MEM64 0x%016llx..0x%016llx -> 0x%016llx\n", + res->start, res->end, pci_addr); + /* Use last M64 BAR to cover M64 window */ phb->ioda.m64_bar_idx = 15; phb->init_m64 = pnv_ioda2_init_m64; -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 6+ messages in thread
end of thread, other threads:[~2014-12-12 4:39 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-12-10 14:29 [PATCH] powerpc/powernv: Print the M64 range information Wei Yang 2014-12-10 22:40 ` Gavin Shan 2014-12-11 7:29 ` Wei Yang 2014-12-11 21:58 ` Gavin Shan 2014-12-12 4:28 ` Wei Yang 2014-12-12 4:39 ` [PATCH] powerpc/powernv: Print the M64 range information in bootup log Wei Yang
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.