linux-kernel - Re: [PATCH] vgaarbiter: documentation grammar correction

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <Z6bKR9K16-oYMLFl@archie.me>
Date: Sat, 8 Feb 2025 10:06:47 +0700
From: Bagas Sanjaya <bagasdotme@...il.com>
To: Pranav Tyagi <pranav.tyagi03@...il.com>,
	Linux Documentation <linux-doc@...r.kernel.org>,
	Linux DRI Development <dri-devel@...ts.freedesktop.org>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Cc: linux-kernel-mentees@...ts.linux.dev, skhan@...uxfoundation.org,
	David Airlie <airlied@...il.com>, Simona Vetter <simona@...ll.ch>,
	Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
	Maxime Ripard <mripard@...nel.org>,
	Thomas Zimmermann <tzimmermann@...e.de>,
	Jonathan Corbet <corbet@....net>
Subject: Re: [PATCH] vgaarbiter: documentation grammar correction

On Fri, Feb 07, 2025 at 10:23:25PM +0530, Pranav Tyagi wrote:
> Corrects the following grammatical issues in the VGA Arbiter documentation:
> - Fixes subject-verb agreement by changing "co-exists" to "co-exist"
> - Corrects pluralization by changing "server" to "servers"
> - Improves sentence structure for clarity
> 

+Cc: DRM folks.

> Signed-off-by: Pranav Tyagi <pranav.tyagi03@...il.com>
> ---
>  Documentation/gpu/vgaarbiter.rst | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/Documentation/gpu/vgaarbiter.rst b/Documentation/gpu/vgaarbiter.rst
> index bde3c0afb059..d1e953712cc2 100644
> --- a/Documentation/gpu/vgaarbiter.rst
> +++ b/Documentation/gpu/vgaarbiter.rst
> @@ -11,9 +11,9 @@ Section 7, Legacy Devices.
>  
>  The Resource Access Control (RAC) module inside the X server [0] existed for
>  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
> +than one legacy device co-exist 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,
> +(e.g. two servers in parallel). Their address assignments conflict. Moreover,
>  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
> @@ -106,7 +106,7 @@ In-kernel interface
>  libpciaccess
>  ------------
>  
> -To use the vga arbiter char device it was implemented an API inside the
> +To use the vga arbiter char device, an API was implemented inside the
>  libpciaccess library. One field was added to struct pci_device (each device
>  on the system)::
>  

The diff looks OK.

Reviewed-by: Bagas Sanjaya <bagasdotme@...il.com>

-- 
An old man doll... just what I always wanted! - Clara

Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)