linux-kernel - Re: [PATCH 1/3] xen: clean up xenbus internal headers

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>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <2acad511-b288-a7eb-8468-dca02c36be0a@suse.com>
Date:   Mon, 9 Jan 2017 08:07:42 +0100
From:   Juergen Gross <jgross@...e.com>
To:     Boris Ostrovsky <boris.ostrovsky@...cle.com>,
        linux-kernel@...r.kernel.org, xen-devel@...ts.xenproject.org
Subject: Re: [PATCH 1/3] xen: clean up xenbus internal headers

On 06/01/17 21:52, Boris Ostrovsky wrote:
> On 01/06/2017 10:05 AM, Juergen Gross wrote:
>> The xenbus driver has an awful mixture of internal and global visible
>> headers: some of the internal used only stuff is defined in the
>> global header include/xen/xenbus.h while some stuff defined in internal
>> headers is used by other drivers, too.
>>
>> Clean this up by moving the external used symbols to
>> include/xen/xenbus.h and the symbols used internal only to a new
> 
> I think
> 
> s/external/externally/
> s/internal/internally/

Hmm, yes.

>> header drivers/xen/xenbus/xenbus.h
> 
> ... and remove xenbus_comms.h

and xenbus_probe.h.

> 
>>
>> Signed-off-by: Juergen Gross <jgross@...e.com>
>> ---
>>  drivers/xen/xenbus/{xenbus_probe.h => xenbus.h} | 64 ++++++++++++++-----------
>>  drivers/xen/xenbus/xenbus_client.c              |  2 +-
>>  drivers/xen/xenbus/xenbus_comms.c               |  2 +-
>>  drivers/xen/xenbus/xenbus_comms.h               | 51 --------------------
>>  drivers/xen/xenbus/xenbus_dev_backend.c         |  2 +-
>>  drivers/xen/xenbus/xenbus_dev_frontend.c        |  4 +-
>>  drivers/xen/xenbus/xenbus_probe.c               |  3 +-
>>  drivers/xen/xenbus/xenbus_probe_backend.c       |  3 +-
>>  drivers/xen/xenbus/xenbus_probe_frontend.c      |  3 +-
>>  drivers/xen/xenbus/xenbus_xs.c                  |  3 +-
>>  drivers/xen/xenfs/super.c                       |  2 +-
>>  drivers/xen/xenfs/xenstored.c                   |  2 +-
>>  include/xen/xenbus.h                            | 12 ++---
>>  13 files changed, 52 insertions(+), 101 deletions(-)
>>  rename drivers/xen/xenbus/{xenbus_probe.h => xenbus.h} (58%)
>>  delete mode 100644 drivers/xen/xenbus/xenbus_comms.h
>>
>> diff --git a/drivers/xen/xenbus/xenbus_probe.h b/drivers/xen/xenbus/xenbus.h
>> similarity index 58%
>> rename from drivers/xen/xenbus/xenbus_probe.h
>> rename to drivers/xen/xenbus/xenbus.h
>> index c9ec7ca..6a80c1e 100644
>> --- a/drivers/xen/xenbus/xenbus_probe.h
>> +++ b/drivers/xen/xenbus/xenbus.h
>> @@ -1,10 +1,7 @@
>> -/******************************************************************************
>> - * xenbus_probe.h
>> - *
>> - * Talks to Xen Store to figure out what devices we have.
>> +/*
>> + * Private include for xenbus communications.
>>   *
>>   * Copyright (C) 2005 Rusty Russell, IBM Corporation
>> - * Copyright (C) 2005 XenSource Ltd.
> 
> Why is this?

Result of merging two header files. Will re-add.


Juergen