* [Qemu-trivial] [PATCH] linux-user/syscall.c: Fix typo issue for using target_vec[i].iov_len instead of target_vec[i].iov_base @ 2015-01-23 10:07 ` Chen Gang S 0 siblings, 0 replies; 16+ messages in thread From: Chen Gang S @ 2015-01-23 10:07 UTC (permalink / raw) To: riku.voipio; +Cc: QEMU Trivial, qemu-devel It is only a typo issue, need use tswapal(target_vec[i].iov_len) for the len. Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com> --- linux-user/syscall.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/linux-user/syscall.c b/linux-user/syscall.c index a66c2ae..8260ed7 100644 --- a/linux-user/syscall.c +++ b/linux-user/syscall.c @@ -1896,7 +1896,7 @@ static void unlock_iovec(struct iovec *vec, abi_ulong target_addr, if (target_vec) { for (i = 0; i < count; i++) { abi_ulong base = tswapal(target_vec[i].iov_base); - abi_long len = tswapal(target_vec[i].iov_base); + abi_long len = tswapal(target_vec[i].iov_len); if (len < 0) { break; } -- 1.9.3 (Apple Git-50) ^ permalink raw reply related [flat|nested] 16+ messages in thread
* [Qemu-devel] [PATCH] linux-user/syscall.c: Fix typo issue for using target_vec[i].iov_len instead of target_vec[i].iov_base @ 2015-01-23 10:07 ` Chen Gang S 0 siblings, 0 replies; 16+ messages in thread From: Chen Gang S @ 2015-01-23 10:07 UTC (permalink / raw) To: riku.voipio; +Cc: QEMU Trivial, qemu-devel It is only a typo issue, need use tswapal(target_vec[i].iov_len) for the len. Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com> --- linux-user/syscall.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/linux-user/syscall.c b/linux-user/syscall.c index a66c2ae..8260ed7 100644 --- a/linux-user/syscall.c +++ b/linux-user/syscall.c @@ -1896,7 +1896,7 @@ static void unlock_iovec(struct iovec *vec, abi_ulong target_addr, if (target_vec) { for (i = 0; i < count; i++) { abi_ulong base = tswapal(target_vec[i].iov_base); - abi_long len = tswapal(target_vec[i].iov_base); + abi_long len = tswapal(target_vec[i].iov_len); if (len < 0) { break; } -- 1.9.3 (Apple Git-50) ^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [Qemu-trivial] [Qemu-devel] [PATCH] linux-user/syscall.c: Fix typo issue for using target_vec[i].iov_len instead of target_vec[i].iov_base 2015-01-23 10:07 ` [Qemu-devel] " Chen Gang S @ 2015-01-23 10:08 ` Peter Maydell -1 siblings, 0 replies; 16+ messages in thread From: Peter Maydell @ 2015-01-23 10:08 UTC (permalink / raw) To: Chen Gang S; +Cc: QEMU Trivial, Riku Voipio, qemu-devel On 23 January 2015 at 10:07, Chen Gang S <gang.chen@sunrus.com.cn> wrote: > It is only a typo issue, need use tswapal(target_vec[i].iov_len) for the > len. > > Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com> > --- > linux-user/syscall.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/linux-user/syscall.c b/linux-user/syscall.c > index a66c2ae..8260ed7 100644 > --- a/linux-user/syscall.c > +++ b/linux-user/syscall.c > @@ -1896,7 +1896,7 @@ static void unlock_iovec(struct iovec *vec, abi_ulong target_addr, > if (target_vec) { > for (i = 0; i < count; i++) { > abi_ulong base = tswapal(target_vec[i].iov_base); > - abi_long len = tswapal(target_vec[i].iov_base); > + abi_long len = tswapal(target_vec[i].iov_len); > if (len < 0) { > break; > } > -- > 1.9.3 (Apple Git-50) Nice catch. Reviewed-by: Peter Maydell <peter.maydell@linaro.org> thanks -- PMM ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Qemu-devel] [PATCH] linux-user/syscall.c: Fix typo issue for using target_vec[i].iov_len instead of target_vec[i].iov_base @ 2015-01-23 10:08 ` Peter Maydell 0 siblings, 0 replies; 16+ messages in thread From: Peter Maydell @ 2015-01-23 10:08 UTC (permalink / raw) To: Chen Gang S; +Cc: QEMU Trivial, Riku Voipio, qemu-devel On 23 January 2015 at 10:07, Chen Gang S <gang.chen@sunrus.com.cn> wrote: > It is only a typo issue, need use tswapal(target_vec[i].iov_len) for the > len. > > Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com> > --- > linux-user/syscall.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/linux-user/syscall.c b/linux-user/syscall.c > index a66c2ae..8260ed7 100644 > --- a/linux-user/syscall.c > +++ b/linux-user/syscall.c > @@ -1896,7 +1896,7 @@ static void unlock_iovec(struct iovec *vec, abi_ulong target_addr, > if (target_vec) { > for (i = 0; i < count; i++) { > abi_ulong base = tswapal(target_vec[i].iov_base); > - abi_long len = tswapal(target_vec[i].iov_base); > + abi_long len = tswapal(target_vec[i].iov_len); > if (len < 0) { > break; > } > -- > 1.9.3 (Apple Git-50) Nice catch. Reviewed-by: Peter Maydell <peter.maydell@linaro.org> thanks -- PMM ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Qemu-trivial] [Qemu-devel] [PATCH] linux-user/syscall.c: Fix typo issue for using target_vec[i].iov_len instead of target_vec[i].iov_base 2015-01-23 10:08 ` Peter Maydell @ 2015-01-23 10:57 ` Chen Gang S -1 siblings, 0 replies; 16+ messages in thread From: Chen Gang S @ 2015-01-23 10:57 UTC (permalink / raw) To: Peter Maydell; +Cc: QEMU Trivial, Chris Metcalf, Riku Voipio, qemu-devel Thank you for all of your work. I plan to let qemu support tile architecture: "can let gcc run testsuite with qemu for tile". It is really hard to me, but I should try, it is my duty (I met it, and it's within my boarder, although no one pay for me). :-) At present, I am just preparing: - Reading architecture related documents (thank tile kernel member, who let me know about all tile architecture related documents). - Reading qemu architecture related code for understanding how to add tile code for qemu. - After preparing, I should provide a schedule for it (the steps, and the time point, and the risks which I may face to). Hope I can finish preparing within this month. Welcome any ideas, suggestions and completions by any members, e.g. - Share any related documents or knowledge for architecture. - Is there any original code for tile? Maybe originally, someone tried qemu for tile, but the related code was not merged into main line. - Suggestions about the schedule: the common steps, the time resources may need, and which risks have to face to. Thanks. On 1/23/15 18:08, Peter Maydell wrote: > On 23 January 2015 at 10:07, Chen Gang S <gang.chen@sunrus.com.cn> wrote: >> It is only a typo issue, need use tswapal(target_vec[i].iov_len) for the >> len. >> >> Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com> >> --- >> linux-user/syscall.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/linux-user/syscall.c b/linux-user/syscall.c >> index a66c2ae..8260ed7 100644 >> --- a/linux-user/syscall.c >> +++ b/linux-user/syscall.c >> @@ -1896,7 +1896,7 @@ static void unlock_iovec(struct iovec *vec, abi_ulong target_addr, >> if (target_vec) { >> for (i = 0; i < count; i++) { >> abi_ulong base = tswapal(target_vec[i].iov_base); >> - abi_long len = tswapal(target_vec[i].iov_base); >> + abi_long len = tswapal(target_vec[i].iov_len); >> if (len < 0) { >> break; >> } >> -- >> 1.9.3 (Apple Git-50) > > Nice catch. > Reviewed-by: Peter Maydell <peter.maydell@linaro.org> > > thanks > -- PMM > -- Chen Gang Open, share, and attitude like air, water, and life which God blessed ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Qemu-devel] [PATCH] linux-user/syscall.c: Fix typo issue for using target_vec[i].iov_len instead of target_vec[i].iov_base @ 2015-01-23 10:57 ` Chen Gang S 0 siblings, 0 replies; 16+ messages in thread From: Chen Gang S @ 2015-01-23 10:57 UTC (permalink / raw) To: Peter Maydell; +Cc: QEMU Trivial, Chris Metcalf, Riku Voipio, qemu-devel Thank you for all of your work. I plan to let qemu support tile architecture: "can let gcc run testsuite with qemu for tile". It is really hard to me, but I should try, it is my duty (I met it, and it's within my boarder, although no one pay for me). :-) At present, I am just preparing: - Reading architecture related documents (thank tile kernel member, who let me know about all tile architecture related documents). - Reading qemu architecture related code for understanding how to add tile code for qemu. - After preparing, I should provide a schedule for it (the steps, and the time point, and the risks which I may face to). Hope I can finish preparing within this month. Welcome any ideas, suggestions and completions by any members, e.g. - Share any related documents or knowledge for architecture. - Is there any original code for tile? Maybe originally, someone tried qemu for tile, but the related code was not merged into main line. - Suggestions about the schedule: the common steps, the time resources may need, and which risks have to face to. Thanks. On 1/23/15 18:08, Peter Maydell wrote: > On 23 January 2015 at 10:07, Chen Gang S <gang.chen@sunrus.com.cn> wrote: >> It is only a typo issue, need use tswapal(target_vec[i].iov_len) for the >> len. >> >> Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com> >> --- >> linux-user/syscall.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/linux-user/syscall.c b/linux-user/syscall.c >> index a66c2ae..8260ed7 100644 >> --- a/linux-user/syscall.c >> +++ b/linux-user/syscall.c >> @@ -1896,7 +1896,7 @@ static void unlock_iovec(struct iovec *vec, abi_ulong target_addr, >> if (target_vec) { >> for (i = 0; i < count; i++) { >> abi_ulong base = tswapal(target_vec[i].iov_base); >> - abi_long len = tswapal(target_vec[i].iov_base); >> + abi_long len = tswapal(target_vec[i].iov_len); >> if (len < 0) { >> break; >> } >> -- >> 1.9.3 (Apple Git-50) > > Nice catch. > Reviewed-by: Peter Maydell <peter.maydell@linaro.org> > > thanks > -- PMM > -- Chen Gang Open, share, and attitude like air, water, and life which God blessed ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Qemu-trivial] [Qemu-devel] [PATCH] linux-user/syscall.c: Fix typo issue for using target_vec[i].iov_len instead of target_vec[i].iov_base 2015-01-23 10:57 ` Chen Gang S @ 2015-01-23 11:21 ` Peter Maydell -1 siblings, 0 replies; 16+ messages in thread From: Peter Maydell @ 2015-01-23 11:21 UTC (permalink / raw) To: Chen Gang S; +Cc: QEMU Trivial, Chris Metcalf, Riku Voipio, qemu-devel On 23 January 2015 at 10:57, Chen Gang S <gang.chen@sunrus.com.cn> wrote: > > Thank you for all of your work. > > I plan to let qemu support tile architecture: "can let gcc run testsuite > with qemu for tile". It is really hard to me, but I should try, it is my > duty (I met it, and it's within my boarder, although no one pay for me). > :-) > Welcome any ideas, suggestions and completions by any members My single biggest suggestion is to make sure you split the work up so that you can post parts of it as you go along. Typically for a target-*/ new port that looks something like * initial framework, but no instruction decode (so all instructions just throw an illegal instruction exception) * support for one or a few simple classes of instruction * gradually fill in the gaps of missing instructions This way you get the benefit of code review on your early patches, so you can write the later ones in line with the comments you get on the first patches. We also get to review the code in smaller and more manageable chunks. I think the last target code to go in was target-tricore; you could look back in the git history and the archives to get an idea of how to structure the patchset and the kinds of review comment that patchseries received. thanks -- PMM ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Qemu-devel] [PATCH] linux-user/syscall.c: Fix typo issue for using target_vec[i].iov_len instead of target_vec[i].iov_base @ 2015-01-23 11:21 ` Peter Maydell 0 siblings, 0 replies; 16+ messages in thread From: Peter Maydell @ 2015-01-23 11:21 UTC (permalink / raw) To: Chen Gang S; +Cc: QEMU Trivial, Chris Metcalf, Riku Voipio, qemu-devel On 23 January 2015 at 10:57, Chen Gang S <gang.chen@sunrus.com.cn> wrote: > > Thank you for all of your work. > > I plan to let qemu support tile architecture: "can let gcc run testsuite > with qemu for tile". It is really hard to me, but I should try, it is my > duty (I met it, and it's within my boarder, although no one pay for me). > :-) > Welcome any ideas, suggestions and completions by any members My single biggest suggestion is to make sure you split the work up so that you can post parts of it as you go along. Typically for a target-*/ new port that looks something like * initial framework, but no instruction decode (so all instructions just throw an illegal instruction exception) * support for one or a few simple classes of instruction * gradually fill in the gaps of missing instructions This way you get the benefit of code review on your early patches, so you can write the later ones in line with the comments you get on the first patches. We also get to review the code in smaller and more manageable chunks. I think the last target code to go in was target-tricore; you could look back in the git history and the archives to get an idea of how to structure the patchset and the kinds of review comment that patchseries received. thanks -- PMM ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Qemu-trivial] [Qemu-devel] [PATCH] linux-user/syscall.c: Fix typo issue for using target_vec[i].iov_len instead of target_vec[i].iov_base 2015-01-23 11:21 ` Peter Maydell @ 2015-01-23 14:01 ` Chen Gang S -1 siblings, 0 replies; 16+ messages in thread From: Chen Gang S @ 2015-01-23 14:01 UTC (permalink / raw) To: Peter Maydell; +Cc: QEMU Trivial, Chris Metcalf, Riku Voipio, qemu-devel On 1/23/15 19:21, Peter Maydell wrote: > On 23 January 2015 at 10:57, Chen Gang S <gang.chen@sunrus.com.cn> wrote: >> >> Thank you for all of your work. >> >> I plan to let qemu support tile architecture: "can let gcc run testsuite >> with qemu for tile". It is really hard to me, but I should try, it is my >> duty (I met it, and it's within my boarder, although no one pay for me). >> :-) > >> Welcome any ideas, suggestions and completions by any members > > My single biggest suggestion is to make sure you split the work > up so that you can post parts of it as you go along. Typically > for a target-*/ new port that looks something like > * initial framework, but no instruction decode (so all instructions > just throw an illegal instruction exception) > * support for one or a few simple classes of instruction > * gradually fill in the gaps of missing instructions > > This way you get the benefit of code review on your early patches, > so you can write the later ones in line with the comments you > get on the first patches. We also get to review the code in > smaller and more manageable chunks. > > I think the last target code to go in was target-tricore; you > could look back in the git history and the archives to get an > idea of how to structure the patchset and the kinds of review > comment that patchseries received. > That sounds a good idea to me. In this way, I will bother qemu upstream frequently (which also means will get much help from qemu upstream for tile, frequently). If can get much help from qemu upstream frequently (in above way), I guess, it is possible to me to finish it: "let gcc run testsuit with tile qemu". :-) What I shall do for tile qemu should obey the related license of qemu (all the related code should belong to qemu upstream). If necessary to follow some copyright working flow, please let me know, I shall follow. Thanks. -- Chen Gang Open, share, and attitude like air, water, and life which God blessed ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Qemu-devel] [PATCH] linux-user/syscall.c: Fix typo issue for using target_vec[i].iov_len instead of target_vec[i].iov_base @ 2015-01-23 14:01 ` Chen Gang S 0 siblings, 0 replies; 16+ messages in thread From: Chen Gang S @ 2015-01-23 14:01 UTC (permalink / raw) To: Peter Maydell; +Cc: QEMU Trivial, Chris Metcalf, Riku Voipio, qemu-devel On 1/23/15 19:21, Peter Maydell wrote: > On 23 January 2015 at 10:57, Chen Gang S <gang.chen@sunrus.com.cn> wrote: >> >> Thank you for all of your work. >> >> I plan to let qemu support tile architecture: "can let gcc run testsuite >> with qemu for tile". It is really hard to me, but I should try, it is my >> duty (I met it, and it's within my boarder, although no one pay for me). >> :-) > >> Welcome any ideas, suggestions and completions by any members > > My single biggest suggestion is to make sure you split the work > up so that you can post parts of it as you go along. Typically > for a target-*/ new port that looks something like > * initial framework, but no instruction decode (so all instructions > just throw an illegal instruction exception) > * support for one or a few simple classes of instruction > * gradually fill in the gaps of missing instructions > > This way you get the benefit of code review on your early patches, > so you can write the later ones in line with the comments you > get on the first patches. We also get to review the code in > smaller and more manageable chunks. > > I think the last target code to go in was target-tricore; you > could look back in the git history and the archives to get an > idea of how to structure the patchset and the kinds of review > comment that patchseries received. > That sounds a good idea to me. In this way, I will bother qemu upstream frequently (which also means will get much help from qemu upstream for tile, frequently). If can get much help from qemu upstream frequently (in above way), I guess, it is possible to me to finish it: "let gcc run testsuit with tile qemu". :-) What I shall do for tile qemu should obey the related license of qemu (all the related code should belong to qemu upstream). If necessary to follow some copyright working flow, please let me know, I shall follow. Thanks. -- Chen Gang Open, share, and attitude like air, water, and life which God blessed ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Qemu-trivial] [Qemu-devel] [PATCH] linux-user/syscall.c: Fix typo issue for using target_vec[i].iov_len instead of target_vec[i].iov_base 2015-01-23 14:01 ` Chen Gang S @ 2015-01-23 13:55 ` Peter Maydell -1 siblings, 0 replies; 16+ messages in thread From: Peter Maydell @ 2015-01-23 13:55 UTC (permalink / raw) To: Chen Gang S; +Cc: QEMU Trivial, Chris Metcalf, Riku Voipio, qemu-devel On 23 January 2015 at 14:01, Chen Gang S <gang.chen@sunrus.com.cn> wrote: > What I shall do for tile qemu should obey the related license of qemu > (all the related code should belong to qemu upstream). If necessary to > follow some copyright working flow, please let me know, I shall follow. We don't have a copyright assignment process. I recommend reading http://wiki.qemu.org/Contribute/SubmitAPatch if you haven't already. thanks -- PMM ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Qemu-devel] [PATCH] linux-user/syscall.c: Fix typo issue for using target_vec[i].iov_len instead of target_vec[i].iov_base @ 2015-01-23 13:55 ` Peter Maydell 0 siblings, 0 replies; 16+ messages in thread From: Peter Maydell @ 2015-01-23 13:55 UTC (permalink / raw) To: Chen Gang S; +Cc: QEMU Trivial, Chris Metcalf, Riku Voipio, qemu-devel On 23 January 2015 at 14:01, Chen Gang S <gang.chen@sunrus.com.cn> wrote: > What I shall do for tile qemu should obey the related license of qemu > (all the related code should belong to qemu upstream). If necessary to > follow some copyright working flow, please let me know, I shall follow. We don't have a copyright assignment process. I recommend reading http://wiki.qemu.org/Contribute/SubmitAPatch if you haven't already. thanks -- PMM ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Qemu-trivial] [Qemu-devel] [PATCH] linux-user/syscall.c: Fix typo issue for using target_vec[i].iov_len instead of target_vec[i].iov_base 2015-01-23 13:55 ` Peter Maydell @ 2015-01-23 21:10 ` Chen Gang S -1 siblings, 0 replies; 16+ messages in thread From: Chen Gang S @ 2015-01-23 21:10 UTC (permalink / raw) To: Peter Maydell; +Cc: QEMU Trivial, Chris Metcalf, Riku Voipio, qemu-devel On 1/23/15 21:55, Peter Maydell wrote: > On 23 January 2015 at 14:01, Chen Gang S <gang.chen@sunrus.com.cn> wrote: >> What I shall do for tile qemu should obey the related license of qemu >> (all the related code should belong to qemu upstream). If necessary to >> follow some copyright working flow, please let me know, I shall follow. > > We don't have a copyright assignment process. I recommend > reading http://wiki.qemu.org/Contribute/SubmitAPatch if you haven't > already. > OK, thank. I assume (or declare): - The code what I write for qemu are all provided by myself (personal), not by any companies or any organizations (no one pay for it). - The documents which I have get are all free (e.g. tile architecture related pdf files), I can reference them freely for development. - If the related code what I have provided are integrated into qemu upstream, qemu upstream have the full right for it (follow with qemu upstream own copyright). I send the mail from my company email address, the reason is gmail is broken in China, my gmail can not be used, so I have to use my another email address in these days (but my gmail is still for Signed-of-by). Welcome any suggestions and completions by any members or organizations. Thanks. -- Chen Gang Open, share, and attitude like air, water, and life which God blessed ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Qemu-devel] [PATCH] linux-user/syscall.c: Fix typo issue for using target_vec[i].iov_len instead of target_vec[i].iov_base @ 2015-01-23 21:10 ` Chen Gang S 0 siblings, 0 replies; 16+ messages in thread From: Chen Gang S @ 2015-01-23 21:10 UTC (permalink / raw) To: Peter Maydell; +Cc: QEMU Trivial, Chris Metcalf, Riku Voipio, qemu-devel On 1/23/15 21:55, Peter Maydell wrote: > On 23 January 2015 at 14:01, Chen Gang S <gang.chen@sunrus.com.cn> wrote: >> What I shall do for tile qemu should obey the related license of qemu >> (all the related code should belong to qemu upstream). If necessary to >> follow some copyright working flow, please let me know, I shall follow. > > We don't have a copyright assignment process. I recommend > reading http://wiki.qemu.org/Contribute/SubmitAPatch if you haven't > already. > OK, thank. I assume (or declare): - The code what I write for qemu are all provided by myself (personal), not by any companies or any organizations (no one pay for it). - The documents which I have get are all free (e.g. tile architecture related pdf files), I can reference them freely for development. - If the related code what I have provided are integrated into qemu upstream, qemu upstream have the full right for it (follow with qemu upstream own copyright). I send the mail from my company email address, the reason is gmail is broken in China, my gmail can not be used, so I have to use my another email address in these days (but my gmail is still for Signed-of-by). Welcome any suggestions and completions by any members or organizations. Thanks. -- Chen Gang Open, share, and attitude like air, water, and life which God blessed ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Qemu-trivial] [PATCH] linux-user/syscall.c: Fix typo issue for using target_vec[i].iov_len instead of target_vec[i].iov_base 2015-01-23 10:07 ` [Qemu-devel] " Chen Gang S @ 2015-01-27 15:39 ` Michael Tokarev -1 siblings, 0 replies; 16+ messages in thread From: Michael Tokarev @ 2015-01-27 15:39 UTC (permalink / raw) To: Chen Gang S, riku.voipio; +Cc: QEMU Trivial, qemu-devel 23.01.2015 13:07, Chen Gang S wrote: > It is only a typo issue, need use tswapal(target_vec[i].iov_len) for the > len. My suggestion: Subject: linux-user/syscall.c: unlock_iovec: fix typo misuse of iov_base instead of iov_len without further explanation in the message body. Applied as-is to trivial, thank you! /mjt > Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com> > --- > linux-user/syscall.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/linux-user/syscall.c b/linux-user/syscall.c > index a66c2ae..8260ed7 100644 > --- a/linux-user/syscall.c > +++ b/linux-user/syscall.c > @@ -1896,7 +1896,7 @@ static void unlock_iovec(struct iovec *vec, abi_ulong target_addr, > if (target_vec) { > for (i = 0; i < count; i++) { > abi_ulong base = tswapal(target_vec[i].iov_base); > - abi_long len = tswapal(target_vec[i].iov_base); > + abi_long len = tswapal(target_vec[i].iov_len); > if (len < 0) { > break; > } > ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Qemu-devel] [PATCH] linux-user/syscall.c: Fix typo issue for using target_vec[i].iov_len instead of target_vec[i].iov_base @ 2015-01-27 15:39 ` Michael Tokarev 0 siblings, 0 replies; 16+ messages in thread From: Michael Tokarev @ 2015-01-27 15:39 UTC (permalink / raw) To: Chen Gang S, riku.voipio; +Cc: QEMU Trivial, qemu-devel 23.01.2015 13:07, Chen Gang S wrote: > It is only a typo issue, need use tswapal(target_vec[i].iov_len) for the > len. My suggestion: Subject: linux-user/syscall.c: unlock_iovec: fix typo misuse of iov_base instead of iov_len without further explanation in the message body. Applied as-is to trivial, thank you! /mjt > Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com> > --- > linux-user/syscall.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/linux-user/syscall.c b/linux-user/syscall.c > index a66c2ae..8260ed7 100644 > --- a/linux-user/syscall.c > +++ b/linux-user/syscall.c > @@ -1896,7 +1896,7 @@ static void unlock_iovec(struct iovec *vec, abi_ulong target_addr, > if (target_vec) { > for (i = 0; i < count; i++) { > abi_ulong base = tswapal(target_vec[i].iov_base); > - abi_long len = tswapal(target_vec[i].iov_base); > + abi_long len = tswapal(target_vec[i].iov_len); > if (len < 0) { > break; > } > ^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2015-01-27 15:39 UTC | newest] Thread overview: 16+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-01-23 10:07 [Qemu-trivial] [PATCH] linux-user/syscall.c: Fix typo issue for using target_vec[i].iov_len instead of target_vec[i].iov_base Chen Gang S 2015-01-23 10:07 ` [Qemu-devel] " Chen Gang S 2015-01-23 10:08 ` [Qemu-trivial] " Peter Maydell 2015-01-23 10:08 ` Peter Maydell 2015-01-23 10:57 ` [Qemu-trivial] " Chen Gang S 2015-01-23 10:57 ` Chen Gang S 2015-01-23 11:21 ` [Qemu-trivial] " Peter Maydell 2015-01-23 11:21 ` Peter Maydell 2015-01-23 14:01 ` [Qemu-trivial] " Chen Gang S 2015-01-23 14:01 ` Chen Gang S 2015-01-23 13:55 ` [Qemu-trivial] " Peter Maydell 2015-01-23 13:55 ` Peter Maydell 2015-01-23 21:10 ` [Qemu-trivial] " Chen Gang S 2015-01-23 21:10 ` Chen Gang S 2015-01-27 15:39 ` [Qemu-trivial] " Michael Tokarev 2015-01-27 15:39 ` [Qemu-devel] " Michael Tokarev
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.