Re: [PATCH 0/2] video: fbdev: fix error handling, convert to pin_user_pages*()
On 2020-06-01 10:25, Andy Shevchenko wrote: On Mon, Jun 1, 2020 at 8:10 PM John Hubbard wrote: On 2020-06-01 03:35, Andy Shevchenko wrote: On Mon, Jun 1, 2020 at 1:00 AM John Hubbard wrote: On 2020-05-31 14:11, Andy Shevchenko wrote: ... JFYI, we have history.git starting from v0.01. OK, thanks for that note. According to that history.git [1], then: drivers/video/pvr2fb.c had get_user_pages_fast() support added to pvr2fb_write() back in 2004, but only for CONFIG_SH_DMA, as part of commit 434502754f2 ("[PATCH] SH Merge") ...and that commit created the minor bug that patch 0001 here addresses. (+Cc Paul just for the sake of completeness.) [1] git://git.kernel.org/pub/scm/linux/kernel/git/tglx/history.git I mentioned this one, but I guess content should be the same. https://git.kernel.org/pub/scm/linux/kernel/git/history/history.git/ Actually, that history.git *starts* at Linux 2.6.12-rc2, It's not true. OK I see, neither a straight "git log" nor git branches will suffice, you have to use tags in order to get to the older versions. while tglx/history.git *ends* at Linux 2.6.12-rc2 (which is in April, 2005). And the commit I was looking for is in 2004. So that's why I needed a different stretch of history. Actually history/history.git contains all of them starting from v0.01. But it ends, indeed, on 2.6.33. thanks, -- John Hubbard NVIDIA
Re: [PATCH 0/2] video: fbdev: fix error handling, convert to pin_user_pages*()
On Mon, Jun 1, 2020 at 8:10 PM John Hubbard wrote: > > On 2020-06-01 03:35, Andy Shevchenko wrote: > > On Mon, Jun 1, 2020 at 1:00 AM John Hubbard wrote: > >> On 2020-05-31 14:11, Andy Shevchenko wrote: > >>> ... > >>> JFYI, we have history.git starting from v0.01. > >>> > >> OK, thanks for that note. According to that history.git [1], > >> then: drivers/video/pvr2fb.c had get_user_pages_fast() support added to > >> pvr2fb_write() back in 2004, but only for CONFIG_SH_DMA, as part of > >> > >> commit 434502754f2 ("[PATCH] SH Merge") > >> > >> ...and that commit created the minor bug that patch 0001 here > >> addresses. (+Cc Paul just for the sake of completeness.) > >> > >> > >> [1] git://git.kernel.org/pub/scm/linux/kernel/git/tglx/history.git > > > > I mentioned this one, but I guess content should be the same. > > > > https://git.kernel.org/pub/scm/linux/kernel/git/history/history.git/ > > > > Actually, that history.git *starts* at Linux 2.6.12-rc2, It's not true. > while > tglx/history.git *ends* at Linux 2.6.12-rc2 (which is in April, 2005). > And the commit I was looking for is in 2004. So that's why I needed a > different stretch of history. Actually history/history.git contains all of them starting from v0.01. But it ends, indeed, on 2.6.33. -- With Best Regards, Andy Shevchenko
Re: [PATCH 0/2] video: fbdev: fix error handling, convert to pin_user_pages*()
On 2020-06-01 03:35, Andy Shevchenko wrote: On Mon, Jun 1, 2020 at 1:00 AM John Hubbard wrote: On 2020-05-31 14:11, Andy Shevchenko wrote: ... JFYI, we have history.git starting from v0.01. OK, thanks for that note. According to that history.git [1], then: drivers/video/pvr2fb.c had get_user_pages_fast() support added to pvr2fb_write() back in 2004, but only for CONFIG_SH_DMA, as part of commit 434502754f2 ("[PATCH] SH Merge") ...and that commit created the minor bug that patch 0001 here addresses. (+Cc Paul just for the sake of completeness.) [1] git://git.kernel.org/pub/scm/linux/kernel/git/tglx/history.git I mentioned this one, but I guess content should be the same. https://git.kernel.org/pub/scm/linux/kernel/git/history/history.git/ Actually, that history.git *starts* at Linux 2.6.12-rc2, while tglx/history.git *ends* at Linux 2.6.12-rc2 (which is in April, 2005). And the commit I was looking for is in 2004. So that's why I needed a different stretch of history. thanks, -- John Hubbard NVIDIA
Re: [PATCH 0/2] video: fbdev: fix error handling, convert to pin_user_pages*()
On Mon, Jun 1, 2020 at 1:00 AM John Hubbard wrote: > On 2020-05-31 14:11, Andy Shevchenko wrote: > > ... > > JFYI, we have history.git starting from v0.01. > > > OK, thanks for that note. According to that history.git [1], > then: drivers/video/pvr2fb.c had get_user_pages_fast() support added to > pvr2fb_write() back in 2004, but only for CONFIG_SH_DMA, as part of > > commit 434502754f2 ("[PATCH] SH Merge") > > ...and that commit created the minor bug that patch 0001 here > addresses. (+Cc Paul just for the sake of completeness.) > > > [1] git://git.kernel.org/pub/scm/linux/kernel/git/tglx/history.git I mentioned this one, but I guess content should be the same. https://git.kernel.org/pub/scm/linux/kernel/git/history/history.git/ -- With Best Regards, Andy Shevchenko
Re: [PATCH 0/2] video: fbdev: fix error handling, convert to pin_user_pages*()
On 2020-05-31 14:11, Andy Shevchenko wrote: ... JFYI, we have history.git starting from v0.01. OK, thanks for that note. According to that history.git [1], then: drivers/video/pvr2fb.c had get_user_pages_fast() support added to pvr2fb_write() back in 2004, but only for CONFIG_SH_DMA, as part of commit 434502754f2 ("[PATCH] SH Merge") ...and that commit created the minor bug that patch 0001 here addresses. (+Cc Paul just for the sake of completeness.) [1] git://git.kernel.org/pub/scm/linux/kernel/git/tglx/history.git thanks, -- John Hubbard NVIDIA
Re: [PATCH 0/2] video: fbdev: fix error handling, convert to pin_user_pages*()
On 2020-05-31 13:58, Sam Ravnborg wrote: ... Thanks, patches are now applied to drm-misc-next. They will hit -next soon, but you will have to wait until next (not the upcoming) merge window before they hit mainline linux. Sam Great! That will work out just fine. thanks, -- John Hubbard NVIDIA
Re: [PATCH 0/2] video: fbdev: fix error handling, convert to pin_user_pages*()
Hi John. On Thu, May 21, 2020 at 09:15:04PM -0700, John Hubbard wrote: > Hi, > > Note that I have only compile-tested this series, although that does > also include cross-compiling for a few other arches. I'm hoping that > this posting will lead to some run-time testing. > > Also: the proposed fix does not have a "Fixes:" tag, nor does it > Cc stable. That's because the issue has been there since the dawn of > git history for the kernel. If it's gone unnoticed this long, then > there is clearly no need for the relatively fast track of putting it > into stable, IMHO. But please correct me if that's wrong. > > Cc: Bartlomiej Zolnierkiewicz > Cc: Arnd Bergmann > Cc: Daniel Vetter > Cc: Gustavo A. R. Silva > Cc: Jani Nikula > Cc: dri-de...@lists.freedesktop.org > Cc: linux-fb...@vger.kernel.org Thanks, patches are now applied to drm-misc-next. They will hit -next soon, but you will have to wait until next (not the upcoming) merge window before they hit mainline linux. Sam > > John Hubbard (2): > video: fbdev: fix error handling for get_user_pages_fast() > video: fbdev: convert get_user_pages() --> pin_user_pages() > > drivers/video/fbdev/pvr2fb.c | 24 ++-- > 1 file changed, 18 insertions(+), 6 deletions(-) > > > base-commit: 051143e1602d90ea71887d92363edd539d411de5 > -- > 2.26.2 > > ___ > dri-devel mailing list > dri-de...@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel
[PATCH 0/2] video: fbdev: fix error handling, convert to pin_user_pages*()
Hi, Note that I have only compile-tested this series, although that does also include cross-compiling for a few other arches. I'm hoping that this posting will lead to some run-time testing. Also: the proposed fix does not have a "Fixes:" tag, nor does it Cc stable. That's because the issue has been there since the dawn of git history for the kernel. If it's gone unnoticed this long, then there is clearly no need for the relatively fast track of putting it into stable, IMHO. But please correct me if that's wrong. Cc: Bartlomiej Zolnierkiewicz Cc: Arnd Bergmann Cc: Daniel Vetter Cc: Gustavo A. R. Silva Cc: Jani Nikula Cc: dri-de...@lists.freedesktop.org Cc: linux-fb...@vger.kernel.org John Hubbard (2): video: fbdev: fix error handling for get_user_pages_fast() video: fbdev: convert get_user_pages() --> pin_user_pages() drivers/video/fbdev/pvr2fb.c | 24 ++-- 1 file changed, 18 insertions(+), 6 deletions(-) base-commit: 051143e1602d90ea71887d92363edd539d411de5 -- 2.26.2