* [Qemu-devel] [PATCH v2] slirp, disas: Replace min/max with MIN/MAX macros
@ 2016-11-29 15:07 Yuval Shaia
2016-12-13 11:56 ` Markus Armbruster
` (3 more replies)
0 siblings, 4 replies; 7+ messages in thread
From: Yuval Shaia @ 2016-11-29 15:07 UTC (permalink / raw)
To: laurent, samuel.thibault, jan.kiszka; +Cc: qemu-devel, yuval.shaia
Signed-off-by: Yuval Shaia <yuval.shaia@oracle.com>
---
v0 -> v1:
* Remove unndeeded "include" as suggested by Fam Zheng
* Change commit message's prefix as suggested by Markus Armbruster
* Utilize MIN macro in two extra places in file disas/m68k.c
v1 -> v2:
* Fix errros from checkpatch
---
disas/m68k.c | 8 ++------
slirp/dhcpv6.c | 2 +-
slirp/ip6_icmp.c | 2 +-
slirp/slirp.c | 2 +-
slirp/slirp.h | 5 -----
slirp/tcp_input.c | 16 ++++++++--------
slirp/tcp_output.c | 6 +++---
slirp/tcp_timer.c | 2 +-
slirp/tcpip.h | 2 +-
9 files changed, 18 insertions(+), 27 deletions(-)
diff --git a/disas/m68k.c b/disas/m68k.c
index 8e7c3f7..073abb9 100644
--- a/disas/m68k.c
+++ b/disas/m68k.c
@@ -4698,10 +4698,6 @@ get_field (const unsigned char *data, enum floatformat_byteorders order,
return result;
}
-#ifndef min
-#define min(a, b) ((a) < (b) ? (a) : (b))
-#endif
-
/* Convert from FMT to a double.
FROM is the address of the extended float.
Store the double in *TO. */
@@ -4733,7 +4729,7 @@ floatformat_to_double (const struct floatformat *fmt,
nan = 0;
while (mant_bits_left > 0)
{
- mant_bits = min (mant_bits_left, 32);
+ mant_bits = MIN(mant_bits_left, 32);
if (get_field (ufrom, fmt->byteorder, fmt->totalsize,
mant_off, mant_bits) != 0)
@@ -4793,7 +4789,7 @@ floatformat_to_double (const struct floatformat *fmt,
while (mant_bits_left > 0)
{
- mant_bits = min (mant_bits_left, 32);
+ mant_bits = MIN(mant_bits_left, 32);
mant = get_field (ufrom, fmt->byteorder, fmt->totalsize,
mant_off, mant_bits);
diff --git a/slirp/dhcpv6.c b/slirp/dhcpv6.c
index 02c51c7..d266611 100644
--- a/slirp/dhcpv6.c
+++ b/slirp/dhcpv6.c
@@ -168,7 +168,7 @@ static void dhcpv6_info_request(Slirp *slirp, struct sockaddr_in6 *srcsas,
sa[0], sa[1], sa[2], sa[3], sa[4], sa[5], sa[6], sa[7],
sa[8], sa[9], sa[10], sa[11], sa[12], sa[13], sa[14],
sa[15], slirp->bootp_filename);
- slen = min(slen, smaxlen);
+ slen = MIN(slen, smaxlen);
*resp++ = slen >> 8; /* option-len high byte */
*resp++ = slen; /* option-len low byte */
resp += slen;
diff --git a/slirp/ip6_icmp.c b/slirp/ip6_icmp.c
index 6d18e28..298a48d 100644
--- a/slirp/ip6_icmp.c
+++ b/slirp/ip6_icmp.c
@@ -95,7 +95,7 @@ void icmp6_send_error(struct mbuf *m, uint8_t type, uint8_t code)
#endif
rip->ip_nh = IPPROTO_ICMPV6;
- const int error_data_len = min(m->m_len,
+ const int error_data_len = MIN(m->m_len,
IF_MTU - (sizeof(struct ip6) + ICMP6_ERROR_MINLEN));
rip->ip_pl = htons(ICMP6_ERROR_MINLEN + error_data_len);
t->m_len = sizeof(struct ip6) + ntohs(rip->ip_pl);
diff --git a/slirp/slirp.c b/slirp/slirp.c
index 6e2b4e5..60539de 100644
--- a/slirp/slirp.c
+++ b/slirp/slirp.c
@@ -774,7 +774,7 @@ void slirp_pollfds_poll(GArray *pollfds, int select_error)
static void arp_input(Slirp *slirp, const uint8_t *pkt, int pkt_len)
{
struct slirp_arphdr *ah = (struct slirp_arphdr *)(pkt + ETH_HLEN);
- uint8_t arp_reply[max(ETH_HLEN + sizeof(struct slirp_arphdr), 64)];
+ uint8_t arp_reply[MAX(ETH_HLEN + sizeof(struct slirp_arphdr), 64)];
struct ethhdr *reh = (struct ethhdr *)arp_reply;
struct slirp_arphdr *rah = (struct slirp_arphdr *)(arp_reply + ETH_HLEN);
int ar_op;
diff --git a/slirp/slirp.h b/slirp/slirp.h
index a1f3139..3877f66 100644
--- a/slirp/slirp.h
+++ b/slirp/slirp.h
@@ -292,9 +292,4 @@ int tcp_emu(struct socket *, struct mbuf *);
int tcp_ctl(struct socket *);
struct tcpcb *tcp_drop(struct tcpcb *tp, int err);
-#ifndef _WIN32
-#define min(x,y) ((x) < (y) ? (x) : (y))
-#define max(x,y) ((x) > (y) ? (x) : (y))
-#endif
-
#endif
diff --git a/slirp/tcp_input.c b/slirp/tcp_input.c
index c5063a9..edb98f0 100644
--- a/slirp/tcp_input.c
+++ b/slirp/tcp_input.c
@@ -596,7 +596,7 @@ findso:
win = sbspace(&so->so_rcv);
if (win < 0)
win = 0;
- tp->rcv_wnd = max(win, (int)(tp->rcv_adv - tp->rcv_nxt));
+ tp->rcv_wnd = MAX(win, (int)(tp->rcv_adv - tp->rcv_nxt));
}
switch (tp->t_state) {
@@ -1065,8 +1065,8 @@ trimthenstep6:
else if (++tp->t_dupacks == TCPREXMTTHRESH) {
tcp_seq onxt = tp->snd_nxt;
u_int win =
- min(tp->snd_wnd, tp->snd_cwnd) / 2 /
- tp->t_maxseg;
+ MIN(tp->snd_wnd, tp->snd_cwnd) /
+ 2 / tp->t_maxseg;
if (win < 2)
win = 2;
@@ -1138,7 +1138,7 @@ trimthenstep6:
if (cw > tp->snd_ssthresh)
incr = incr * incr / cw;
- tp->snd_cwnd = min(cw + incr, TCP_MAXWIN<<tp->snd_scale);
+ tp->snd_cwnd = MIN(cw + incr, TCP_MAXWIN << tp->snd_scale);
}
if (acked > so->so_snd.sb_cc) {
tp->snd_wnd -= so->so_snd.sb_cc;
@@ -1586,11 +1586,11 @@ tcp_mss(struct tcpcb *tp, u_int offer)
switch (so->so_ffamily) {
case AF_INET:
- mss = min(IF_MTU, IF_MRU) - sizeof(struct tcphdr)
+ mss = MIN(IF_MTU, IF_MRU) - sizeof(struct tcphdr)
+ sizeof(struct ip);
break;
case AF_INET6:
- mss = min(IF_MTU, IF_MRU) - sizeof(struct tcphdr)
+ mss = MIN(IF_MTU, IF_MRU) - sizeof(struct tcphdr)
+ sizeof(struct ip6);
break;
default:
@@ -1598,8 +1598,8 @@ tcp_mss(struct tcpcb *tp, u_int offer)
}
if (offer)
- mss = min(mss, offer);
- mss = max(mss, 32);
+ mss = MIN(mss, offer);
+ mss = MAX(mss, 32);
if (mss < tp->t_maxseg || offer != 0)
tp->t_maxseg = mss;
diff --git a/slirp/tcp_output.c b/slirp/tcp_output.c
index 819db27..90b5c37 100644
--- a/slirp/tcp_output.c
+++ b/slirp/tcp_output.c
@@ -88,7 +88,7 @@ tcp_output(struct tcpcb *tp)
again:
sendalot = 0;
off = tp->snd_nxt - tp->snd_una;
- win = min(tp->snd_wnd, tp->snd_cwnd);
+ win = MIN(tp->snd_wnd, tp->snd_cwnd);
flags = tcp_outflags[tp->t_state];
@@ -127,7 +127,7 @@ again:
}
}
- len = min(so->so_snd.sb_cc, win) - off;
+ len = MIN(so->so_snd.sb_cc, win) - off;
if (len < 0) {
/*
@@ -193,7 +193,7 @@ again:
* taking into account that we are limited by
* TCP_MAXWIN << tp->rcv_scale.
*/
- long adv = min(win, (long)TCP_MAXWIN << tp->rcv_scale) -
+ long adv = MIN(win, (long)TCP_MAXWIN << tp->rcv_scale) -
(tp->rcv_adv - tp->rcv_nxt);
if (adv >= (long) (2 * tp->t_maxseg))
diff --git a/slirp/tcp_timer.c b/slirp/tcp_timer.c
index f9060c7..52ef5f9 100644
--- a/slirp/tcp_timer.c
+++ b/slirp/tcp_timer.c
@@ -233,7 +233,7 @@ tcp_timers(register struct tcpcb *tp, int timer)
* to go below this.)
*/
{
- u_int win = min(tp->snd_wnd, tp->snd_cwnd) / 2 / tp->t_maxseg;
+ u_int win = MIN(tp->snd_wnd, tp->snd_cwnd) / 2 / tp->t_maxseg;
if (win < 2)
win = 2;
tp->snd_cwnd = tp->t_maxseg;
diff --git a/slirp/tcpip.h b/slirp/tcpip.h
index 7bdb971..07dbf2c 100644
--- a/slirp/tcpip.h
+++ b/slirp/tcpip.h
@@ -85,7 +85,7 @@ struct tcpiphdr {
/* This is the difference between the size of a tcpiphdr structure, and the
* size of actual ip+tcp headers, rounded up since we need to align data. */
#define TCPIPHDR_DELTA\
- (max(0,\
+ (MAX(0,\
(sizeof(struct tcpiphdr)\
- sizeof(struct ip) - sizeof(struct tcphdr) + 3) & ~3))
--
1.8.3.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [PATCH v2] slirp, disas: Replace min/max with MIN/MAX macros
2016-11-29 15:07 [Qemu-devel] [PATCH v2] slirp, disas: Replace min/max with MIN/MAX macros Yuval Shaia
@ 2016-12-13 11:56 ` Markus Armbruster
2016-12-13 12:33 ` Laurent Vivier
` (2 subsequent siblings)
3 siblings, 0 replies; 7+ messages in thread
From: Markus Armbruster @ 2016-12-13 11:56 UTC (permalink / raw)
To: Yuval Shaia; +Cc: laurent, samuel.thibault, jan.kiszka, qemu-devel
Yuval Shaia <yuval.shaia@oracle.com> writes:
> Signed-off-by: Yuval Shaia <yuval.shaia@oracle.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [PATCH v2] slirp, disas: Replace min/max with MIN/MAX macros
2016-11-29 15:07 [Qemu-devel] [PATCH v2] slirp, disas: Replace min/max with MIN/MAX macros Yuval Shaia
2016-12-13 11:56 ` Markus Armbruster
@ 2016-12-13 12:33 ` Laurent Vivier
2016-12-13 12:57 ` Stefan Weil
2016-12-13 15:57 ` Laurent Vivier
2016-12-20 22:53 ` Samuel Thibault
3 siblings, 1 reply; 7+ messages in thread
From: Laurent Vivier @ 2016-12-13 12:33 UTC (permalink / raw)
To: Yuval Shaia, samuel.thibault, jan.kiszka; +Cc: qemu-devel, Stefan Weil
Le 29/11/2016 à 16:07, Yuval Shaia a écrit :
> diff --git a/slirp/slirp.h b/slirp/slirp.h
> index a1f3139..3877f66 100644
> --- a/slirp/slirp.h
> +++ b/slirp/slirp.h
> @@ -292,9 +292,4 @@ int tcp_emu(struct socket *, struct mbuf *);
> int tcp_ctl(struct socket *);
> struct tcpcb *tcp_drop(struct tcpcb *tp, int err);
>
> -#ifndef _WIN32
> -#define min(x,y) ((x) < (y) ? (x) : (y))
> -#define max(x,y) ((x) > (y) ? (x) : (y))
> -#endif
> -
It seems the defines have been added because they are already defined
for Win32.
If we remove them, do we know if MIN()/MAX() are defined for Win32?
[CC: host W32 maintainer]
Thanks,
Laurent
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [PATCH v2] slirp, disas: Replace min/max with MIN/MAX macros
2016-12-13 12:33 ` Laurent Vivier
@ 2016-12-13 12:57 ` Stefan Weil
2016-12-13 19:16 ` Markus Armbruster
0 siblings, 1 reply; 7+ messages in thread
From: Stefan Weil @ 2016-12-13 12:57 UTC (permalink / raw)
To: Laurent Vivier, Yuval Shaia, samuel.thibault, jan.kiszka; +Cc: qemu-devel
Am 13.12.2016 um 13:33 schrieb Laurent Vivier:
> Le 29/11/2016 à 16:07, Yuval Shaia a écrit :
>> diff --git a/slirp/slirp.h b/slirp/slirp.h
>> index a1f3139..3877f66 100644
>> --- a/slirp/slirp.h
>> +++ b/slirp/slirp.h
>> @@ -292,9 +292,4 @@ int tcp_emu(struct socket *, struct mbuf *);
>> int tcp_ctl(struct socket *);
>> struct tcpcb *tcp_drop(struct tcpcb *tp, int err);
>>
>> -#ifndef _WIN32
>> -#define min(x,y) ((x) < (y) ? (x) : (y))
>> -#define max(x,y) ((x) > (y) ? (x) : (y))
>> -#endif
>> -
> It seems the defines have been added because they are already defined
> for Win32.
>
> If we remove them, do we know if MIN()/MAX() are defined for Win32?
> [CC: host W32 maintainer]
>
> Thanks,
> Laurent
Neither MIN nor MAX are defined in the header files provided with Mingw-w64.
Cheers,
Stefan
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [PATCH v2] slirp, disas: Replace min/max with MIN/MAX macros
2016-11-29 15:07 [Qemu-devel] [PATCH v2] slirp, disas: Replace min/max with MIN/MAX macros Yuval Shaia
2016-12-13 11:56 ` Markus Armbruster
2016-12-13 12:33 ` Laurent Vivier
@ 2016-12-13 15:57 ` Laurent Vivier
2016-12-20 22:53 ` Samuel Thibault
3 siblings, 0 replies; 7+ messages in thread
From: Laurent Vivier @ 2016-12-13 15:57 UTC (permalink / raw)
To: Yuval Shaia, laurent, samuel.thibault, jan.kiszka; +Cc: qemu-devel
On 29/11/2016 16:07, Yuval Shaia wrote:
> Signed-off-by: Yuval Shaia <yuval.shaia@oracle.com>
Reviewed-by: Laurent Vivier <lvivier@redhat.com>
> ---
> v0 -> v1:
> * Remove unndeeded "include" as suggested by Fam Zheng
> * Change commit message's prefix as suggested by Markus Armbruster
> * Utilize MIN macro in two extra places in file disas/m68k.c
> v1 -> v2:
> * Fix errros from checkpatch
> ---
> disas/m68k.c | 8 ++------
> slirp/dhcpv6.c | 2 +-
> slirp/ip6_icmp.c | 2 +-
> slirp/slirp.c | 2 +-
> slirp/slirp.h | 5 -----
> slirp/tcp_input.c | 16 ++++++++--------
> slirp/tcp_output.c | 6 +++---
> slirp/tcp_timer.c | 2 +-
> slirp/tcpip.h | 2 +-
> 9 files changed, 18 insertions(+), 27 deletions(-)
>
> diff --git a/disas/m68k.c b/disas/m68k.c
> index 8e7c3f7..073abb9 100644
> --- a/disas/m68k.c
> +++ b/disas/m68k.c
> @@ -4698,10 +4698,6 @@ get_field (const unsigned char *data, enum floatformat_byteorders order,
> return result;
> }
>
> -#ifndef min
> -#define min(a, b) ((a) < (b) ? (a) : (b))
> -#endif
> -
> /* Convert from FMT to a double.
> FROM is the address of the extended float.
> Store the double in *TO. */
> @@ -4733,7 +4729,7 @@ floatformat_to_double (const struct floatformat *fmt,
> nan = 0;
> while (mant_bits_left > 0)
> {
> - mant_bits = min (mant_bits_left, 32);
> + mant_bits = MIN(mant_bits_left, 32);
>
> if (get_field (ufrom, fmt->byteorder, fmt->totalsize,
> mant_off, mant_bits) != 0)
> @@ -4793,7 +4789,7 @@ floatformat_to_double (const struct floatformat *fmt,
>
> while (mant_bits_left > 0)
> {
> - mant_bits = min (mant_bits_left, 32);
> + mant_bits = MIN(mant_bits_left, 32);
>
> mant = get_field (ufrom, fmt->byteorder, fmt->totalsize,
> mant_off, mant_bits);
> diff --git a/slirp/dhcpv6.c b/slirp/dhcpv6.c
> index 02c51c7..d266611 100644
> --- a/slirp/dhcpv6.c
> +++ b/slirp/dhcpv6.c
> @@ -168,7 +168,7 @@ static void dhcpv6_info_request(Slirp *slirp, struct sockaddr_in6 *srcsas,
> sa[0], sa[1], sa[2], sa[3], sa[4], sa[5], sa[6], sa[7],
> sa[8], sa[9], sa[10], sa[11], sa[12], sa[13], sa[14],
> sa[15], slirp->bootp_filename);
> - slen = min(slen, smaxlen);
> + slen = MIN(slen, smaxlen);
> *resp++ = slen >> 8; /* option-len high byte */
> *resp++ = slen; /* option-len low byte */
> resp += slen;
> diff --git a/slirp/ip6_icmp.c b/slirp/ip6_icmp.c
> index 6d18e28..298a48d 100644
> --- a/slirp/ip6_icmp.c
> +++ b/slirp/ip6_icmp.c
> @@ -95,7 +95,7 @@ void icmp6_send_error(struct mbuf *m, uint8_t type, uint8_t code)
> #endif
>
> rip->ip_nh = IPPROTO_ICMPV6;
> - const int error_data_len = min(m->m_len,
> + const int error_data_len = MIN(m->m_len,
> IF_MTU - (sizeof(struct ip6) + ICMP6_ERROR_MINLEN));
> rip->ip_pl = htons(ICMP6_ERROR_MINLEN + error_data_len);
> t->m_len = sizeof(struct ip6) + ntohs(rip->ip_pl);
> diff --git a/slirp/slirp.c b/slirp/slirp.c
> index 6e2b4e5..60539de 100644
> --- a/slirp/slirp.c
> +++ b/slirp/slirp.c
> @@ -774,7 +774,7 @@ void slirp_pollfds_poll(GArray *pollfds, int select_error)
> static void arp_input(Slirp *slirp, const uint8_t *pkt, int pkt_len)
> {
> struct slirp_arphdr *ah = (struct slirp_arphdr *)(pkt + ETH_HLEN);
> - uint8_t arp_reply[max(ETH_HLEN + sizeof(struct slirp_arphdr), 64)];
> + uint8_t arp_reply[MAX(ETH_HLEN + sizeof(struct slirp_arphdr), 64)];
> struct ethhdr *reh = (struct ethhdr *)arp_reply;
> struct slirp_arphdr *rah = (struct slirp_arphdr *)(arp_reply + ETH_HLEN);
> int ar_op;
> diff --git a/slirp/slirp.h b/slirp/slirp.h
> index a1f3139..3877f66 100644
> --- a/slirp/slirp.h
> +++ b/slirp/slirp.h
> @@ -292,9 +292,4 @@ int tcp_emu(struct socket *, struct mbuf *);
> int tcp_ctl(struct socket *);
> struct tcpcb *tcp_drop(struct tcpcb *tp, int err);
>
> -#ifndef _WIN32
> -#define min(x,y) ((x) < (y) ? (x) : (y))
> -#define max(x,y) ((x) > (y) ? (x) : (y))
> -#endif
> -
> #endif
> diff --git a/slirp/tcp_input.c b/slirp/tcp_input.c
> index c5063a9..edb98f0 100644
> --- a/slirp/tcp_input.c
> +++ b/slirp/tcp_input.c
> @@ -596,7 +596,7 @@ findso:
> win = sbspace(&so->so_rcv);
> if (win < 0)
> win = 0;
> - tp->rcv_wnd = max(win, (int)(tp->rcv_adv - tp->rcv_nxt));
> + tp->rcv_wnd = MAX(win, (int)(tp->rcv_adv - tp->rcv_nxt));
> }
>
> switch (tp->t_state) {
> @@ -1065,8 +1065,8 @@ trimthenstep6:
> else if (++tp->t_dupacks == TCPREXMTTHRESH) {
> tcp_seq onxt = tp->snd_nxt;
> u_int win =
> - min(tp->snd_wnd, tp->snd_cwnd) / 2 /
> - tp->t_maxseg;
> + MIN(tp->snd_wnd, tp->snd_cwnd) /
> + 2 / tp->t_maxseg;
>
> if (win < 2)
> win = 2;
> @@ -1138,7 +1138,7 @@ trimthenstep6:
>
> if (cw > tp->snd_ssthresh)
> incr = incr * incr / cw;
> - tp->snd_cwnd = min(cw + incr, TCP_MAXWIN<<tp->snd_scale);
> + tp->snd_cwnd = MIN(cw + incr, TCP_MAXWIN << tp->snd_scale);
> }
> if (acked > so->so_snd.sb_cc) {
> tp->snd_wnd -= so->so_snd.sb_cc;
> @@ -1586,11 +1586,11 @@ tcp_mss(struct tcpcb *tp, u_int offer)
>
> switch (so->so_ffamily) {
> case AF_INET:
> - mss = min(IF_MTU, IF_MRU) - sizeof(struct tcphdr)
> + mss = MIN(IF_MTU, IF_MRU) - sizeof(struct tcphdr)
> + sizeof(struct ip);
> break;
> case AF_INET6:
> - mss = min(IF_MTU, IF_MRU) - sizeof(struct tcphdr)
> + mss = MIN(IF_MTU, IF_MRU) - sizeof(struct tcphdr)
> + sizeof(struct ip6);
> break;
> default:
> @@ -1598,8 +1598,8 @@ tcp_mss(struct tcpcb *tp, u_int offer)
> }
>
> if (offer)
> - mss = min(mss, offer);
> - mss = max(mss, 32);
> + mss = MIN(mss, offer);
> + mss = MAX(mss, 32);
> if (mss < tp->t_maxseg || offer != 0)
> tp->t_maxseg = mss;
>
> diff --git a/slirp/tcp_output.c b/slirp/tcp_output.c
> index 819db27..90b5c37 100644
> --- a/slirp/tcp_output.c
> +++ b/slirp/tcp_output.c
> @@ -88,7 +88,7 @@ tcp_output(struct tcpcb *tp)
> again:
> sendalot = 0;
> off = tp->snd_nxt - tp->snd_una;
> - win = min(tp->snd_wnd, tp->snd_cwnd);
> + win = MIN(tp->snd_wnd, tp->snd_cwnd);
>
> flags = tcp_outflags[tp->t_state];
>
> @@ -127,7 +127,7 @@ again:
> }
> }
>
> - len = min(so->so_snd.sb_cc, win) - off;
> + len = MIN(so->so_snd.sb_cc, win) - off;
>
> if (len < 0) {
> /*
> @@ -193,7 +193,7 @@ again:
> * taking into account that we are limited by
> * TCP_MAXWIN << tp->rcv_scale.
> */
> - long adv = min(win, (long)TCP_MAXWIN << tp->rcv_scale) -
> + long adv = MIN(win, (long)TCP_MAXWIN << tp->rcv_scale) -
> (tp->rcv_adv - tp->rcv_nxt);
>
> if (adv >= (long) (2 * tp->t_maxseg))
> diff --git a/slirp/tcp_timer.c b/slirp/tcp_timer.c
> index f9060c7..52ef5f9 100644
> --- a/slirp/tcp_timer.c
> +++ b/slirp/tcp_timer.c
> @@ -233,7 +233,7 @@ tcp_timers(register struct tcpcb *tp, int timer)
> * to go below this.)
> */
> {
> - u_int win = min(tp->snd_wnd, tp->snd_cwnd) / 2 / tp->t_maxseg;
> + u_int win = MIN(tp->snd_wnd, tp->snd_cwnd) / 2 / tp->t_maxseg;
> if (win < 2)
> win = 2;
> tp->snd_cwnd = tp->t_maxseg;
> diff --git a/slirp/tcpip.h b/slirp/tcpip.h
> index 7bdb971..07dbf2c 100644
> --- a/slirp/tcpip.h
> +++ b/slirp/tcpip.h
> @@ -85,7 +85,7 @@ struct tcpiphdr {
> /* This is the difference between the size of a tcpiphdr structure, and the
> * size of actual ip+tcp headers, rounded up since we need to align data. */
> #define TCPIPHDR_DELTA\
> - (max(0,\
> + (MAX(0,\
> (sizeof(struct tcpiphdr)\
> - sizeof(struct ip) - sizeof(struct tcphdr) + 3) & ~3))
>
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [PATCH v2] slirp, disas: Replace min/max with MIN/MAX macros
2016-12-13 12:57 ` Stefan Weil
@ 2016-12-13 19:16 ` Markus Armbruster
0 siblings, 0 replies; 7+ messages in thread
From: Markus Armbruster @ 2016-12-13 19:16 UTC (permalink / raw)
To: Stefan Weil
Cc: Laurent Vivier, Yuval Shaia, samuel.thibault, jan.kiszka, qemu-devel
Stefan Weil <sw@weilnetz.de> writes:
> Am 13.12.2016 um 13:33 schrieb Laurent Vivier:
>> Le 29/11/2016 à 16:07, Yuval Shaia a écrit :
>>> diff --git a/slirp/slirp.h b/slirp/slirp.h
>>> index a1f3139..3877f66 100644
>>> --- a/slirp/slirp.h
>>> +++ b/slirp/slirp.h
>>> @@ -292,9 +292,4 @@ int tcp_emu(struct socket *, struct mbuf *);
>>> int tcp_ctl(struct socket *);
>>> struct tcpcb *tcp_drop(struct tcpcb *tp, int err);
>>>
>>> -#ifndef _WIN32
>>> -#define min(x,y) ((x) < (y) ? (x) : (y))
>>> -#define max(x,y) ((x) > (y) ? (x) : (y))
>>> -#endif
>>> -
>> It seems the defines have been added because they are already defined
>> for Win32.
>>
>> If we remove them, do we know if MIN()/MAX() are defined for Win32?
>> [CC: host W32 maintainer]
>>
>> Thanks,
>> Laurent
>
> Neither MIN nor MAX are defined in the header files provided with Mingw-w64.
osdep.h provides them, and it's by convention included first by every
.c.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [PATCH v2] slirp, disas: Replace min/max with MIN/MAX macros
2016-11-29 15:07 [Qemu-devel] [PATCH v2] slirp, disas: Replace min/max with MIN/MAX macros Yuval Shaia
` (2 preceding siblings ...)
2016-12-13 15:57 ` Laurent Vivier
@ 2016-12-20 22:53 ` Samuel Thibault
3 siblings, 0 replies; 7+ messages in thread
From: Samuel Thibault @ 2016-12-20 22:53 UTC (permalink / raw)
To: Yuval Shaia; +Cc: laurent, jan.kiszka, qemu-devel
I have applied it to my tree, thanks!
Samuel
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2016-12-20 22:53 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-11-29 15:07 [Qemu-devel] [PATCH v2] slirp, disas: Replace min/max with MIN/MAX macros Yuval Shaia
2016-12-13 11:56 ` Markus Armbruster
2016-12-13 12:33 ` Laurent Vivier
2016-12-13 12:57 ` Stefan Weil
2016-12-13 19:16 ` Markus Armbruster
2016-12-13 15:57 ` Laurent Vivier
2016-12-20 22:53 ` Samuel Thibault
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.