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

Re: [Minios-devel] [UNIKRAFT/LIBLUA PATCH v2 3/4] Add patches directory



Hi Felipe

Apologize for causing confusion.

It is caused my debugging code for your patch.

Reviewed-by: Jia He <justin.he@xxxxxxx>
---
Cheers,
Justin (Jia He)

On 2019/9/16 15:17, Felipe Huici wrote:
Hi Justin,

That's strange, is this using v2 of the series? Because if you look at patch #3 
(file: patches/0001-add-initrd-rename-entry-point.patch ) you can see that both 
cstr and img are declared and used, and so I don't get those warnings.

Thanks,

-- Felipe

On 16.09.19, 06:52, "Justin He (Arm Technology China)" <Justin.He@xxxxxxx> 
wrote:

     Hi Felipe
     Sorry, I have to withdraw my acceptance for this patch because my compiler 
complains
     on x86, gcc version 7.4.0 (Ubuntu 7.4.0-1ubuntu1~18.04.1)
     /root/hj/UK/apps/helloworld/build/liblua/origin/lua-5.3.5/src/lua.c:561:9: warning: 
unused variable ‘cstr’ [-Wunused-variable]
        char *cstr;
              ^~~~
     /root/hj/UK/apps/helloworld/build/liblua/origin/lua-5.3.5/src/lua.c:560:32: warning: 
unused variable ‘img’ [-Wunused-variable]
        struct ukplat_memregion_desc img;
                                     ^~~
> -----Original Message-----
     > From: Minios-devel <minios-devel-bounces@xxxxxxxxxxxxxxxxxxxx> On Behalf
     > Of Justin He (Arm Technology China)
     > Sent: 2019年9月16日 12:43
     > To: Felipe Huici <felipe.huici@xxxxxxxxx>; minios-devel@xxxxxxxxxxxxx
     > Subject: Re: [Minios-devel] [UNIKRAFT/LIBLUA PATCH v2 3/4] Add patches
     > directory
     >
     > Hi Felipe
     >
     > LGTM
     > Reviewed-by: Jia He <justin.he@xxxxxxx>
     > > -----Original Message-----
     > > From: Minios-devel <minios-devel-bounces@xxxxxxxxxxxxxxxxxxxx> On
     > Behalf
     > > Of Felipe Huici
     > > Sent: 2019年9月11日 20:01
     > > To: minios-devel@xxxxxxxxxxxxx
     > > Cc: Felipe Huici <felipe.huici@xxxxxxxxx>
     > > Subject: [Minios-devel] [UNIKRAFT/LIBLUA PATCH v2 3/4] Add patches
     > > directory
     > >
     > > Patch to rename main function and to add initrd support to main loop.
     > >
     > > Signed-off-by: Felipe Huici <felipe.huici@xxxxxxxxx>
     > > ---
     > >  patches/0001-add-initrd-rename-entry-point.patch | 42
     > > ++++++++++++++++++++++++
     > >  1 file changed, 42 insertions(+)
     > >  create mode 100644 patches/0001-add-initrd-rename-entry-point.patch
     > >
     > > diff --git a/patches/0001-add-initrd-rename-entry-point.patch
     > > b/patches/0001-add-initrd-rename-entry-point.patch
     > > new file mode 100644
     > > index 0000000..135a0c4
     > > --- /dev/null
     > > +++ b/patches/0001-add-initrd-rename-entry-point.patch
     > > @@ -0,0 +1,42 @@
     > > +--- a/src/lua.c      2017-04-19 19:29:57.000000000 +0200
     > > ++++ b/src/lua.c      2019-09-03 13:36:09.437807890 +0200
     > > +@@ -6,6 +6,8 @@
     > > +
     > > + #define lua_c
     > > +
     > > ++#include <uk/plat/memory.h>
     > > ++
     > > + #include "lprefix.h"
     > > +
     > > +
     > > +@@ -555,6 +557,9 @@
     > > +   int argc = (int)lua_tointeger(L, 1);
     > > +   char **argv = (char **)lua_touserdata(L, 2);
     > > +   int script;
     > > ++  struct ukplat_memregion_desc img;
     > > ++  char *cstr;
     > > ++
     > > +   int args = collectargs(argv, &script);
     > > +   luaL_checkversion(L);  /* check that interpreter has correct 
version */
     > > +   if (argv[0] && argv[0][0]) progname = argv[0];
     > > +@@ -581,6 +586,11 @@
     > > +     return 0;
     > > +   if (args & has_i)  /* -i option? */
     > > +     doREPL(L);  /* do read-eval-print loop */
     > > ++  /* see if script is available from initrd */
     > > ++  else if (ukplat_memregion_find_initrd0(&img) >= 0) {
     > > ++    cstr = (char *)img.base;
     > > ++    dostring(L, cstr, "initrd");
     > > ++  }
     > > +   else if (script == argc && !(args & (has_e | has_v))) {  /* no 
arguments?
     > > */
     > > +     if (lua_stdin_is_tty()) {  /* running in interactive mode? */
     > > +       print_version();
     > > +@@ -593,7 +603,7 @@
     > > + }
     > > +
     > > +
     > > +-int main (int argc, char **argv) {
     > > ++int lua_main (int argc, char **argv) {
     > > +   int status, result;
     > > +   lua_State *L = luaL_newstate();  /* create state */
     > > +   if (L == NULL) {
     > > --
     > > 2.11.0
     > >
     > >
     > > _______________________________________________
     > > Minios-devel mailing list
     > > Minios-devel@xxxxxxxxxxxxxxxxxxxx
     > > https://lists.xenproject.org/mailman/listinfo/minios-devel
     > IMPORTANT NOTICE: The contents of this email and any attachments are
     > confidential and may also be privileged. If you are not the intended
     > recipient, please notify the sender immediately and do not disclose the
     > contents to any other person, use it for any purpose, or store or copy 
the
     > information in any medium. Thank you.
     > _______________________________________________
     > Minios-devel mailing list
     > Minios-devel@xxxxxxxxxxxxxxxxxxxx
     > https://lists.xenproject.org/mailman/listinfo/minios-devel
     IMPORTANT NOTICE: The contents of this email and any attachments are 
confidential and may also be privileged. If you are not the intended recipient, 
please notify the sender immediately and do not disclose the contents to any 
other person, use it for any purpose, or store or copy the information in any 
medium. Thank you.
_______________________________________________
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®.