> * Use either while holding wait_queue_head::lock or when used for wakeups
> - * with an extra smp_mb() like:
> + * with an extra smp_mb() like::
Independent of any philosophical discussion not allowing a setence to
end with a single ':' is completely idiotic. Please fix the tooling
instead to
Em Thu, 10 May 2018 01:38:38 -0700
Christoph Hellwig escreveu:
> > * Use either while holding wait_queue_head::lock or when used for wakeups
> > - * with an extra smp_mb() like:
> > + * with an extra smp_mb() like::
>
> Independent of any philosophical discussion not allowing a setence to
>
Hi Alan,
Sorry for the delay on the response, I only have time to proper analyze this
now.
On 24/04/2018 10:15, Alan Douglas wrote:
> Hi,
>
> On 10 April 2018 18:15 Gustavo Pimentel wrote:
>> Changes the pcie_raise_irq function signature, namely the interrupt_num
>> variable type from u8 to u16
Hi
09.05.2018, 16:11, "Jonathan Corbet" :
> On Wed, 9 May 2018 09:32:18 -0300
> Mauro Carvalho Chehab wrote:
>
>> > Looks good to me, thank you!
>> > What tree should this be forwarded to, or folks from linux doc will pick
>> it up?
>> >
>> > Acked-by: Evgeniy Polyakov
>>
>> Jon prefers if
On Thu, May 10, 2018 at 06:38:05AM -0300, Mauro Carvalho Chehab wrote:
> Em Thu, 10 May 2018 01:38:38 -0700
> Christoph Hellwig escreveu:
>
> > > * Use either while holding wait_queue_head::lock or when used for
> > > wakeups
> > > - * with an extra smp_mb() like:
> > > + * with an extra smp_m
On Wed, May 09, 2018 at 08:45:18AM -0600, Jonathan Corbet wrote:
> On Wed, 9 May 2018 10:41:20 +0200
> Peter Zijlstra wrote:
>
> > > This is easily done by using "::" instead of just ":".
> >
> > And I'll voice my objection once again. This makes a regular comment
> > worse. This rst stuff is
Em Thu, 10 May 2018 14:20:36 +0200
Peter Zijlstra escreveu:
> On Thu, May 10, 2018 at 06:38:05AM -0300, Mauro Carvalho Chehab wrote:
> > Em Thu, 10 May 2018 01:38:38 -0700
> > Christoph Hellwig escreveu:
> >
> > > > * Use either while holding wait_queue_head::lock or when used for
> > > >
On Thu, 10 May 2018 14:23:35 +0200
Andrea Parri wrote:
> only
> remember that other people (including some developers running into the
> "disadventure" of opening an RST doc. from their preferred text editor
> and being brought to conclude: "WTH! I need to open a web browser, I
> guess...") _us
On Thu, 10 May 2018 06:38:05 -0300
Mauro Carvalho Chehab wrote:
> (Peter said):
> > Independent of any philosophical discussion not allowing a setence to
> > end with a single ':' is completely idiotic. Please fix the tooling
> > instead to allow it, as it is very important for being able to jus
On Thu, 10 May 2018 07:30:12 -0600
Jonathan Corbet wrote:
> > (Peter said):
> > > Independent of any philosophical discussion not allowing a setence to
> > > end with a single ':' is completely idiotic. Please fix the tooling
> > > instead to allow it, as it is very important for being able to
Em Thu, 10 May 2018 07:30:12 -0600
Jonathan Corbet escreveu:
> On Thu, 10 May 2018 06:38:05 -0300
> Mauro Carvalho Chehab wrote:
>
> > (Peter said):
> > > Independent of any philosophical discussion not allowing a setence to
> > > end with a single ':' is completely idiotic. Please fix the t
When handling code-blocks, usual parsing should be disabled.
Indicate it by using the keyword disagram.
Signed-off-by: Mauro Carvalho Chehab
---
include/linux/wait.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/linux/wait.h b/include/linux/wait.h
index d9f131ecf708
Comments that end with a comma and have certain keywords
should be presented as code-blocks
Signed-off-by: Mauro Carvalho Chehab
---
scripts/kernel-doc | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/scripts/kernel-doc b/scripts/kernel-doc
index 0057d8eafcc1..8ce2d0f54369 1
On Thu, 10 May 2018 11:21:13 -0300
Mauro Carvalho Chehab wrote:
> The problem with a hint-based mechanism is that it will generate
> false hints. If added, we may end by needing to add extra tags to
> disable the hints mechanism where it gets wrong, or to periodically
> do code changes at kernel-
On 05/10/2018 07:38 AM, Mauro Carvalho Chehab wrote:
> When handling code-blocks, usual parsing should be disabled.
> Indicate it by using the keyword disagram.
diagram.
Please add such keyword(s) to one of
Documentation/doc-guide/{kernel-doc,sphinx}.rst.
>
>
On Thu, May 10, 2018 at 10:38:16AM -0400, Mauro Carvalho Chehab wrote:
> Comments that end with a comma and have certain keywords
> should be presented as code-blocks
I think this is a bit fragile. Why not just search for ':\n'? Is
there ever a case where we want to write:
/**
* foo is a bar:
Em Thu, 10 May 2018 09:38:46 -0600
Jonathan Corbet escreveu:
> On Thu, 10 May 2018 11:21:13 -0300
> Mauro Carvalho Chehab wrote:
>
> > The problem with a hint-based mechanism is that it will generate
> > false hints. If added, we may end by needing to add extra tags to
> > disable the hints mec
On Thu, 10 May 2018 09:34:56 -0700
Matthew Wilcox wrote:
> I think this is a bit fragile. Why not just search for ':\n'? Is
> there ever a case where we want to write:
>
> /**
> * foo is a bar:
> * wibble
> */
> and have wibble not be a code-block?
Yeah, we might want to write something li
On Thu, May 10, 2018 at 07:15:59AM -0600, Jonathan Corbet wrote:
> On Thu, 10 May 2018 14:23:35 +0200
> Andrea Parri wrote:
>
> > only
> > remember that other people (including some developers running into the
> > "disadventure" of opening an RST doc. from their preferred text editor
> > and bein
Em Thu, 10 May 2018 13:42:58 -0300
Mauro Carvalho Chehab escreveu:
> Em Thu, 10 May 2018 09:38:46 -0600
> Jonathan Corbet escreveu:
>
> > On Thu, 10 May 2018 11:21:13 -0300
> > Mauro Carvalho Chehab wrote:
> >
> > > The problem with a hint-based mechanism is that it will generate
> > > false
Em Thu, 10 May 2018 18:52:20 +0200
Andrea Parri escreveu:
> On Thu, May 10, 2018 at 07:15:59AM -0600, Jonathan Corbet wrote:
> > On Thu, 10 May 2018 14:23:35 +0200
> > Andrea Parri wrote:
> >
> > > only
> > > remember that other people (including some developers running into the
> > > "disadv
On Wed, May 09, 2018 at 10:18:45AM -0300, Mauro Carvalho Chehab wrote:
> The cgroup-v2.txt is already in ReST format. So, move it to the
> admin-guide, where it belongs.
>
> Cc: Tejun Heo
> Cc: Li Zefan
> Cc: Johannes Weiner
> Signed-off-by: Mauro Carvalho Chehab
Acked-by: Tejun Heo
Thanks.
Trivial patch to adjust the text formatting to wrap at 80 columns. No
actual content has changed.
Signed-off-by: Justin Skists
---
Documentation/process/5.Posting.rst | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/Documentation/process/5.Posting.rst
b/Docum
On Thu, 10 May 2018 11:40:18 -0700
Tejun Heo wrote:
> On Wed, May 09, 2018 at 10:18:45AM -0300, Mauro Carvalho Chehab wrote:
> > The cgroup-v2.txt is already in ReST format. So, move it to the
> > admin-guide, where it belongs.
> >
> > Cc: Tejun Heo
> > Cc: Li Zefan
> > Cc: Johannes Weiner
>
On Thu, 10 May 2018 20:37:37 +0100
Justin Skists wrote:
> Trivial patch to adjust the text formatting to wrap at 80 columns. No
> actual content has changed.
>
> Signed-off-by: Justin Skists
Applied, thanks.
jon
--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the bo
On Wed, 9 May 2018 16:23:41 +0800
"Huang, Ying" wrote:
> From: Huang Ying
>
> Add document for newly added thp_swpout, thp_swpout_fallback fields in
> /proc/vmstat.
>
> Signed-off-by: "Huang, Ying"
> Cc: "Kirill A. Shutemov"
> Cc: Andrea Arcangeli
> Cc: Johannes Weiner
Applied, thanks.
On Wed, 9 May 2018 10:18:43 -0300
Mauro Carvalho Chehab wrote:
> Patches 1 to 5 on this series contain the patches that weren't yet
> applied from the past patch series and touch only at Documentation.
> There are two changes there:
> patch 2: fixed the description and added a c/c to cgroup ma
Em Thu, 10 May 2018 14:22:35 -0600
Jonathan Corbet escreveu:
> On Wed, 9 May 2018 10:18:43 -0300
> Mauro Carvalho Chehab wrote:
>
> > Patches 1 to 5 on this series contain the patches that weren't yet
> > applied from the past patch series and touch only at Documentation.
> > There are two cha
On 05/10/2018 05:09 AM, Wang YanQing wrote:
> For me, as a reader whose mother language isn't English, the
> old words bring a little difficulty to catch the meaning, this
> patch rewords the subsection in a more clarificatory way.
>
> This patch also add blank lines as separator at two places
> t
"Alastair D'Silva" writes:
> diff --git a/include/uapi/misc/ocxl.h b/include/uapi/misc/ocxl.h
> index 0af83d80fb3e..8d2748e69c84 100644
> --- a/include/uapi/misc/ocxl.h
> +++ b/include/uapi/misc/ocxl.h
> @@ -48,6 +48,15 @@ struct ocxl_ioctl_metadata {
> __u64 reserved[13]; // Total of 16*u64
"Alastair D'Silva" writes:
> diff --git a/include/uapi/misc/ocxl.h b/include/uapi/misc/ocxl.h
> index 8d2748e69c84..bb80f294b429 100644
> --- a/include/uapi/misc/ocxl.h
> +++ b/include/uapi/misc/ocxl.h
> @@ -72,5 +75,6 @@ struct ocxl_ioctl_irq_fd {
> #define OCXL_IOCTL_IRQ_SET_FD_IOW(OCX
From: Alastair D'Silva
Switch the use of TIDR on it's CPU feature, rather than assuming it
is available based on architecture.
Signed-off-by: Alastair D'Silva
---
arch/powerpc/kernel/process.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/powerpc/kernel/process
From: Alastair D'Silva
In order for a userspace AFU driver to call the POWER9 specific
OCXL_IOCTL_ENABLE_P9_WAIT, it needs to verify that it can actually
make that call.
Signed-off-by: Alastair D'Silva
---
drivers/misc/ocxl/file.c | 25 +
include/uapi/misc/ocxl.h | 6 +
From: Alastair D'Silva
This patch adds a CPU feature bit to show whether the CPU has
the TIDR register available, enabling as_notify/wait in userspace.
Signed-off-by: Alastair D'Silva
---
arch/powerpc/include/asm/cputable.h | 3 ++-
arch/powerpc/kernel/dt_cpu_ftrs.c | 1 +
2 files changed, 3
From: Alastair D'Silva
Signed-off-by: Alastair D'Silva
---
Documentation/accelerators/ocxl.rst | 11 +++
1 file changed, 11 insertions(+)
diff --git a/Documentation/accelerators/ocxl.rst
b/Documentation/accelerators/ocxl.rst
index ddcc58d01cfb..14cefc020e2d 100644
--- a/Documentation/
From: Alastair D'Silva
The function removes the process element from NPU cache.
Signed-off-by: Alastair D'Silva
---
arch/powerpc/include/asm/pnv-ocxl.h | 2 +-
arch/powerpc/platforms/powernv/ocxl.c | 4 ++--
drivers/misc/ocxl/link.c | 2 +-
3 files changed, 4 insertions(+), 4 de
From: Alastair D'Silva
The current implementation of TID allocation, using a global IDR, may
result in an errant process starving the system of available TIDs.
Instead, use task_pid_nr(), as mentioned by the original author. The
scenario described which prevented it's use is not applicable, as
se
From: Alastair D'Silva
The Power 9 as_notify/wait feature provides a lower latency way to
signal a thread that work is complete. This series enables the use of
this feature from OpenCAPI adapters, as well as addressing a potential
starvation issue when allocating thread IDs.
Changelog:
v5:
Rem
From: Alastair D'Silva
In order to successfully issue as_notify, an AFU needs to know the TID
to notify, which in turn means that this information should be
available in userspace so it can be communicated to the AFU.
Signed-off-by: Alastair D'Silva
---
drivers/misc/ocxl/context.c | 5 +
39 matches
Mail list logo
