aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/core/lib
Commit message (Collapse)AuthorAge
* Merge pull request #13975 from danzh2010/adjustrcvbufGravatar Yang Gao2018-01-12
|\ | | | | change udp_server receive/send buffer size and set SO_RXQ_OVFL
* | Revert "Revert "Set error status correctly on server side""Gravatar Ken Payson2018-01-11
| | | | | | | | This reverts commit 951f84aea00a1f8a65cf160d7d8f342c30593000.
* | Revert "Set error status correctly on server side"Gravatar David G. Quintas2018-01-11
| |
| * formatGravatar Dan Zhang2018-01-11
| |
| * %zd->%dGravatar Dan Zhang2018-01-11
| |
| * change to int typeGravatar Dan Zhang2018-01-11
| |
| * fix portability errorsGravatar Dan Zhang2018-01-11
| |
* | Merge pull request #13676 from markdroth/inlined_vectorGravatar Mark D. Roth2018-01-11
|\ \ | | | | | | Implement InlinedVector independently of absl.
* | | exec_ctx_fwd.h should never have been in public headersGravatar Vijay Pai2018-01-10
| | |
| | * change to pass in valueGravatar Dan Zhang2018-01-10
| | |
| * | Implement InlinedVector independently of absl.Gravatar Mark D. Roth2018-01-10
| | |
| | * Adjust receiv buffer via setsockopt for udp_server's listening socket.Gravatar Dan Zhang2018-01-10
| |/ |/| | | | | | | Since this socket is used for all incoming traffic, its current buffer 1MB is appearantly too small. Change it to 10 MB for now.
* | Merge pull request #13912 from ncteisen/nullptrGravatar David G. Quintas2018-01-09
|\ \ | |/ |/| Missing s/NULL/nullptr
* | Merge pull request #13933 from kpayson64/avoid_stallGravatar kpayson642018-01-08
|\ \ | | | | | | Avoid stall
| * | Fix bug with pollhup workaroundGravatar Ken Payson2018-01-05
| | |
* | | Merge pull request #13931 from apolcyn/merge_18xGravatar apolcyn2018-01-05
|\ \ \ | | | | | | | | Merge 1.8.x into master
| * | | Manual merge conflict resolve: remove exec_ctx param from tcp_freeGravatar Alexander Polcyn2018-01-05
| | | |
| | * | Fix a Python spinlock bugGravatar Ken Payson2018-01-05
| | | |
* | | | Merge pull request #13900 from vjpai/odrGravatar Vijay Pai2018-01-05
|\ \ \ \ | | | | | | | | | | Wrap duplicated-name definitions in anonymous namespace (except for iomgr)
| | * | | Merge remote-tracking branch 'upstream/v1.8.x' into merge_18xGravatar Alexander Polcyn2018-01-05
| |/| | | |/| | | |
* | | | | Set error status correctly on server sideGravatar Ken Payson2018-01-04
| | | | |
| | * | | Mark tcp errors as UNAVAILABLE in UV tcp codeGravatar murgatroid992018-01-04
| | | | |
| | | | * Missing s/NULL/nullptrGravatar ncteisen2018-01-03
| |_|_|/ |/| | |
| * | | Revert unneeded changesGravatar Vijay Pai2018-01-03
| | | |
| * | | Wrap duplicated-name definitions in anonymous namespaceGravatar Vijay Pai2018-01-02
| | | |
* | | | Revert "Fix a Python spinlock bug"Gravatar Mehrdad Afshari2018-01-02
|/ / / | | | | | | | | | This reverts commit 2df509fc0e8628d6d4431139ce953c70796a21eb.
* | | Fix struct namesGravatar Vijay Pai2018-01-01
| | |
* | | Merge branch v1.8.x into masterGravatar Mehrdad Afshari2017-12-31
|\| | | | | | | | | | | Upmerge v1.8.3 into master
| * | Relax log-level when polling engines are undefinedGravatar Mehrdad Afshari2017-12-29
| | |
* | | Merge branch 'master' into startcbGravatar Dan Zhang2017-12-20
|\ \ \
| * \ \ Merge pull request #13834 from dgquintas/debug_location_namespaceGravatar David G. Quintas2017-12-20
| |\ \ \ | | | | | | | | | | Add namespace to macro expansion
* | | | | Add a start_cb to grpc_udp_listener to be called when listener isGravatar Dan Zhang2017-12-20
|/ / / / | | | | | | | | | | | | created.
| * | | Add namespace to macro expansionGravatar David Garcia Quintas2017-12-20
| | | |
* | | | Merge pull request #13811 from ita9naiwa/masterGravatar Noah Eisen2017-12-19
|\ \ \ \ | | | | | | | | | | Typo fix
* \ \ \ \ Merge pull request #13831 from dgquintas/fix_wrong_slice_unrefGravatar David G. Quintas2017-12-19
|\ \ \ \ \ | | | | | | | | | | | | Fix wrong unreffing of slice
* \ \ \ \ \ Merge pull request #13833 from ncteisen/commentsGravatar Noah Eisen2017-12-19
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | Add ownership semantic comments to error.h
| | | | * | Merge pull request #13665 from kpayson64/poll_spin_bugGravatar kpayson642017-12-19
| | | | |\ \ | | | | | | | | | | | | | | Fix a Python spinlock bug
| * | | | | | Add ownership semantic comments to error.hGravatar ncteisen2017-12-19
| | | | | | |
| | * | | | | Fix wrong unreffing of sliceGravatar David Garcia Quintas2017-12-19
| |/ / / / / |/| | | | |
| | | | * | Fix a Python spinlock bugGravatar Ken Payson2017-12-19
| | | | | |
* | | | | | Merge pull request #13494 from dgquintas/backoff_cppGravatar David G. Quintas2017-12-19
|\ \ \ \ \ \ | | | | | | | | | | | | | | C++-ize backoff
| * | | | | | Create copy of "ac" data instead of stretching the locked sectionGravatar David Garcia Quintas2017-12-19
| | | | | | |
| * | | | | | Fix use-after-free caused by unsync'd access in tcp_client_posix.Gravatar David Garcia Quintas2017-12-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | tc_on_alarm() and on_writable() race, resulting in the following: ``` D1219 08:59:33.425951347 86323 tcp_client_posix.cc:143] CLIENT_CONNECT: ipv4:127.0.0.1:27465: on_writable: error="No Error" D1219 08:59:33.426032150 86342 tcp_client_posix.cc:104] CLIENT_CONNECT: ipv4:127.0.0.1:27465: on_alarm: error="No Error" // At this point, note that the callbacks are running on different threads. D1219 08:59:33.426063521 86323 tcp_client_posix.cc:218] XXX on_writable ac->addr_str 0x603000008dd0 before unlock. # refs 2->1. Done 0 // on_writable() unrefs while still holding the lock. Because refs > 0, it marks its "done" as false and unlocks. D1219 08:59:33.426125130 86342 tcp_client_posix.cc:113] XXX tc_on_alarm ac->addr_str 0x603000008dd0 before unlock. # refs 1->0. Done 1 // right after on_writable() unlocks, tc_on_alarm() acquires the lock and unrefs, this time getting to zero and marking its "done" as true. // It then proceeds to destroy "ac", and, in particular for this failure, "ac->addr_str". D1219 08:59:33.426139370 86323 tcp_client_posix.cc:234] XXX on_writable about to read from ac->addr_str 0x603000008dd0. Done 0, error=OS Error // When on_writable() tries to read ac->addr_str to assemble its error details, it causes a use-after-free. ``` The problem is the lock isn't held long enough by on_writable(). Alternatively, a copy of ac->addr_str could be made in on_writable() while still holding the lock, but that seems more fragile. It doesn't seem that holding the lock longer would be a performance issue, given we are in a failure scenario.
* | | | | | | Fix internal TSAN bugGravatar Noah Eisen2017-12-19
| |/ / / / / |/| | | | |
* | | | | | Merge pull request #13798 from apolcyn/min_max_rpc_version_constantsGravatar apolcyn2017-12-18
|\ \ \ \ \ \ | | | | | | | | | | | | | | Define RPC protocol version constants
| * | | | | | Define RPC protocol version constantsGravatar Alexander Polcyn2017-12-18
| | | | | | |
| | | * | | | minor typo : becuase -> becauseGravatar ita9naiwa2017-12-19
| |_|/ / / / |/| | | | |
* | | | | | Merge pull request #13776 from ncteisen/executor-tsanGravatar Noah Eisen2017-12-15
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fix shutdown executor tsan bug
* \ \ \ \ \ \ Merge pull request #13771 from yang-g/deleteGravatar Yang Gao2017-12-15
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Mark tcp errors as UNAVAILABLE
| | | | * \ \ \ Merge branch 'master' of github.com:grpc/grpc into backoff_cppGravatar David Garcia Quintas2017-12-15
| | | | |\ \ \ \ | |_|_|_|/ / / / |/| | | | | | |
generated by cgit v1.2.3 (git 2.39.1) at 2024-07-08 03:24:05 +0000