* [PATCH] video: fbdev: simplefb: Fix info message during probe @ 2020-12-28 18:39 Peter Robinson 2020-12-29 13:02 ` Hans de Goede 0 siblings, 1 reply; 5+ messages in thread From: Peter Robinson @ 2020-12-28 18:39 UTC (permalink / raw) To: Hans de Goede, linux-fbdev, dri-devel; +Cc: Peter Robinson The info message was showing the mapped address for the framebuffer. To avoid security problems, all virtual addresses are converted to __ptrval__, so the message has pointless information: simple-framebuffer 3ea9b000.framebuffer: framebuffer at 0x3ea9b000, 0x12c000 bytes, mapped to 0x(____ptrval____) Drop the extraneous bits to clean up the message: simple-framebuffer 3ea9b000.framebuffer: framebuffer at 0x3ea9b000, 0x12c000 bytes Signed-off-by: Peter Robinson <pbrobinson@gmail.com> --- drivers/video/fbdev/simplefb.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/video/fbdev/simplefb.c b/drivers/video/fbdev/simplefb.c index 533a047d07a2..62f0ded70681 100644 --- a/drivers/video/fbdev/simplefb.c +++ b/drivers/video/fbdev/simplefb.c @@ -477,9 +477,8 @@ static int simplefb_probe(struct platform_device *pdev) simplefb_clocks_enable(par, pdev); simplefb_regulators_enable(par, pdev); - dev_info(&pdev->dev, "framebuffer at 0x%lx, 0x%x bytes, mapped to 0x%p\n", - info->fix.smem_start, info->fix.smem_len, - info->screen_base); + dev_info(&pdev->dev, "framebuffer at 0x%lx, 0x%x bytes\n", + info->fix.smem_start, info->fix.smem_len); dev_info(&pdev->dev, "format=%s, mode=%dx%dx%d, linelength=%d\n", params.format->name, info->var.xres, info->var.yres, -- 2.29.2 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] video: fbdev: simplefb: Fix info message during probe 2020-12-28 18:39 [PATCH] video: fbdev: simplefb: Fix info message during probe Peter Robinson @ 2020-12-29 13:02 ` Hans de Goede 2021-01-07 17:04 ` Daniel Vetter 0 siblings, 1 reply; 5+ messages in thread From: Hans de Goede @ 2020-12-29 13:02 UTC (permalink / raw) To: Peter Robinson, linux-fbdev, dri-devel Hi, On 12/28/20 7:39 PM, Peter Robinson wrote: > The info message was showing the mapped address for the framebuffer. To avoid > security problems, all virtual addresses are converted to __ptrval__, so > the message has pointless information: > > simple-framebuffer 3ea9b000.framebuffer: framebuffer at 0x3ea9b000, 0x12c000 bytes, mapped to 0x(____ptrval____) > > Drop the extraneous bits to clean up the message: > > simple-framebuffer 3ea9b000.framebuffer: framebuffer at 0x3ea9b000, 0x12c000 bytes > > Signed-off-by: Peter Robinson <pbrobinson@gmail.com> Thanks, patch looks good to me: Reviewed-by: Hans de Goede <hdegoede@redhat.com> Regards, Hans > --- > drivers/video/fbdev/simplefb.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/drivers/video/fbdev/simplefb.c b/drivers/video/fbdev/simplefb.c > index 533a047d07a2..62f0ded70681 100644 > --- a/drivers/video/fbdev/simplefb.c > +++ b/drivers/video/fbdev/simplefb.c > @@ -477,9 +477,8 @@ static int simplefb_probe(struct platform_device *pdev) > simplefb_clocks_enable(par, pdev); > simplefb_regulators_enable(par, pdev); > > - dev_info(&pdev->dev, "framebuffer at 0x%lx, 0x%x bytes, mapped to 0x%p\n", > - info->fix.smem_start, info->fix.smem_len, > - info->screen_base); > + dev_info(&pdev->dev, "framebuffer at 0x%lx, 0x%x bytes\n", > + info->fix.smem_start, info->fix.smem_len); > dev_info(&pdev->dev, "format=%s, mode=%dx%dx%d, linelength=%d\n", > params.format->name, > info->var.xres, info->var.yres, > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] video: fbdev: simplefb: Fix info message during probe 2020-12-29 13:02 ` Hans de Goede @ 2021-01-07 17:04 ` Daniel Vetter 2021-01-20 11:16 ` Hans de Goede 0 siblings, 1 reply; 5+ messages in thread From: Daniel Vetter @ 2021-01-07 17:04 UTC (permalink / raw) To: Hans de Goede; +Cc: Peter Robinson, linux-fbdev, dri-devel Hi Hans, On Tue, Dec 29, 2020 at 02:02:30PM +0100, Hans de Goede wrote: > Hi, > > On 12/28/20 7:39 PM, Peter Robinson wrote: > > The info message was showing the mapped address for the framebuffer. To avoid > > security problems, all virtual addresses are converted to __ptrval__, so > > the message has pointless information: > > > > simple-framebuffer 3ea9b000.framebuffer: framebuffer at 0x3ea9b000, 0x12c000 bytes, mapped to 0x(____ptrval____) > > > > Drop the extraneous bits to clean up the message: > > > > simple-framebuffer 3ea9b000.framebuffer: framebuffer at 0x3ea9b000, 0x12c000 bytes > > > > Signed-off-by: Peter Robinson <pbrobinson@gmail.com> > > Thanks, patch looks good to me: > > Reviewed-by: Hans de Goede <hdegoede@redhat.com> Since you have commit rights for drm-misc I'm assuming you're also going to push this one? Thanks, Daniel > > Regards, > > Hans > > > --- > > drivers/video/fbdev/simplefb.c | 5 ++--- > > 1 file changed, 2 insertions(+), 3 deletions(-) > > > > diff --git a/drivers/video/fbdev/simplefb.c b/drivers/video/fbdev/simplefb.c > > index 533a047d07a2..62f0ded70681 100644 > > --- a/drivers/video/fbdev/simplefb.c > > +++ b/drivers/video/fbdev/simplefb.c > > @@ -477,9 +477,8 @@ static int simplefb_probe(struct platform_device *pdev) > > simplefb_clocks_enable(par, pdev); > > simplefb_regulators_enable(par, pdev); > > > > - dev_info(&pdev->dev, "framebuffer at 0x%lx, 0x%x bytes, mapped to 0x%p\n", > > - info->fix.smem_start, info->fix.smem_len, > > - info->screen_base); > > + dev_info(&pdev->dev, "framebuffer at 0x%lx, 0x%x bytes\n", > > + info->fix.smem_start, info->fix.smem_len); > > dev_info(&pdev->dev, "format=%s, mode=%dx%dx%d, linelength=%d\n", > > params.format->name, > > info->var.xres, info->var.yres, > > > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] video: fbdev: simplefb: Fix info message during probe 2021-01-07 17:04 ` Daniel Vetter @ 2021-01-20 11:16 ` Hans de Goede 2021-01-20 11:26 ` Daniel Vetter 0 siblings, 1 reply; 5+ messages in thread From: Hans de Goede @ 2021-01-20 11:16 UTC (permalink / raw) To: Daniel Vetter; +Cc: Peter Robinson, linux-fbdev, dri-devel Hi, On 1/7/21 6:04 PM, Daniel Vetter wrote: > Hi Hans, > > On Tue, Dec 29, 2020 at 02:02:30PM +0100, Hans de Goede wrote: >> Hi, >> >> On 12/28/20 7:39 PM, Peter Robinson wrote: >>> The info message was showing the mapped address for the framebuffer. To avoid >>> security problems, all virtual addresses are converted to __ptrval__, so >>> the message has pointless information: >>> >>> simple-framebuffer 3ea9b000.framebuffer: framebuffer at 0x3ea9b000, 0x12c000 bytes, mapped to 0x(____ptrval____) >>> >>> Drop the extraneous bits to clean up the message: >>> >>> simple-framebuffer 3ea9b000.framebuffer: framebuffer at 0x3ea9b000, 0x12c000 bytes >>> >>> Signed-off-by: Peter Robinson <pbrobinson@gmail.com> >> >> Thanks, patch looks good to me: >> >> Reviewed-by: Hans de Goede <hdegoede@redhat.com> > > Since you have commit rights for drm-misc I'm assuming you're also going > to push this one? I knew there was some discussion about using drm-misc for drivers/video/fbdev stuff but I missed that it was decided to go ahead with that. Good to know that this is handled through drm-misc now. I've pushed this patch to drm-misc-next. Regards, Hans >>> --- >>> drivers/video/fbdev/simplefb.c | 5 ++--- >>> 1 file changed, 2 insertions(+), 3 deletions(-) >>> >>> diff --git a/drivers/video/fbdev/simplefb.c b/drivers/video/fbdev/simplefb.c >>> index 533a047d07a2..62f0ded70681 100644 >>> --- a/drivers/video/fbdev/simplefb.c >>> +++ b/drivers/video/fbdev/simplefb.c >>> @@ -477,9 +477,8 @@ static int simplefb_probe(struct platform_device *pdev) >>> simplefb_clocks_enable(par, pdev); >>> simplefb_regulators_enable(par, pdev); >>> >>> - dev_info(&pdev->dev, "framebuffer at 0x%lx, 0x%x bytes, mapped to 0x%p\n", >>> - info->fix.smem_start, info->fix.smem_len, >>> - info->screen_base); >>> + dev_info(&pdev->dev, "framebuffer at 0x%lx, 0x%x bytes\n", >>> + info->fix.smem_start, info->fix.smem_len); >>> dev_info(&pdev->dev, "format=%s, mode=%dx%dx%d, linelength=%d\n", >>> params.format->name, >>> info->var.xres, info->var.yres, >>> >> >> _______________________________________________ >> dri-devel mailing list >> dri-devel@lists.freedesktop.org >> https://lists.freedesktop.org/mailman/listinfo/dri-devel > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] video: fbdev: simplefb: Fix info message during probe 2021-01-20 11:16 ` Hans de Goede @ 2021-01-20 11:26 ` Daniel Vetter 0 siblings, 0 replies; 5+ messages in thread From: Daniel Vetter @ 2021-01-20 11:26 UTC (permalink / raw) To: Hans de Goede; +Cc: Peter Robinson, Linux Fbdev development list, dri-devel On Wed, Jan 20, 2021 at 12:16 PM Hans de Goede <hdegoede@redhat.com> wrote: > > Hi, > > On 1/7/21 6:04 PM, Daniel Vetter wrote: > > Hi Hans, > > > > On Tue, Dec 29, 2020 at 02:02:30PM +0100, Hans de Goede wrote: > >> Hi, > >> > >> On 12/28/20 7:39 PM, Peter Robinson wrote: > >>> The info message was showing the mapped address for the framebuffer. To avoid > >>> security problems, all virtual addresses are converted to __ptrval__, so > >>> the message has pointless information: > >>> > >>> simple-framebuffer 3ea9b000.framebuffer: framebuffer at 0x3ea9b000, 0x12c000 bytes, mapped to 0x(____ptrval____) > >>> > >>> Drop the extraneous bits to clean up the message: > >>> > >>> simple-framebuffer 3ea9b000.framebuffer: framebuffer at 0x3ea9b000, 0x12c000 bytes > >>> > >>> Signed-off-by: Peter Robinson <pbrobinson@gmail.com> > >> > >> Thanks, patch looks good to me: > >> > >> Reviewed-by: Hans de Goede <hdegoede@redhat.com> > > > > Since you have commit rights for drm-misc I'm assuming you're also going > > to push this one? > > I knew there was some discussion about using drm-misc for drivers/video/fbdev stuff > but I missed that it was decided to go ahead with that. > > Good to know that this is handled through drm-misc now. This is official since a while: commit 6a7553e8d84d5322d883cb83bb9888c49a0f04e0 Author: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> Date: Fri Aug 9 15:46:40 2019 +0200 MAINTAINERS: handle fbdev changes through drm-misc tree Cheers, Daniel > > I've pushed this patch to drm-misc-next. > > Regards, > > Hans > > > > >>> --- > >>> drivers/video/fbdev/simplefb.c | 5 ++--- > >>> 1 file changed, 2 insertions(+), 3 deletions(-) > >>> > >>> diff --git a/drivers/video/fbdev/simplefb.c b/drivers/video/fbdev/simplefb.c > >>> index 533a047d07a2..62f0ded70681 100644 > >>> --- a/drivers/video/fbdev/simplefb.c > >>> +++ b/drivers/video/fbdev/simplefb.c > >>> @@ -477,9 +477,8 @@ static int simplefb_probe(struct platform_device *pdev) > >>> simplefb_clocks_enable(par, pdev); > >>> simplefb_regulators_enable(par, pdev); > >>> > >>> - dev_info(&pdev->dev, "framebuffer at 0x%lx, 0x%x bytes, mapped to 0x%p\n", > >>> - info->fix.smem_start, info->fix.smem_len, > >>> - info->screen_base); > >>> + dev_info(&pdev->dev, "framebuffer at 0x%lx, 0x%x bytes\n", > >>> + info->fix.smem_start, info->fix.smem_len); > >>> dev_info(&pdev->dev, "format=%s, mode=%dx%dx%d, linelength=%d\n", > >>> params.format->name, > >>> info->var.xres, info->var.yres, > >>> > >> > >> _______________________________________________ > >> dri-devel mailing list > >> dri-devel@lists.freedesktop.org > >> https://lists.freedesktop.org/mailman/listinfo/dri-devel > > > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-01-20 12:46 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2020-12-28 18:39 [PATCH] video: fbdev: simplefb: Fix info message during probe Peter Robinson 2020-12-29 13:02 ` Hans de Goede 2021-01-07 17:04 ` Daniel Vetter 2021-01-20 11:16 ` Hans de Goede 2021-01-20 11:26 ` Daniel Vetter
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox