search for: corbet

On Fri, 30 Nov 2018 14:12:19 -0800 Jarkko Sakkinen <jarkko.sakkinen at linux.intel.com> wrote: > As a maintainer myself (and based on somewhat disturbed feedback from > other maintainers) I can only make the conclusion that nobody knows what > the responsibility part here means. > > I would interpret, if I read it like at lawyer at least, that even for > existing code you

...t;vbabka at suse.cz> > Cc: Joonsoo Kim <iamjoonsoo.kim at lge.com> > Cc: Mel Gorman <mgorman at suse.de> > Cc: Hugh Dickins <hughd at google.com> > Cc: Rafael Aquini <aquini at redhat.com> > Cc: virtualization at lists.linux-foundation.org > Cc: Jonathan Corbet <corbet at lwn.net> > Cc: John Einar Reitan <john.reitan at foss.arm.com> > Cc: dri-devel at lists.freedesktop.org > Cc: Sergey Senozhatsky <sergey.senozhatsky at gmail.com> > Signed-off-by: Gioh Kim <gi-oh.kim at profitbricks.com> > Signed-off-by: Minchan Kim...

...t;vbabka at suse.cz> > Cc: Joonsoo Kim <iamjoonsoo.kim at lge.com> > Cc: Mel Gorman <mgorman at suse.de> > Cc: Hugh Dickins <hughd at google.com> > Cc: Rafael Aquini <aquini at redhat.com> > Cc: virtualization at lists.linux-foundation.org > Cc: Jonathan Corbet <corbet at lwn.net> > Cc: John Einar Reitan <john.reitan at foss.arm.com> > Cc: dri-devel at lists.freedesktop.org > Cc: Sergey Senozhatsky <sergey.senozhatsky at gmail.com> > Signed-off-by: Gioh Kim <gi-oh.kim at profitbricks.com> > Signed-off-by: Minchan Kim...

...M- related UAPI docs do). - The %IDENTIFIER kerneldoc syntax is intended for "constants", and is more appropriate in this context than ``IDENTIFIER`` or &IDENTIFIER. The DRM docs aren't very consistent on this. Cc: Randy Dunlap <rdunlap at infradead.org> Cc: Jonathan Corbet <corbet at lwn.net> Signed-off-by: Vegard Nossum <vegard.nossum at oracle.com> --- include/uapi/drm/nouveau_drm.h | 56 ++++++++++++++++------------------ 1 file changed, 27 insertions(+), 29 deletions(-) diff --git a/include/uapi/drm/nouveau_drm.h b/include/uapi/drm/nouveau_drm.h ind...

This is based on Linux 5.7, plus one prerequisite patch: "mm/gup: update pin_user_pages.rst for "case 3" (mmu notifiers)" [1] Changes since v1: removed references to set_page_dirty*(), in response to Souptick Joarder's review (thanks!). Cover letter for v1, edited/updated slightly: It recently became clear to me that there are some get_user_pages*() callers that

Hi, It recently became clear to me that there are some get_user_pages*() callers that don't fit neatly into any of the four cases that are so far listed in pin_user_pages.rst. vhost.c is one of those. Add a Case 5 to the documentation, and refer to that when converting vhost.c. Thanks to Jan Kara for helping me (again) in understanding the interaction between get_user_pages() and page

On Fri, 2018-11-30 at 14:26 -0800, Jarkko Sakkinen wrote: > On Fri, Nov 30, 2018 at 03:14:59PM -0700, Jonathan Corbet wrote: [...] > > Have you read Documentation/process/code-of-conduct- > > interpretation.rst? > > As has been pointed out, it contains a clear answer to how things > > should be interpreted here. > > Ugh, was not aware that there two documents. > > Yeah, defin...

Hi Jon, On Fri, Nov 30, 2018 at 11:15 PM Jonathan Corbet <corbet at lwn.net> wrote: > On Fri, 30 Nov 2018 14:12:19 -0800 > Jarkko Sakkinen <jarkko.sakkinen at linux.intel.com> wrote: > > > As a maintainer myself (and based on somewhat disturbed feedback from > > other maintainers) I can only make the conclusion that nobod...

...d a fifth case, to help explain that there is a general pattern that requires pin_user_pages*() API calls. Cc: Vlastimil Babka <vbabka at suse.cz> Cc: Jan Kara <jack at suse.cz> Cc: J?r?me Glisse <jglisse at redhat.com> Cc: Dave Chinner <david at fromorbit.com> Cc: Jonathan Corbet <corbet at lwn.net> Cc: linux-doc at vger.kernel.org Cc: linux-fsdevel at vger.kernel.org Signed-off-by: John Hubbard <jhubbard at nvidia.com> --- Documentation/core-api/pin_user_pages.rst | 20 ++++++++++++++++++++ 1 file changed, 20 insertions(+) diff --git a/Documentation/core-api/...

...d a fifth case, to help explain that there is a general pattern that requires pin_user_pages*() API calls. Cc: Vlastimil Babka <vbabka at suse.cz> Cc: Jan Kara <jack at suse.cz> Cc: J?r?me Glisse <jglisse at redhat.com> Cc: Dave Chinner <david at fromorbit.com> Cc: Jonathan Corbet <corbet at lwn.net> Cc: linux-doc at vger.kernel.org Cc: linux-fsdevel at vger.kernel.org Signed-off-by: John Hubbard <jhubbard at nvidia.com> --- Documentation/core-api/pin_user_pages.rst | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) diff --git a/Documentation/core-api/pi...

...gt; - The %IDENTIFIER kerneldoc syntax is intended for "constants", and is > more appropriate in this context than ``IDENTIFIER`` or &IDENTIFIER. > The DRM docs aren't very consistent on this. > > Cc: Randy Dunlap <rdunlap at infradead.org> > Cc: Jonathan Corbet <corbet at lwn.net> > Signed-off-by: Vegard Nossum <vegard.nossum at oracle.com> This all looks good to me. Thanks for your help. Reviewed-by: Randy Dunlap <rdunlap at infradead.org> Tested-by: Randy Dunlap <rdunlap at infradead.org> I do see one thing that I don'...

...at there is a general pattern > that requires pin_user_pages*() API calls. > > Cc: Vlastimil Babka <vbabka at suse.cz> > Cc: Jan Kara <jack at suse.cz> > Cc: J?r?me Glisse <jglisse at redhat.com> > Cc: Dave Chinner <david at fromorbit.com> > Cc: Jonathan Corbet <corbet at lwn.net> > Cc: linux-doc at vger.kernel.org > Cc: linux-fsdevel at vger.kernel.org > Signed-off-by: John Hubbard <jhubbard at nvidia.com> > --- > Documentation/core-api/pin_user_pages.rst | 20 ++++++++++++++++++++ > 1 file changed, 20 insertions(+) > &g...

This patch describes what a subsystem should do for non-lru movable page supporting. Cc: Jonathan Corbet <corbet at lwn.net> Signed-off-by: Minchan Kim <minchan at kernel.org> --- Documentation/filesystems/vfs.txt | 11 ++++++- Documentation/vm/page_migration | 69 ++++++++++++++++++++++++++++++++++++++- 2 files changed, 78 insertions(+), 2 deletions(-) diff --git a/Documentation/files...

...This document is > +meant as a guide to using the various memory barriers provided by Linux, but > +in case of any doubt (and there are many) please ask. It might be worth adding you and me to the top of the file, to save Paul Cc'ing us on questions (get_maintainer.pl points at poor old Corbet for this file). But yes, it seems that something like this is required. Will

...This document is > +meant as a guide to using the various memory barriers provided by Linux, but > +in case of any doubt (and there are many) please ask. It might be worth adding you and me to the top of the file, to save Paul Cc'ing us on questions (get_maintainer.pl points at poor old Corbet for this file). But yes, it seems that something like this is required. Will

...err_foo: return ret; Provides some symmetry and makes it easy to add more cases as code calling goto does not need to worry about how is the error handled. Cc: Dan Carpenter <dan.carpenter at oracle.com> Cc: Julia Lawall <julia.lawall at lip6.fr> Cc: Jonathan Corbet <corbet at lwn.net> --- tools/virtio/ringtest/main.h | 4 +++- Documentation/CodingStyle | 44 ++++++++++++++++++++++++++++++++++++++++++-- 2 files changed, 45 insertions(+), 3 deletions(-) diff --git a/tools/virtio/ringtest/main.h b/tools/virtio/ringtest/main.h index 16917ac..e4d76c3 1...

...err_foo: return ret; Provides some symmetry and makes it easy to add more cases as code calling goto does not need to worry about how is the error handled. Cc: Dan Carpenter <dan.carpenter at oracle.com> Cc: Julia Lawall <julia.lawall at lip6.fr> Cc: Jonathan Corbet <corbet at lwn.net> --- tools/virtio/ringtest/main.h | 4 +++- Documentation/CodingStyle | 44 ++++++++++++++++++++++++++++++++++++++++++-- 2 files changed, 45 insertions(+), 3 deletions(-) diff --git a/tools/virtio/ringtest/main.h b/tools/virtio/ringtest/main.h index 16917ac..e4d76c3 1...

On 03/30/2016 09:12 AM, Minchan Kim wrote: > This patch describes what a subsystem should do for non-lru movable > page supporting. Intentionally reading this first without studying the code to better catch things that would seem obvious otherwise. > Cc: Jonathan Corbet <corbet at lwn.net> > Signed-off-by: Minchan Kim <minchan at kernel.org> > --- > Documentation/filesystems/vfs.txt | 11 ++++++- > Documentation/vm/page_migration | 69 ++++++++++++++++++++++++++++++++++++++- > 2 files changed, 78 insertions(+), 2 deletions(-) >...

On 03/30/2016 09:12 AM, Minchan Kim wrote: > This patch describes what a subsystem should do for non-lru movable > page supporting. Intentionally reading this first without studying the code to better catch things that would seem obvious otherwise. > Cc: Jonathan Corbet <corbet at lwn.net> > Signed-off-by: Minchan Kim <minchan at kernel.org> > --- > Documentation/filesystems/vfs.txt | 11 ++++++- > Documentation/vm/page_migration | 69 ++++++++++++++++++++++++++++++++++++++- > 2 files changed, 78 insertions(+), 2 deletions(-) >...

...0/2016 09:12 AM, Minchan Kim wrote: > >This patch describes what a subsystem should do for non-lru movable > >page supporting. > > Intentionally reading this first without studying the code to better > catch things that would seem obvious otherwise. > > >Cc: Jonathan Corbet <corbet at lwn.net> > >Signed-off-by: Minchan Kim <minchan at kernel.org> > >--- > > Documentation/filesystems/vfs.txt | 11 ++++++- > > Documentation/vm/page_migration | 69 ++++++++++++++++++++++++++++++++++++++- > > 2 files changed, 78 insertions(+),...