* [PATCH] Documentation: vgaarbiter: fix typos
@ 2010-12-01 1:34 Nicolas Kaiser
2010-12-02 0:41 ` Randy Dunlap
0 siblings, 1 reply; 6+ messages in thread
From: Nicolas Kaiser @ 2010-12-01 1:34 UTC (permalink / raw)
To: Randy Dunlap; +Cc: Jesse Barnes, Tiago Vignatti, linux-doc, linux-kernel
Fixed typos.
Signed-off-by: Nicolas Kaiser <nikai@nikai.net>
---
Documentation/vgaarbiter.txt | 15 +++++++--------
1 files changed, 7 insertions(+), 8 deletions(-)
diff --git a/Documentation/vgaarbiter.txt b/Documentation/vgaarbiter.txt
index 43a9b06..8145f8f 100644
--- a/Documentation/vgaarbiter.txt
+++ b/Documentation/vgaarbiter.txt
@@ -14,11 +14,10 @@ the legacy VGA arbitration task (besides other bus management tasks) when more
than one legacy device co-exists on the same machine. But the problem happens
when these devices are trying to be accessed by different userspace clients
(e.g. two server in parallel). Their address assignments conflict. Moreover,
-ideally, being an userspace application, it is not the role of the the X
-server to control bus resources. Therefore an arbitration scheme outside of
-the X server is needed to control the sharing of these resources. This
-document introduces the operation of the VGA arbiter implemented for Linux
-kernel.
+ideally, being a userspace application, it is not the role of the X server to
+control bus resources. Therefore an arbitration scheme outside of the X server
+is needed to control the sharing of these resources. This document introduces
+the operation of the VGA arbiter implemented for the Linux kernel.
----------------------------------------------------------------------------
@@ -39,7 +38,7 @@ I.1 vgaarb
The vgaarb is a module of the Linux Kernel. When it is initially loaded, it
scans all PCI devices and adds the VGA ones inside the arbitration. The
arbiter then enables/disables the decoding on different devices of the VGA
-legacy instructions. Device which do not want/need to use the arbiter may
+legacy instructions. Devices which do not want/need to use the arbiter may
explicitly tell it by calling vga_set_legacy_decoding().
The kernel exports a char device interface (/dev/vga_arbiter) to the clients,
@@ -95,7 +94,7 @@ In the case of devices hot-{un,}plugged, there is a hook - pci_notify() - to
notify them being added/removed in the system and automatically added/removed
in the arbiter.
-There's also a in-kernel API of the arbiter in the case of DRM, vgacon and
+There's also an in-kernel API of the arbiter in the case of DRM, vgacon and
others which may use the arbiter.
@@ -117,7 +116,7 @@ Besides it, in pci_system were added:
struct pci_device *vga_default_dev;
-The vga_count is usually need to keep informed how many cards are being
+The vga_count is usually needed to keep informed how many cards are being
arbitrated, so for instance if there's only one then it can totally escape the
scheme.
--
1.7.2.2
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH] Documentation: vgaarbiter: fix typos
2010-12-01 1:34 [PATCH] Documentation: vgaarbiter: fix typos Nicolas Kaiser
@ 2010-12-02 0:41 ` Randy Dunlap
2010-12-06 17:25 ` Jesse Barnes
0 siblings, 1 reply; 6+ messages in thread
From: Randy Dunlap @ 2010-12-02 0:41 UTC (permalink / raw)
To: Nicolas Kaiser; +Cc: Jesse Barnes, Tiago Vignatti, linux-doc, linux-kernel
On Wed, 1 Dec 2010 02:34:57 +0100 Nicolas Kaiser wrote:
> Fixed typos.
>
> Signed-off-by: Nicolas Kaiser <nikai@nikai.net>
> ---
> Documentation/vgaarbiter.txt | 15 +++++++--------
> 1 files changed, 7 insertions(+), 8 deletions(-)
>
> diff --git a/Documentation/vgaarbiter.txt b/Documentation/vgaarbiter.txt
> index 43a9b06..8145f8f 100644
> --- a/Documentation/vgaarbiter.txt
> +++ b/Documentation/vgaarbiter.txt
> @@ -14,11 +14,10 @@ the legacy VGA arbitration task (besides other bus management tasks) when more
> than one legacy device co-exists on the same machine. But the problem happens
> when these devices are trying to be accessed by different userspace clients
> (e.g. two server in parallel). Their address assignments conflict. Moreover,
> -ideally, being an userspace application, it is not the role of the the X
> -server to control bus resources. Therefore an arbitration scheme outside of
> -the X server is needed to control the sharing of these resources. This
> -document introduces the operation of the VGA arbiter implemented for Linux
> -kernel.
> +ideally, being a userspace application, it is not the role of the X server to
> +control bus resources. Therefore an arbitration scheme outside of the X server
> +is needed to control the sharing of these resources. This document introduces
> +the operation of the VGA arbiter implemented for the Linux kernel.
>
> ----------------------------------------------------------------------------
>
> @@ -39,7 +38,7 @@ I.1 vgaarb
> The vgaarb is a module of the Linux Kernel. When it is initially loaded, it
> scans all PCI devices and adds the VGA ones inside the arbitration. The
> arbiter then enables/disables the decoding on different devices of the VGA
> -legacy instructions. Device which do not want/need to use the arbiter may
> +legacy instructions. Devices which do not want/need to use the arbiter may
> explicitly tell it by calling vga_set_legacy_decoding().
>
> The kernel exports a char device interface (/dev/vga_arbiter) to the clients,
> @@ -95,7 +94,7 @@ In the case of devices hot-{un,}plugged, there is a hook - pci_notify() - to
> notify them being added/removed in the system and automatically added/removed
> in the arbiter.
>
> -There's also a in-kernel API of the arbiter in the case of DRM, vgacon and
> +There's also an in-kernel API of the arbiter in the case of DRM, vgacon and
> others which may use the arbiter.
>
How about:
There is also an in-kernel API of the arbiter in case DRM, vgacon, or other
drivers want to use it.
>
> @@ -117,7 +116,7 @@ Besides it, in pci_system were added:
> struct pci_device *vga_default_dev;
>
>
> -The vga_count is usually need to keep informed how many cards are being
> +The vga_count is usually needed to keep informed how many cards are being
> arbitrated, so for instance if there's only one then it can totally escape the
> scheme.
Awkward. How about:
The vga_count is used to track how many cards are being arbitrated, so for
instance, if there is only one card, then it can completely escape arbitration.
>
> --
---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] Documentation: vgaarbiter: fix typos
2010-12-02 0:41 ` Randy Dunlap
@ 2010-12-06 17:25 ` Jesse Barnes
2010-12-06 17:33 ` Randy Dunlap
0 siblings, 1 reply; 6+ messages in thread
From: Jesse Barnes @ 2010-12-06 17:25 UTC (permalink / raw)
To: Randy Dunlap; +Cc: Nicolas Kaiser, Tiago Vignatti, linux-doc, linux-kernel
On Wed, 1 Dec 2010 16:41:41 -0800
Randy Dunlap <rdunlap@xenotime.net> wrote:
> On Wed, 1 Dec 2010 02:34:57 +0100 Nicolas Kaiser wrote:
>
> > Fixed typos.
> >
> > Signed-off-by: Nicolas Kaiser <nikai@nikai.net>
> > ---
> > Documentation/vgaarbiter.txt | 15 +++++++--------
> > 1 files changed, 7 insertions(+), 8 deletions(-)
> >
> > diff --git a/Documentation/vgaarbiter.txt b/Documentation/vgaarbiter.txt
> > index 43a9b06..8145f8f 100644
> > --- a/Documentation/vgaarbiter.txt
> > +++ b/Documentation/vgaarbiter.txt
> > @@ -14,11 +14,10 @@ the legacy VGA arbitration task (besides other bus management tasks) when more
> > than one legacy device co-exists on the same machine. But the problem happens
> > when these devices are trying to be accessed by different userspace clients
> > (e.g. two server in parallel). Their address assignments conflict. Moreover,
> > -ideally, being an userspace application, it is not the role of the the X
> > -server to control bus resources. Therefore an arbitration scheme outside of
> > -the X server is needed to control the sharing of these resources. This
> > -document introduces the operation of the VGA arbiter implemented for Linux
> > -kernel.
> > +ideally, being a userspace application, it is not the role of the X server to
> > +control bus resources. Therefore an arbitration scheme outside of the X server
> > +is needed to control the sharing of these resources. This document introduces
> > +the operation of the VGA arbiter implemented for the Linux kernel.
> >
> > ----------------------------------------------------------------------------
> >
> > @@ -39,7 +38,7 @@ I.1 vgaarb
> > The vgaarb is a module of the Linux Kernel. When it is initially loaded, it
> > scans all PCI devices and adds the VGA ones inside the arbitration. The
> > arbiter then enables/disables the decoding on different devices of the VGA
> > -legacy instructions. Device which do not want/need to use the arbiter may
> > +legacy instructions. Devices which do not want/need to use the arbiter may
> > explicitly tell it by calling vga_set_legacy_decoding().
> >
> > The kernel exports a char device interface (/dev/vga_arbiter) to the clients,
> > @@ -95,7 +94,7 @@ In the case of devices hot-{un,}plugged, there is a hook - pci_notify() - to
> > notify them being added/removed in the system and automatically added/removed
> > in the arbiter.
> >
> > -There's also a in-kernel API of the arbiter in the case of DRM, vgacon and
> > +There's also an in-kernel API of the arbiter in the case of DRM, vgacon and
> > others which may use the arbiter.
> >
>
> How about:
>
> There is also an in-kernel API of the arbiter in case DRM, vgacon, or other
> drivers want to use it.
>
> >
> > @@ -117,7 +116,7 @@ Besides it, in pci_system were added:
> > struct pci_device *vga_default_dev;
> >
> >
> > -The vga_count is usually need to keep informed how many cards are being
> > +The vga_count is usually needed to keep informed how many cards are being
> > arbitrated, so for instance if there's only one then it can totally escape the
> > scheme.
>
> Awkward. How about:
>
> The vga_count is used to track how many cards are being arbitrated, so for
> instance, if there is only one card, then it can completely escape arbitration.
Randy, I assume you'll pick this up once you receive a version you're
ok with?
Thanks,
--
Jesse Barnes, Intel Open Source Technology Center
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] Documentation: vgaarbiter: fix typos
2010-12-06 17:25 ` Jesse Barnes
@ 2010-12-06 17:33 ` Randy Dunlap
2011-05-14 10:51 ` [PATCH] Documentation: vgaarbiter: fix typos v2 Nicolas Kaiser
0 siblings, 1 reply; 6+ messages in thread
From: Randy Dunlap @ 2010-12-06 17:33 UTC (permalink / raw)
To: Jesse Barnes; +Cc: Nicolas Kaiser, Tiago Vignatti, linux-doc, linux-kernel
On Mon, 6 Dec 2010 09:25:44 -0800 Jesse Barnes wrote:
> On Wed, 1 Dec 2010 16:41:41 -0800
> Randy Dunlap <rdunlap@xenotime.net> wrote:
>
> > On Wed, 1 Dec 2010 02:34:57 +0100 Nicolas Kaiser wrote:
> >
> > > Fixed typos.
> > >
> > > Signed-off-by: Nicolas Kaiser <nikai@nikai.net>
> > > ---
> > > Documentation/vgaarbiter.txt | 15 +++++++--------
> > > 1 files changed, 7 insertions(+), 8 deletions(-)
> > >
> > > diff --git a/Documentation/vgaarbiter.txt b/Documentation/vgaarbiter.txt
> > > index 43a9b06..8145f8f 100644
> > > --- a/Documentation/vgaarbiter.txt
> > > +++ b/Documentation/vgaarbiter.txt
> > > @@ -14,11 +14,10 @@ the legacy VGA arbitration task (besides other bus management tasks) when more
> > > than one legacy device co-exists on the same machine. But the problem happens
> > > when these devices are trying to be accessed by different userspace clients
> > > (e.g. two server in parallel). Their address assignments conflict. Moreover,
> > > -ideally, being an userspace application, it is not the role of the the X
> > > -server to control bus resources. Therefore an arbitration scheme outside of
> > > -the X server is needed to control the sharing of these resources. This
> > > -document introduces the operation of the VGA arbiter implemented for Linux
> > > -kernel.
> > > +ideally, being a userspace application, it is not the role of the X server to
> > > +control bus resources. Therefore an arbitration scheme outside of the X server
> > > +is needed to control the sharing of these resources. This document introduces
> > > +the operation of the VGA arbiter implemented for the Linux kernel.
> > >
> > > ----------------------------------------------------------------------------
> > >
> > > @@ -39,7 +38,7 @@ I.1 vgaarb
> > > The vgaarb is a module of the Linux Kernel. When it is initially loaded, it
> > > scans all PCI devices and adds the VGA ones inside the arbitration. The
> > > arbiter then enables/disables the decoding on different devices of the VGA
> > > -legacy instructions. Device which do not want/need to use the arbiter may
> > > +legacy instructions. Devices which do not want/need to use the arbiter may
> > > explicitly tell it by calling vga_set_legacy_decoding().
> > >
> > > The kernel exports a char device interface (/dev/vga_arbiter) to the clients,
> > > @@ -95,7 +94,7 @@ In the case of devices hot-{un,}plugged, there is a hook - pci_notify() - to
> > > notify them being added/removed in the system and automatically added/removed
> > > in the arbiter.
> > >
> > > -There's also a in-kernel API of the arbiter in the case of DRM, vgacon and
> > > +There's also an in-kernel API of the arbiter in the case of DRM, vgacon and
> > > others which may use the arbiter.
> > >
> >
> > How about:
> >
> > There is also an in-kernel API of the arbiter in case DRM, vgacon, or other
> > drivers want to use it.
> >
> > >
> > > @@ -117,7 +116,7 @@ Besides it, in pci_system were added:
> > > struct pci_device *vga_default_dev;
> > >
> > >
> > > -The vga_count is usually need to keep informed how many cards are being
> > > +The vga_count is usually needed to keep informed how many cards are being
> > > arbitrated, so for instance if there's only one then it can totally escape the
> > > scheme.
> >
> > Awkward. How about:
> >
> > The vga_count is used to track how many cards are being arbitrated, so for
> > instance, if there is only one card, then it can completely escape arbitration.
>
> Randy, I assume you'll pick this up once you receive a version you're
> ok with?
I wasn't sure who would, but I can do that.
---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***
^ permalink raw reply [flat|nested] 6+ messages in thread* [PATCH] Documentation: vgaarbiter: fix typos v2
2010-12-06 17:33 ` Randy Dunlap
@ 2011-05-14 10:51 ` Nicolas Kaiser
2011-05-14 16:14 ` Randy Dunlap
0 siblings, 1 reply; 6+ messages in thread
From: Nicolas Kaiser @ 2011-05-14 10:51 UTC (permalink / raw)
To: Randy Dunlap; +Cc: Jesse Barnes, Tiago Vignatti, linux-doc, linux-kernel
Fixed typos.
v2: Incorporated changes by Randy Dunlap.
Signed-off-by: Nicolas Kaiser <nikai@nikai.net>
---
* Randy Dunlap <rdunlap@xenotime.net>:
> On Mon, 6 Dec 2010 09:25:44 -0800 Jesse Barnes wrote:
> > Randy, I assume you'll pick this up once you receive a version you're
> > ok with?
>
> I wasn't sure who would, but I can do that.
Sorry for the delay. I completely forgot about this patch.
Documentation/vgaarbiter.txt | 20 +++++++++-----------
1 files changed, 9 insertions(+), 11 deletions(-)
diff --git a/Documentation/vgaarbiter.txt b/Documentation/vgaarbiter.txt
index 43a9b06..4710230 100644
--- a/Documentation/vgaarbiter.txt
+++ b/Documentation/vgaarbiter.txt
@@ -14,11 +14,10 @@ the legacy VGA arbitration task (besides other bus management tasks) when more
than one legacy device co-exists on the same machine. But the problem happens
when these devices are trying to be accessed by different userspace clients
(e.g. two server in parallel). Their address assignments conflict. Moreover,
-ideally, being an userspace application, it is not the role of the the X
-server to control bus resources. Therefore an arbitration scheme outside of
-the X server is needed to control the sharing of these resources. This
-document introduces the operation of the VGA arbiter implemented for Linux
-kernel.
+ideally, being a userspace application, it is not the role of the X server to
+control bus resources. Therefore an arbitration scheme outside of the X server
+is needed to control the sharing of these resources. This document introduces
+the operation of the VGA arbiter implemented for the Linux kernel.
----------------------------------------------------------------------------
@@ -39,7 +38,7 @@ I.1 vgaarb
The vgaarb is a module of the Linux Kernel. When it is initially loaded, it
scans all PCI devices and adds the VGA ones inside the arbitration. The
arbiter then enables/disables the decoding on different devices of the VGA
-legacy instructions. Device which do not want/need to use the arbiter may
+legacy instructions. Devices which do not want/need to use the arbiter may
explicitly tell it by calling vga_set_legacy_decoding().
The kernel exports a char device interface (/dev/vga_arbiter) to the clients,
@@ -95,8 +94,8 @@ In the case of devices hot-{un,}plugged, there is a hook - pci_notify() - to
notify them being added/removed in the system and automatically added/removed
in the arbiter.
-There's also a in-kernel API of the arbiter in the case of DRM, vgacon and
-others which may use the arbiter.
+There is also an in-kernel API of the arbiter in case DRM, vgacon, or other
+drivers want to use it.
I.2 libpciaccess
@@ -117,9 +116,8 @@ Besides it, in pci_system were added:
struct pci_device *vga_default_dev;
-The vga_count is usually need to keep informed how many cards are being
-arbitrated, so for instance if there's only one then it can totally escape the
-scheme.
+The vga_count is used to track how many cards are being arbitrated, so for
+instance, if there is only one card, then it can completely escape arbitration.
These functions below acquire VGA resources for the given card and mark those
--
1.7.5.rc3
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH] Documentation: vgaarbiter: fix typos v2
2011-05-14 10:51 ` [PATCH] Documentation: vgaarbiter: fix typos v2 Nicolas Kaiser
@ 2011-05-14 16:14 ` Randy Dunlap
0 siblings, 0 replies; 6+ messages in thread
From: Randy Dunlap @ 2011-05-14 16:14 UTC (permalink / raw)
To: Nicolas Kaiser; +Cc: Jesse Barnes, Tiago Vignatti, linux-doc, linux-kernel
On Sat, 14 May 2011 12:51:52 +0200 Nicolas Kaiser wrote:
> Fixed typos.
>
> v2: Incorporated changes by Randy Dunlap.
>
> Signed-off-by: Nicolas Kaiser <nikai@nikai.net>
> ---
> * Randy Dunlap <rdunlap@xenotime.net>:
> > On Mon, 6 Dec 2010 09:25:44 -0800 Jesse Barnes wrote:
>
> > > Randy, I assume you'll pick this up once you receive a version you're
> > > ok with?
> >
> > I wasn't sure who would, but I can do that.
>
> Sorry for the delay. I completely forgot about this patch.
Applied, thanks.
> Documentation/vgaarbiter.txt | 20 +++++++++-----------
> 1 files changed, 9 insertions(+), 11 deletions(-)
>
> diff --git a/Documentation/vgaarbiter.txt b/Documentation/vgaarbiter.txt
> index 43a9b06..4710230 100644
> --- a/Documentation/vgaarbiter.txt
> +++ b/Documentation/vgaarbiter.txt
> @@ -14,11 +14,10 @@ the legacy VGA arbitration task (besides other bus management tasks) when more
> than one legacy device co-exists on the same machine. But the problem happens
> when these devices are trying to be accessed by different userspace clients
> (e.g. two server in parallel). Their address assignments conflict. Moreover,
> -ideally, being an userspace application, it is not the role of the the X
> -server to control bus resources. Therefore an arbitration scheme outside of
> -the X server is needed to control the sharing of these resources. This
> -document introduces the operation of the VGA arbiter implemented for Linux
> -kernel.
> +ideally, being a userspace application, it is not the role of the X server to
> +control bus resources. Therefore an arbitration scheme outside of the X server
> +is needed to control the sharing of these resources. This document introduces
> +the operation of the VGA arbiter implemented for the Linux kernel.
>
> ----------------------------------------------------------------------------
>
> @@ -39,7 +38,7 @@ I.1 vgaarb
> The vgaarb is a module of the Linux Kernel. When it is initially loaded, it
> scans all PCI devices and adds the VGA ones inside the arbitration. The
> arbiter then enables/disables the decoding on different devices of the VGA
> -legacy instructions. Device which do not want/need to use the arbiter may
> +legacy instructions. Devices which do not want/need to use the arbiter may
> explicitly tell it by calling vga_set_legacy_decoding().
>
> The kernel exports a char device interface (/dev/vga_arbiter) to the clients,
> @@ -95,8 +94,8 @@ In the case of devices hot-{un,}plugged, there is a hook - pci_notify() - to
> notify them being added/removed in the system and automatically added/removed
> in the arbiter.
>
> -There's also a in-kernel API of the arbiter in the case of DRM, vgacon and
> -others which may use the arbiter.
> +There is also an in-kernel API of the arbiter in case DRM, vgacon, or other
> +drivers want to use it.
>
>
> I.2 libpciaccess
> @@ -117,9 +116,8 @@ Besides it, in pci_system were added:
> struct pci_device *vga_default_dev;
>
>
> -The vga_count is usually need to keep informed how many cards are being
> -arbitrated, so for instance if there's only one then it can totally escape the
> -scheme.
> +The vga_count is used to track how many cards are being arbitrated, so for
> +instance, if there is only one card, then it can completely escape arbitration.
>
>
> These functions below acquire VGA resources for the given card and mark those
> --
---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2011-05-14 16:14 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-12-01 1:34 [PATCH] Documentation: vgaarbiter: fix typos Nicolas Kaiser
2010-12-02 0:41 ` Randy Dunlap
2010-12-06 17:25 ` Jesse Barnes
2010-12-06 17:33 ` Randy Dunlap
2011-05-14 10:51 ` [PATCH] Documentation: vgaarbiter: fix typos v2 Nicolas Kaiser
2011-05-14 16:14 ` Randy Dunlap
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox