Re: [PATCHv2 4/8] Fix name schema for tx-rx sock struct

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

 



Hi Leandro,

can you please re-send the patches from before Christmas and try a new
clone first? I have just switched the git-server back on again, today is
the first day we have electricity back. That is,

    git://eden-feed.erg.abdn.ac.uk/dccp_exp	{dccp,ccid4}

is up again; the CCID4 patches are exactly as before Christmas.

Gerrit

Quoting Leandro:
| Hi Gerrit,
|    the patches was applied against:
| 
| # git-describe
| v2.6.23-rc6-10457-g5ff93cc
| 
| SHA1 ID: 5ff93cc505b6ced6aa6d4c65df6d9beb6ae10fac
| 
| I cloned it at 2007-12-20, 18:19:49 (GMT -3).
| 
| Leandro.
| 
| 2007/12/21, Gerrit Renker <gerrit@xxxxxxxxxxxxxx>:
| > And your naming scheme changed from version 1 to version 2:
| >
| > This is patch v2:
| > |       switch (optname) {
| > |       case DCCP_SOCKOPT_CCID_TX_INFO:
| > | -             if (len < sizeof(hctx->tfrchctx_tfrc))
| > | +             if (len < sizeof(hctx->ttx_tfrc))
| > |                       return -EINVAL;
| > | -             len = sizeof(hctx->tfrchctx_tfrc);
| > | -             val = &hctx->tfrchctx_tfrc;
| > | +             len = sizeof(hctx->ttx_tfrc);
| > | +             val = &hctx->ttx_tfrc;
| > |               break;
| >
| > ... and this is patch v1:
| >
| > |       switch (optname) {
| > |       case DCCP_SOCKOPT_CCID_TX_INFO:
| > | -             if (len < sizeof(hctx->tfrchctx_tfrc))
| > | +             if (len < sizeof(hctx->tfrc_tx))
| > |                       return -EINVAL;
| > | -             len = sizeof(hctx->tfrchctx_tfrc);
| > | -             val = &hctx->tfrchctx_tfrc;
| > | +             len = sizeof(hctx->tfrc_tx);
| > | +             val = &hctx->tfrc_tx;
| > |               break;
| > 'tfrc_tx' becomes 'ttx_tfrc'. I think that is all needs more work and in
| > particular discussion with Arnaldo.
| >
| > With the given inconsistencies I can't put the set into the test tree.
| > Can you please resolve the naming scheme inconsistencies and the version
| > problems and resubmit a fresh patch set after discussing with Arnaldo.
| >
| 
| 
| -- 
| Leandro Melo de Sales.
| Pervasive and Embedded Computing Laboratory
| BRisa and E-Phone Projects Manager
| Network Admin @ http://embedded.ufcg.edu.br/indexen.html
| +55 83 3310-1404 (extension 208)
| 
| "There are 10 types of people in this world: those who understand
| binary, those who don't"
| 

-- 
-
To unsubscribe from this list: send the line "unsubscribe dccp" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel]     [IETF DCCP]     [Linux Networking]     [Git]     [Security]     [Linux Assembly]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]

  Powered by Linux