On Thu, 7 Jul 2016 09:45:08 -0300 Mauricio Faria De Oliveira wrote:
Use the DMA_ATTR_NO_WARN attribute on dma_map_sg() calls of nvme driver.
Signed-off-by: Mauricio Faria de Oliveira
Reviewed-by: Gabriel Krisman Bertazi
---
Changelog:
v2:
- address warnings from checkpatch.pl (line wrapp
On Thu 2016-07-07 20:27:13, Topi Miettinen wrote:
> On 07/07/16 09:16, Petr Mladek wrote:
> > On Sun 2016-07-03 15:08:07, Topi Miettinen wrote:
> >> The attached patch would make any uses of capabilities generate audit
> >> messages. It works for simple tests as you can see from the commit
> >> mes
* Byungchul Park wrote:
> On Fri, Jul 08, 2016 at 07:50:39AM +0900, SeongJae Park wrote:
> > > I will add my opinion in korean.
> >
> > Thank you for kind and faithful review. I agree with most of your opinions
> > and
> > suggestions. Most of your suggestions looks much better than mine.
>
On Fri, Jul 8, 2016 at 7:13 PM, Ingo Molnar wrote:
>
> * Byungchul Park wrote:
>
>> On Fri, Jul 08, 2016 at 07:50:39AM +0900, SeongJae Park wrote:
>> > > I will add my opinion in korean.
>> >
>> > Thank you for kind and faithful review. I agree with most of your
>> > opinions and
>> > suggestio
On 07/08/2016 04:54 AM, Masayoshi Mizuma wrote:
Here, I think the error messages should not be suppressed because
the return value of nvme_map_data() is BLK_MQ_RQ_QUEUE_ERROR, so
the IO returns as -EIO.
Agree; good point. fixed in v3.
Thanks for reviewing.
--
Mauricio Faria de Oliveira
IBM L
Introduce the DMA_ATTR_NO_WARN attribute, and document it.
Signed-off-by: Mauricio Faria de Oliveira
---
Changelog:
v3:
- dma: none.
v2:
- all: address warnings from checkpatch.pl (line wrapping and typos)
Documentation/DMA-attributes.txt | 17 +
include/linux/dma-attrs.h
This patchset introduces dma_attr DMA_ATTR_NO_WARN (just like __GFP_NOWARN),
which tells the DMA-mapping subsystem to suppress allocation failure reports.
On some architectures allocation failures are reported with error messages
to the system logs. Although this can help to identify and debug pr
Use the DMA_ATTR_NO_WARN attribute on dma_map_sg() calls of nvme driver.
Signed-off-by: Mauricio Faria de Oliveira
Reviewed-by: Gabriel Krisman Bertazi
---
Changelog:
v3:
- nvme: use DMA_ATTR_NO_WARN when ret = BLK_MQ_RQ_QUEUE_BUSY (io will be
requeued) but not when ret = BLK_MQ_RQ_QUEUE_
Add support for the DMA_ATTR_NO_WARN attribute on powerpc iommu code.
Signed-off-by: Mauricio Faria de Oliveira
---
Changelog:
v3:
- powerpc: none
v2:
- all: address warnings from checkpatch.pl (line wrapping and typos)
arch/powerpc/kernel/iommu.c | 6 --
1 file changed, 4 insertions(
From: Markus Heiser
Implements the minimal boilerplate for Sphinx HTML theme customization.
Signed-off-by: Markus Heiser
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/conf.py | 9 -
Documentation/sphinx-static/theme_overrides.css | 9 +
2 fil
That's the second series of patches related to DocBook to ReST
conversion. With this patch series, we're ready to merge it
upstream.
There are still one thing to do: there are some new updates at
the DocBook pages on two topic branches (cec and vsp1). Those
changes should also be converted, in ord
There are some ioctls in upper case. This is not the standard.
Put them on lowercase, to match what's done with other ioctls.
Signed-off-by: Mauro Carvalho Chehab
---
...LEGACY_CMD.rst => fe-dishnetwork-send-legacy-cmd.rst} | 0
.../media/dvb/{FE_GET_EVENT.rst => fe-get-event.rst} | 0
...
From: Markus Heiser
The kernel-include directive is needed to include the auto generated rst
content from a build (pre-) process. E.g. the linux_tv Makefile
generates intermediate reST-files from header files. Since there is a O=
option:
make O=dir [targets] Locate all output files in "dir"
W
From: Markus Heiser
Include auto-generate reST header files. BTW fixed linux_tv/Makefile.
Signed-off-by: Markus Heiser
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/linux_tv/Makefile | 7 +--
Documentation/linux_tv/media/dvb/audio_h.rst| 6 +-
Documentatio
This file is actually not used to build the media uAPI docbook.
So, remove it.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/linux_tv/conf.py | 221 -
1 file changed, 221 deletions(-)
delete mode 100644 Documentation/linux_tv/conf.py
diff --git
On some syscall descriptions, the tables are described after
the return value. Do that inside descriptions.
Signed-off-by: Mauro Carvalho Chehab
---
.../linux_tv/media/dvb/audio-set-attributes.rst | 2 --
Documentation/linux_tv/media/dvb/audio-set-ext-id.rst | 2 --
.../linux_tv/media/dv
From: Markus Heiser
The layout of (table) captions in the RTD theme is a bit ugly and the
bordered, red colored of inline literals is a bit to gaudy. The
requirements has been discussed in the ML [1].
captions:
- captions should have 100% (not 85%) font size
- hide the permalink symbol as l
From: Markus Heiser
The default table layout of the RTD theme does not fit for vast tables,
like the ones we have in the linux_tv project. This has been discussed
on the ML [1].
The RTD theme is a two column layout, with a navigation column on the
left and a content column on the right:
content
The content of those macros are all declared at the v4l2-std-id
table. So, point to it.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/linux_tv/videodev2.h.rst.exceptions | 94 ---
1 file changed, 48 insertions(+), 46 deletions(-)
diff --git a/Documentation/linux_tv/
The conversion of this file didn't happen too well. We want
the items numbered, and format it just like what we did with
part 1 of the document.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/linux_tv/media/dvb/dvbapi.rst | 25 -
1 file changed, 16 insertions(+),
From: Markus Heiser
To honor the:
make O=dir [targets] Locate all output files in "dir"
* activate kernel-include directive
* export BUILDDIR=$(BUILDDIR)
* linux_tv: replace '.. include::' with '.. kernel-include::
$BUILDDIR/'
Signed-off-by: Markus Heiser
Signed-off-by: Mauro Carvalho Cheh
We should not let comments to mangle with the symbols
parsing. Unfortunately, videodev2.h has lots of those
in the middle of enums and structs. So, we need to improve
our parser to discard them.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/sphinx/parse-headers.pl | 17 ++---
After checking that all enum fields are documented at the
corresponding table on the rst file, let's point to the
table, instead of ignore the symbols.
A few symbols are not meant to be documented, as they're
deprecated stuff. keep ignoring them.
One enum field is not documented. Either it is obs
Add one extra escape character to avoid those warnings:
Documentation/linux_tv/videodev2.h.rst:6: WARNING: Inline
substitution_reference start-string without end-string.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/sphinx/parse-headers.pl | 2 +-
1 file changed, 1 insertion(+)
When something is deprecated, highlight it, as we want it
to be clearer to the reader.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/sphinx/parse-headers.pl | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Documentation/sphinx/parse-headers.pl
b/Documentation/sphinx/parse-headers.
At videodev2.h, we have hundreds of symbols that don't
currently have a reference yet. Let's ignore for how, while
we don't improve those cross-refs.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/sphinx/parse-headers.pl | 4
1 file changed, 4 insertions(+)
diff --git a/Documentati
Make it look just like v4l and DVB parts.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/linux_tv/media/mediactl/media-controller.rst | 12 +---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/Documentation/linux_tv/media/mediactl/media-controller.rst
b/Documentati
The include blocks were not properly displayed. Fix it.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/linux_tv/media/dvb/intro.rst | 18 --
1 file changed, 18 deletions(-)
diff --git a/Documentation/linux_tv/media/dvb/intro.rst
b/Documentation/linux_tv/media/dvb/intro.
The dmx.h header has two things that causes the parser to
break while handling enums:
per-header enums and the '{' starts on a new line
Both makes the parser to get lexical marks to be detected
as if they were symbols.
Fix it.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/sphinx/pars
Fixes this warning:
Documentation/linux_tv/media/v4l/dmabuf.rst:150: WARNING: undefined label:
vidioc_dqbuf (if the link has no caption the label must precede a section
header)
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/linux_tv/media/v4l/dmabuf.rst | 2 +-
1 file changed, 1 in
When I wrote the MC next gen patches, I also improved the media
controller documentation and added documentation for
MEDIA_IOC_G_TOPOLOGY, but I forgot to add the credits on that
time.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/linux_tv/media/mediactl/media-controller.rst | 7 +++
Several references were not converted right. That's why
so many symbols were lost when parsing videodev2.h header.
Fix them.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/linux_tv/media/dvb/dmx_types.rst | 8 +-
.../linux_tv/media/dvb/fe-bandwidth-t.rst | 14 +-
.../lin
The png image was not base64 decoded correctly.
Fix it.
Signed-off-by: Mauro Carvalho Chehab
---
.../linux_tv/media/dvb/intro_files/dvbstb.png | Bin 22703 -> 22655 bytes
1 file changed, 0 insertions(+), 0 deletions(-)
diff --git a/Documentation/linux_tv/media/dvb/intro_files/dvbstb.png
This file comes from the uAPI definition header, and
should be auto-generated, to be in sync with Kernel changes.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/linux_tv/Makefile | 10 +++--
Documentation/linux_tv/net.h.rst| 59 -
Docum
Now that the reference problems were solved, let's not
ignore anymore the pix formats, as all of them are already
documented.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/linux_tv/media/v4l/pixfmt-nv12.rst | 1 +
Documentation/linux_tv/media/v4l/pixfmt-nv12m.rst | 2 +
Documenta
The video, audio and OSD APIs are obsolete. V4L2 should be
used instead. So, remove them from this intro item.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/linux_tv/media/dvb/intro.rst | 5 -
1 file changed, 5 deletions(-)
diff --git a/Documentation/linux_tv/media/dvb/intro.rst
b
Fixes this warning:
Documentation/linux_tv/media/v4l/dev-overlay.rst:247: WARNING: Title underline
too short.
struct v4l2_clip [4]_
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/linux_tv/media/v4l/dev-overlay.rst | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
The V4L2 is the only part of the doc that has the word "Specification"
and mentions its version on the title.
Having the version there was important in the past, while we were
getting rid of V4L version 1. But, as v1 is long gone, all it lasts
is history (with is, btw, covered on the spec). So, no
This is an auto-generated header. Remove the hardcoded one
and do the right thing here.
NOTE: this is a deprecated API. So, we won't make any
effort to try identifying the meaning of this obscure
API that is used only on a legacy driver.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/li
This file comes from the uAPI definition header, and
should be auto-generated, to be in sync with Kernel changes.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/linux_tv/Makefile| 5 +-
Documentation/linux_tv/ca.h.rst| 97 --
Documenta
There's no need for all caps at its name. As the book title is
now showing at the top of each page, let's use Camel Case, to
make it less bold.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/linux_tv/index.rst | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentat
This file comes from the uAPI definition header, and
should be auto-generated, to be in sync with Kernel changes.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/linux_tv/Makefile | 5 +-
Documentation/linux_tv/video.h.rst| 280 --
Docum
Make it look like V4L, DVB and MC docbooks initial page.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/linux_tv/media/rc/remote_controllers.rst | 16
1 file changed, 12 insertions(+), 4 deletions(-)
diff --git a/Documentation/linux_tv/media/rc/remote_controllers.rst
b
This file should be auto-generated from the header files,
and not hardcoded.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/linux_tv/Makefile | 5 +-
Documentation/linux_tv/dmx.h.rst| 162
Documentation/linux_tv/dmx.h.rst.exceptions
The typedef handler should do two things to be generic:
1) parse typedef enums;
2) accept both possible syntaxes:
typedef struct foo { .. } foo_t;
typedef struct { .. } foo_t;
Unfortunately, this is needed to parse some legacy DVB
files, like dvb/audio.h.
Signed-off-by: Maur
Add a :widths: at the flat-table, to make it to look nicer.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/linux_tv/media/gen-errors.rst | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/linux_tv/media/gen-errors.rst
b/Documentation/linux_tv/media/gen-errors.rst
index 6d
The tables were not properly converted. It looked a little
ackward already at DocBook, but the conversion made it worse.
Fix them.
Signed-off-by: Mauro Carvalho Chehab
---
.../media/mediactl/media-ioc-g-topology.rst| 63 +++---
1 file changed, 6 insertions(+), 57 deletio
Add a :widths: at the flat-table, to improve the visual.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/linux_tv/media/mediactl/media-ioc-enum-entities.rst | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/linux_tv/media/mediactl/media-ioc-enum-entities.rst
b/Documentati
Remove a 'm' at the end of the last phrase.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/linux_tv/media/mediactl/media-controller-model.rst | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/linux_tv/media/mediactl/media-controller-model.rst
b/Documentat
Those cross references should point to media control syscalls,
and not to V4L ones.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/linux_tv/media/mediactl/media-func-close.rst | 2 +-
Documentation/linux_tv/media/mediactl/media-func-ioctl.rst | 4 ++--
Documentation/linux_tv/media/mediac
When typedef is used on its multiline format, we need to
also parse enum and struct in the same line.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/sphinx/parse-headers.pl | 9 +++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/Documentation/sphinx/parse-headers.pl
Cleanup the Makefile and handle the V=1 flag and make it
to work when specifying an output directory with O=dir
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/Makefile.sphinx | 2 +-
Documentation/linux_tv/Makefile | 49 +++--
2 files changed, 33 in
This script parses a header file and converts it into a
parsed-literal block, creating references for ioctls,
defines, typedefs, enums and structs.
It also allow an external file to modify the rules, in
order to fix the expressions.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/sphinx/
This file is auto-generated with DocBook, from the uapi header.
Do the same with Sphinx.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/Makefile.sphinx| 1 +
Documentation/linux_tv/Makefile | 12 +
Documentation/linux_tv/frontend.h.rst
Just like V4L and DVB parts, add the copyright symbol.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/linux_tv/media/mediactl/media-controller.rst | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/Documentation/linux_tv/media/mediactl/media-controller.rst
b/Docu
Remove most of ignore stuff for defines, pointing them to the
proper tables/sections.
Signed-off-by: Mauro Carvalho Chehab
---
.../linux_tv/media/v4l/vidioc-dv-timings-cap.rst | 8 +-
.../linux_tv/media/v4l/vidioc-g-dv-timings.rst | 26 +-
Documentation/linux_tv/videodev2.h.rst.exceptio
As we had to escape the symbols for the ReST markup to not do
the wrong thing, the logic to discover start/end of strings
are not trivial. Improve the end delimiter detection, in order
to highlight more occurrences of the strings.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/sphinx/par
Now that we have one syscall per page, using :cpp:function::
cleans up almost all warnings, with is a great thing.
Signed-off-by: Mauro Carvalho Chehab
---
.../media/dvb/audio-bilingual-channel-select.rst | 2 +-
.../linux_tv/media/dvb/audio-channel-select.rst | 2 +-
.../linux_tv/med
The Digital TV section is ackward for two reasons:
1) it is the only one with everything in upper case;
2) its name is associated with the European digital TV standard.
Rename the part name, and add a notice that it refers to what's
known as "DVB API".
Signed-off-by: Mauro Carvalho Chehab
-
Be more formal about the valid symbols that are expected by
the parser, to match what c language expects.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/sphinx/parse-headers.pl | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/Documentation/sphinx/parse
I suspect that this is a left over from Markus tests.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/linux_tv/media/dvb/audio_h.rst | 2 --
1 file changed, 2 deletions(-)
diff --git a/Documentation/linux_tv/media/dvb/audio_h.rst
b/Documentation/linux_tv/media/dvb/audio_h.rst
index 0ea0
On 07/08/2016 03:02 PM, Mauro Carvalho Chehab wrote:
> That's the second series of patches related to DocBook to ReST
> conversion. With this patch series, we're ready to merge it
> upstream.
>
> There are still one thing to do: there are some new updates at
> the DocBook pages on two topic branch
As commented on the patch series I just submitted, we finished the conversion
of the Media uAPI book from DocBook to ReST.
For now, I'm placing the new documentation, after parsed by Sphinx, at this
place:
https://mchehab.fedorapeople.org/media_API_book/
There are some instructions there
On 07/08/2016 03:34 PM, Mauro Carvalho Chehab wrote:
> As commented on the patch series I just submitted, we finished the conversion
> of the Media uAPI book from DocBook to ReST.
>
> For now, I'm placing the new documentation, after parsed by Sphinx, at this
> place:
> https://mchehab.fedor
Em Fri, 8 Jul 2016 15:45:52 +0200
Hans Verkuil escreveu:
> On 07/08/2016 03:34 PM, Mauro Carvalho Chehab wrote:
> > As commented on the patch series I just submitted, we finished the
> > conversion
> > of the Media uAPI book from DocBook to ReST.
> >
> > For now, I'm placing the new documentati
Commit 707e65831d3b("[media] DocBook: add dmabuf as streaming I/O
in VIDIOC_REQBUFS description") added DMABUF to reqbufs description,
but, as we're migrating to ReST markup, we need to keep it in sync
with the change.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/media/uapi/v4l/vidioc-
Changeset 8c9f46095176 ("[media] DocBook: mention the memory type to
be set for all streaming I/O") updated the media DocBook to mention
the need of filling the memory types. We need to keep the ReST
doc updated to such change.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/media/uapi/v4
Hello Mauro,
On Fri, Jul 8, 2016 at 3:05 PM, Mauro Carvalho Chehab
wrote:
> Changeset 8c9f46095176 ("[media] DocBook: mention the memory type to
> be set for all streaming I/O") updated the media DocBook to mention
> the need of filling the memory types. We need to keep the ReST
> doc updated to
Changeset 811c6d6a4243 ("[media] V4L: fix the Z16 format definition")
fixed the definition for DocBook, but we need to replicate it
also to ReST.
Signed-off-by: Mauro Carvalho Chehab
---
Documentation/media/uapi/v4l/pixfmt-z16.rst | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
Hello Mauro,
On Fri, Jul 8, 2016 at 2:41 PM, Mauro Carvalho Chehab
wrote:
> Commit 707e65831d3b("[media] DocBook: add dmabuf as streaming I/O
> in VIDIOC_REQBUFS description") added DMABUF to reqbufs description,
> but, as we're migrating to ReST markup, we need to keep it in sync
> with the chan
This patch adds the DMA_ATTR_PRIVILEGED_EXECUTABLE attribute to the
DMA-mapping subsystem.
Some architectures require that writable mappings also be non-executable at
lesser-privileged levels of execution. This attribute is used to indicate
to the DMA-mapping subsystem that it should do whatever
71 matches
Mail list logo
