[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Minios-devel] [UNIKRAFT/LIBBENCHMARK PATCH 1/5] Introduce library skeleton



Forgot to say: Config.uk should also have "select LIBNEWLIBC"

-- Felipe

On 10.10.19, 13:45, "Minios-devel on behalf of Felipe Huici" 
<minios-devel-bounces@xxxxxxxxxxxxxxxxxxxx on behalf of Felipe.Huici@xxxxxxxxx> 
wrote:

    Hi Vlad,
    
    Please find some minor comments below.
    
    -- Felipe
    
    On 30.09.19, 22:44, "Vlad-Andrei BĂDOIU (78692)" 
<vlad_andrei.badoiu@xxxxxxxxxxxxxxx> wrote:
    
        We use the BSD-3-Clause license for this library glue code.
        
        Signed-off-by: Vlad-Andrei Badoiu <vlad_andrei.badoiu@xxxxxxxxxxxxxxx>
        ---
         .gitignore      | 27 +++++++++++++++++++++++++++
         CODING_STYLE.md |  4 ++++
         CONTRIBUTING.md |  4 ++++
         COPYING.md      | 30 ++++++++++++++++++++++++++++++
         Config.uk       |  9 +++++++++
         MAINTAINERS.md  | 11 +++++++++++
         README.md       |  7 +++++++
         7 files changed, 92 insertions(+)
         create mode 100644 .gitignore
         create mode 100644 CODING_STYLE.md
         create mode 100644 CONTRIBUTING.md
         create mode 100644 COPYING.md
         create mode 100644 Config.uk
         create mode 100644 MAINTAINERS.md
         create mode 100644 README.md
        
        diff --git a/.gitignore b/.gitignore
        new file mode 100644
        index 0000000..75d2fd3
        --- /dev/null
        +++ b/.gitignore
        @@ -0,0 +1,27 @@
        +*.depend
        +*.o
        +*.a
        +*.d
        +*.so
        +*.orig
        +*.rej
        +*.bak
        +*.swp
        +*~
        +,*
        +\#*\#
        +.\#*
        +\#_*\#
        +.\#_*
        +!.gitignore
        +.*
        +
        +# gnu global files
        +GPATH
        +GRTAGS
        +GSYMS
        +GTAGS
        +
        +# cscope files
        +cscope.*
        +ncscope.*
        diff --git a/CODING_STYLE.md b/CODING_STYLE.md
        new file mode 100644
        index 0000000..5730041
        --- /dev/null
        +++ b/CODING_STYLE.md
        @@ -0,0 +1,4 @@
        +Coding Style
        +============
        +
        +Please refer to the `CODING_STYLE.md` file in the main Unikraft 
repository.
        diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
        new file mode 100644
        index 0000000..5f55eca
        --- /dev/null
        +++ b/CONTRIBUTING.md
        @@ -0,0 +1,4 @@
        +Contributing to Unikraft
        +=======================
        +
        +Please refer to the `CONTRIBUTING.md` file in the main Unikraft 
repository.
        diff --git a/COPYING.md b/COPYING.md
        new file mode 100644
        index 0000000..9e844d7
        --- /dev/null
        +++ b/COPYING.md
        @@ -0,0 +1,30 @@
        +License
        +=======
        +
        +Unikraft benchmark wrapper
        +-----------------------
        +
        +This repository contains wrapper code to build benchmark with Unikraft.
        +Each C code file in this repository should declare who is the
        +copyright owner and under which terms and conditions the code is
        +licensed. If such a licence note is missing, the following copyright
        +notice will apply:
        +
        +       Copyright (c) 2019, University Politehnica of Bucharest. All 
rights 
        +       reserved.
        +
        +       This library is free software; you can redistribute it and/or
        +       modify it under the terms of the GNU Lesser General Public
        +       License as published by the Free Software Foundation; either
        +       version 2 of the License, or (at your option) any later version.
        +       
        +       This library is distributed in the hope that it will be useful,
        +       but WITHOUT ANY WARRANTY; without even the implied warranty of
        +       MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the 
GNU
        +       Lesser General Public License for more details.
        +       
        +       You should have received a copy of the GNU Lesser General Public
        +       License along with this library in the file COPYING.LIB;
        +       if not, write to the Free Software Foundation, Inc.,
        +       59 Temple Place - Suite 330, Boston, MA 02111-1307, USA
        +
        diff --git a/Config.uk b/Config.uk
        new file mode 100644
        index 0000000..cae2f30
        --- /dev/null
        +++ b/Config.uk
        @@ -0,0 +1,9 @@
        +menuconfig LIBBENCHMARK
        +       bool "Benchmark - A microbenchmark support library"
    
    Might be good to write Google Benchmark, just benchmark is a bit generic.
    
        +       select LIBCXX
        +       select LIBPTHREAD_EMBEDDED
        +       select CXX_THREADS
        +       select UKUNISTD
        +       select UKSYSINFO
        +       default n
        +
    
    Please add select LIBUKTIME, I couldn't compile without it.
    
        diff --git a/MAINTAINERS.md b/MAINTAINERS.md
        new file mode 100644
        index 0000000..f8f998f
        --- /dev/null
        +++ b/MAINTAINERS.md
        @@ -0,0 +1,11 @@
        +Maintainers List
        +================
        +
        +For notes on how to read this information, please refer to 
`MAINTAINERS.md` in
        +the main Unikraft repository.
        +
        +       LIBBENCHMARK-UNIKRAFT
        +       M:      Felipe Huici <felipe.huici@xxxxxxxxx>
        +       M:      Vlad-Andrei Badoiu <vlad_andrei.badoiu@xxxxxxxxxxxxxxx>
        +       L:      minios-devel@xxxxxxxxxxxxx
        +       F: *
        diff --git a/README.md b/README.md
        new file mode 100644
        index 0000000..91a3844
        --- /dev/null
        +++ b/README.md
        @@ -0,0 +1,7 @@
        +benchmark for Unikraft
        +===================
        +
        +This is the port of benchmark for Unikraft as external library.
    
    We should probably add the LIBS line about dependencies, i.e., LIBS := 
$(UK_LIBS)/pthread-embedded:$(UK_LIBS)/compiler-rt:$(UK_LIBS)/libcxx:$(UK_LIBS)/libcxxabi:$(UK_LIBS)/libunwind:$(UK_LIBS)/newlib:$(UK_LIBS)/googlebenchmark
    
        +
        +Please refer to the `README.md` as well as the documentation in the 
`doc/`
        +subdirectory of the main unikraft repository.
        -- 
        2.20.1
        
        
    
    _______________________________________________
    Minios-devel mailing list
    Minios-devel@xxxxxxxxxxxxxxxxxxxx
    https://lists.xenproject.org/mailman/listinfo/minios-devel

_______________________________________________
Minios-devel mailing list
Minios-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/minios-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.