{"push_id":964688123,"size":6,"distinct_size":6,"ref":"refs/heads/master","head":"13754e4c3d1b78945ecba225216f29d71334b77d","before":"3b8f2c8bcf7c6bf1339aa3c3f2d6f129b7878052","commits":[{"sha":"1b18aa1e79a0b343087d08075f117e821c33b930","author":{"email":"46ff6e41b02a6ff6bd50cc64faab96b75208d63d@sourceware.org","name":"GDB Administrator"},"message":"Automatic date update in version.in","distinct":true,"url":"https://api.github.com/repos/bminor/binutils-gdb/commits/1b18aa1e79a0b343087d08075f117e821c33b930"},{"sha":"c1d9289fef41b82aa22f63f74aa8e730ec898d3c","author":{"email":"2a3695df0edcde19d41e033e172c391a56c7af17@redhat.com","name":"Nick Clifton"},"message":"Fix the encoding of the MSP430's RRUX instruction.\n\n\tPR target/19561\nopcdoe\t* msp430-dis.c (print_insn_msp430): Add a special case for\n\tdecoding an RRC instruction with the ZC bit set in the extension\n\tword.\n\ninclude\t* opcode/msp430.h (IGNORE_CARRY_BIT): New define.\n\t(RRUX): Synthesise using case 2 rather than 7.\n\ngas\t* config/tc-msp430.c (msp430_operands): Remove case 7.  Use case 2\n\tto handle encoding of RRUX instruction.\n\t* testsuite/gas/msp430/msp430x.s: Add more tests of the extended\n\tshift instructions.\n\t* testsuite/gas/msp430/msp430x.d: Update expected disassembly.","distinct":true,"url":"https://api.github.com/repos/bminor/binutils-gdb/commits/c1d9289fef41b82aa22f63f74aa8e730ec898d3c"},{"sha":"39306124611b7c5a0bb12cba253364723fc1c5ee","author":{"email":"2a3695df0edcde19d41e033e172c391a56c7af17@redhat.com","name":"Nick Clifton"},"message":"Remove support for creating ARM NOREAD sections.\n\ngas\t* config/obj-elf.c (obj_elf_change_section): Remove support for\n\tARM NOREAD sections.\n\t* config/tc-arm.c (arm_elf_section_letter): Delete.\n\t* config/tc-arm.h (md_elf_section_letter): Delete.\n\t* doc/c-arm.texi (ARM Section Attribute): Delete section.\n\t* testsuite/gas/arm/section-execute-only.d: Delete.\n\t* testsuite/gas/arm/section-execute-only.s: Delete.\n\nld\t* testsuite/ld-arm/arm-elf.exp: Remove ARM NOREAD section tests.\n\t* testsuite/ld-arm/thumb1-input-section-flag-match.d: Delete.\n\t* testsuite/ld-arm/thumb1-input-section-flag-match.s: Delete.\n\t* testsuite/ld-arm/thumb1-noread-not-present-mixing-two-section.d: Delete.\n\t* testsuite/ld-arm/thumb1-noread-not-present-mixing-two-section.s: Delete.\n\t* testsuite/ld-arm/thumb1-noread-present-one-section.d: Delete.\n\t* testsuite/ld-arm/thumb1-noread-present-one-section.s: Delete.\n\t* testsuite/ld-arm/thumb1-noread-present-two-section.d: Delete.\n\t* testsuite/ld-arm/thumb1-noread-present-two-section.s: Delete.","distinct":true,"url":"https://api.github.com/repos/bminor/binutils-gdb/commits/39306124611b7c5a0bb12cba253364723fc1c5ee"},{"sha":"e42e5352d1d1e8a262178f606a5df3d0d988f78a","author":{"email":"82697eede7783c80d18ae8c0d02575d32c9ac359@linaro.org","name":"Yao Qi"},"message":"waiting_for_stop_reply around remote_fileio_request\n\nHi,\nI see this error when GDB connects with qemu,\n\n(gdb) n\n....\nSending packet: $vCont;c#a8...Ack\nPacket received: Ffstat,00000001,f6fff038\nCannot execute this command while the target is running.\nUse the \"interrupt\" command to stop the target\nand then try again.\n\nlooks we don't set rs->waiting_for_stop_reply to zero\nbefore handle fileio request,\n\n #10 0x00000000005edb64 in target_write (len=64, offset=4143968312, buf=0x7fffffffd570 \"\\375\\377\\377\\377\", annex=0x0, object=TARGET_OBJECT_MEMORY,\n    ops=<optimised out>) at /home/yao/SourceCode/gnu/gdb/git/gdb/target.c:1922\n #11 target_write_memory (memaddr=memaddr@entry=4143968312, myaddr=myaddr@entry=0x7fffffffd6a0 \"\", len=len@entry=64)\n    at /home/yao/SourceCode/gnu/gdb/git/gdb/target.c:1500\n #12 0x00000000004b2b41 in remote_fileio_func_fstat (buf=0x127b258 \"\") at /home/yao/SourceCode/gnu/gdb/git/gdb/remote-fileio.c:1037\n #13 0x00000000004b1878 in do_remote_fileio_request (uiout=<optimised out>, buf_arg=buf_arg@entry=0x127b240)\n    at /home/yao/SourceCode/gnu/gdb/git/gdb/remote-fileio.c:1204\n #14 0x00000000005b8c7c in catch_exceptions_with_msg (func_uiout=<optimised out>, func=func@entry=0x4b1800 <do_remote_fileio_request>,\n    func_args=func_args@entry=0x127b240, gdberrmsg=gdberrmsg@entry=0x0, mask=mask@entry=RETURN_MASK_ALL)\n    at /home/yao/SourceCode/gnu/gdb/git/gdb/exceptions.c:187\n #15 0x00000000005b8dea in catch_exceptions (uiout=<optimised out>, func=func@entry=0x4b1800 <do_remote_fileio_request>, func_args=func_args@entry=0x127b240,\n    mask=mask@entry=RETURN_MASK_ALL) at /home/yao/SourceCode/gnu/gdb/git/gdb/exceptions.c:167\n #16 0x00000000004b2fff in remote_fileio_request (buf=0x127b240 \"Xf6fff038,0:\", ctrlc_pending_p=0) at /home/yao/SourceCode/gnu/gdb/git/gdb/remote-fileio.c:1255\n #17 0x0000000000496f12 in remote_wait_as (ptid=..., status=0x7fffffffdb20, options=1) at /home/yao/SourceCode/gnu/gdb/git/gdb/remote.c:6997\n\nhowever, we did set rs->waiting_for_stop_reply to zero before Luis's\npatch https://sourceware.org/ml/gdb-patches/2015-10/msg00336.html\n\nIn fact, Luis's patch v1\nhttps://sourceware.org/ml/gdb-patches/2015-08/msg00809.html is about\nsetting rs->waiting_for_stop_reply back to one after\nremote_fileio_request, which is correct.  However during the review, the\npatch is changed and ends up with \"not setting rs->waiting_for_stop_reply\nto zero\".\n\nI manually test GDB, but I don't have a way to run regression tests.\n\ngdb:\n\n2016-02-04  Yao Qi  <yao.qi@linaro.org>\n\n\t* remote.c (remote_wait_as): Set rs->waiting_for_stop_reply to\n\t0 before handling 'F' and set it back afterwards.","distinct":true,"url":"https://api.github.com/repos/bminor/binutils-gdb/commits/e42e5352d1d1e8a262178f606a5df3d0d988f78a"},{"sha":"31d913c7e4e84edf88b4324c3a9a8fb31c8206ec","author":{"email":"82697eede7783c80d18ae8c0d02575d32c9ac359@linaro.org","name":"Yao Qi"},"message":"[testsuite] Remove BASEDIR\n\nBASEDIR was added by https://sourceware.org/ml/gdb-patches/2013-10/msg00587.html\nin order to handle the different directory layout in serial testing\nand parallel testing.  BASEDIR is \"gdb.base\" in serial testing and is\n\"outputs/gdb.base/TESTNAME\" in parallel testing.  However, it doesn't\nwork if the GDBserver is in remote target, like this,\n\n$ make check RUNTESTFLAGS='--target_board=remote-gdbserver-on-localhost foll-vfork.exp foll-exec.exp'\nFAIL: gdb.base/foll-exec.exp: continue to first exec catchpoint (the program exited)\nFAIL: gdb.base/foll-vfork.exp: exec: vfork and exec child follow, to main bp: continue to bp (the program exited)\nFAIL: gdb.base/foll-vfork.exp: exec: vfork child follow, finish after tcatch vfork: finish (the program exited)\nFAIL: gdb.base/foll-vfork.exp: exec: vfork relations in info inferiors: continue to bp (the program exited)\n\nthese tests fail because the executable can't be found.  With target\nboard native-gdbserver, the program is spawned this way,\n\n spawn ../gdbserver/gdbserver --once :2347 /scratch/yao/gdb/build-git/x86_64/gdb/testsuite/gdb.base/foll-vfork\n\nso BASEDIR is correct.  However, with target board\nremote-gdbserver-on-localhost, the program is spawned\n\n  spawn /usr/bin/ssh -l yao localhost /scratch/yao/gdb/build-git/x86_64/gdb/testsuite/../gdbserver/gdbserver --once :2346 /scratch/yao/gdb/build-git/x86_64/gdb/testsuite/gdb.base/foll-vfork\n\nso BASEDIR (either \"gdb.base\" or \"outputs/gdb.base/TESTNAME\") makes no\nsense.\n\nI had a fix that pass absolute directory to BASEDIR, but it assumes\nthat directory structure is the same on build and target, and it\ndoesn't work in remote host case.  The current fix in this patch is\nto get the directory from argv[0].  In any case, the program to be\nexec'ed is at the same directory with the main program.\n\nNote that these tests do \"next N\" to let program stop at the desired\nline, but it is fragile, because GDB for different targets may skip\nfunction prologue slightly differently, so I replace some of them by\n\"tbreak on LINE NUMBER and continue\".\n\ngdb/testsuite:\n\n2016-02-04  Yao Qi  <yao.qi@linaro.org>\n\n\t* gdb.base/foll-exec-mode.c: Include limits.h.\n\t(main): Add parameters argc and argv.  Get directory from\n\targv[0].\n\t* gdb.base/foll-exec-mode.exp: Don't pass -DBASEDIR in\n\tcompilation.\n\t* gdb.base/foll-exec.c: Include limits.h.\n\t(main): Add parameters argc and argv.\n\tGet directory from argv[0].\n\t* gdb.base/foll-exec.exp: Don't pass -DBASEDIR in compilation.\n\tAdjust tests on the number of lines as source code changed.\n\t* gdb.base/foll-vfork-exit.c: Include limits.h.\n\t(main): Add one line of statement before vfork.\n\t* gdb.base/foll-vfork.c: Include limits.h and string.h.\n\t(main): Add parameters argc and argv.  Get directory from\n\targv[0].\n\t* gdb.base/foll-vfork.exp: Don't pass -DBASEDIR in compilation.\n\t(setup_gdb): Set tbreak to skip some source lines.\n\t* gdb.multi/bkpt-multi-exec.c: Include limits.h.\n\t(main): Add parameters argc and argv.  Get directory from\n\targv[0].\n\t* gdb.multi/bkpt-multi-exec.exp: Don't pass -DBASEDIR in\n\tcompilation.\n\t* gdb.multi/multi-arch-exec.c: Include limits.h and string.h.\n\t(main): Add parameters argc and argv.  Get directory from\n\targv[0].\n\t* gdb.multi/multi-arch-exec.exp: Don't pass -DBASEDIR in\n\tcompilation.","distinct":true,"url":"https://api.github.com/repos/bminor/binutils-gdb/commits/31d913c7e4e84edf88b4324c3a9a8fb31c8206ec"},{"sha":"13754e4c3d1b78945ecba225216f29d71334b77d","author":{"email":"2a3695df0edcde19d41e033e172c391a56c7af17@redhat.com","name":"Nick Clifton"},"message":"Prevent possible undefined behaviour computing the size of the scache by usingunsigned integers instead of signed integers.\n\n\t* cgen-scache.c (scache_option_handler): Prevent possible\n\tundefined behaviour computing the size of the scache by using\n\tunsigned integers instead of signed integers.","distinct":true,"url":"https://api.github.com/repos/bminor/binutils-gdb/commits/13754e4c3d1b78945ecba225216f29d71334b77d"}]}