Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1440857

Re: [PATCH v2] relay: Add global mode support for buffer-only channels

Show all headers | View raw

On Mon, 11 Jul 2016 12:47:36 +0530 akash.goel@intel.com wrote:

> From: Akash Goel <akash.goel@intel.com>
> 
> The following patch added support to use channels with no associated files.
> 	relay: add buffer-only channels; useful for early logging

hm, 8 years ago.  Normally we refer to previous commits using the form
20d8b67c06fa5e74f44e ("relay: add buffer-only channels; useful for
early logging").  But this one is so old that we should inform readers
about its vintage, so this form:

commit 20d8b67c06fa5e74f44e80b0a0fd68c8327f7c6a
Author: Eduard - Gabriel Munteanu <eduard.munteanu@linux360.ro>
Date:   Fri Jul 25 19:45:12 2008 -0700

    relay: add buffer-only channels; useful for early logging

would be better.

> This is useful when the exact location of relay file is not known or the
> the parent directory of relay file is not available, while creating the
> channel and the logging has to start right from the boot.
> 
> But there was no provision to use global mode with buffer-only channels,
> which is added by this patch, without modifying the interface where initially
> there will be a dummy invocation of create_buf_file callback through which
> kernel client can convey the need of a global buffer.
> 
> For the use case where drivers/kernel clients want a simple interface for the
> userspace, which enables them to capture data/logs from relay file in order &
> without any post processing, support of Global buffer mode is warranted.
>
> ...
>
> @@ -706,6 +727,7 @@ int relay_late_setup_files(struct rchan *chan,
>  
>  	return err;
>  }
> +EXPORT_SYMBOL_GPL(relay_late_setup_files);

The export is unneeded and undocumented.

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v2] relay: Add global mode support for buffer-only channels akash.goel@intel.com - 2016-07-11 09:10 +0200
  Re: [PATCH v2] relay: Add global mode support for buffer-only  channels Andrew Morton <akpm@linux-foundation.org> - 2016-07-11 22:20 +0200
    Re: [PATCH v2] relay: Add global mode support for buffer-only  channels Chris Wilson <chris@chris-wilson.co.uk> - 2016-07-12 11:50 +0200
      Re: [PATCH v2] relay: Add global mode support for buffer-only  channels "Goel, Akash" <akash.goel@intel.com> - 2016-07-12 15:00 +0200
        Re: [PATCH v2] relay: Add global mode support for buffer-only  channels Chris Wilson <chris@chris-wilson.co.uk> - 2016-07-12 15:10 +0200
          Re: [PATCH v2] relay: Add global mode support for buffer-only  channels "Goel, Akash" <akash.goel@intel.com> - 2016-07-12 15:30 +0200
            Re: [PATCH v2] relay: Add global mode support for buffer-only  channels Andrew Morton <akpm@linux-foundation.org> - 2016-07-12 22:00 +0200
              [PATCH v3] relay: Add global mode support for buffer-only channels akash.goel@intel.com - 2016-07-13 09:30 +0200
                Re: [PATCH v3] relay: Add global mode support for buffer-only  channels kbuild test robot <lkp@intel.com> - 2016-07-13 10:40 +0200
                [PATCH v4] relay: Add global mode support for buffer-only channels akash.goel@intel.com - 2016-07-13 12:00 +0200

csiph-web