/[LeafOK_CVS]/lbbs/src/section_list.c
ViewVC logotype

Diff of /lbbs/src/section_list.c

Parent Directory Parent Directory | Revision Log Revision Log | View Patch Patch

Revision 1.22 by sysadm, Sun May 25 23:47:36 2025 UTC Revision 1.42 by sysadm, Tue Oct 14 02:39:34 2025 UTC
# Line 14  Line 14 
14   *                                                                         *   *                                                                         *
15   ***************************************************************************/   ***************************************************************************/
16    
 #define _GNU_SOURCE  
   
 #include "section_list.h"  
17  #include "log.h"  #include "log.h"
18    #include "section_list.h"
19  #include "trie_dict.h"  #include "trie_dict.h"
20    #include <errno.h>
21    #include <signal.h>
22  #include <stdio.h>  #include <stdio.h>
23    #include <stdlib.h>
24  #include <string.h>  #include <string.h>
 #include <signal.h>  
25  #include <unistd.h>  #include <unistd.h>
26  #include <stdlib.h>  #include <sys/ipc.h>
 #include <errno.h>  
27  #include <sys/param.h>  #include <sys/param.h>
28  #include <sys/sem.h>  #include <sys/sem.h>
29  #include <sys/shm.h>  #include <sys/shm.h>
 #include <sys/ipc.h>  
30    
31  #ifdef _SEM_SEMUN_UNDEFINED  #ifdef _SEM_SEMUN_UNDEFINED
32  union semun  union semun
# Line 41  union semun Line 39  union semun
39  };  };
40  #endif // #ifdef _SEM_SEMUN_UNDEFINED  #endif // #ifdef _SEM_SEMUN_UNDEFINED
41    
42  #define ARTICLE_BLOCK_PER_SHM 400                 // sizeof(ARTICLE_BLOCK) * ARTICLE_BLOCK_PER_SHM is the size of each shm segment to allocate  #define SECTION_TRY_LOCK_WAIT_TIME 1 // second
43  #define ARTICLE_BLOCK_SHM_COUNT_LIMIT 200 // limited by length (8-bit) of proj_id in ftok(path, proj_id)  #define SECTION_TRY_LOCK_TIMES 10
44    
45    #define ARTICLE_BLOCK_PER_SHM 1000               // sizeof(ARTICLE_BLOCK) * ARTICLE_BLOCK_PER_SHM is the size of each shm segment to allocate
46    #define ARTICLE_BLOCK_SHM_COUNT_LIMIT 80 // limited by length (8-bit) of proj_id in ftok(path, proj_id)
47  #define ARTICLE_BLOCK_PER_POOL (ARTICLE_BLOCK_PER_SHM * ARTICLE_BLOCK_SHM_COUNT_LIMIT)  #define ARTICLE_BLOCK_PER_POOL (ARTICLE_BLOCK_PER_SHM * ARTICLE_BLOCK_SHM_COUNT_LIMIT)
48    
49  #define CALCULATE_PAGE_THRESHOLD 100 // Adjust to tune performance of move topic  #define CALCULATE_PAGE_THRESHOLD 100 // Adjust to tune performance of moving topic between sections
50    
51  #define SID_STR_LEN 5 // 32-bit + NULL  #define SID_STR_LEN 5 // 32-bit + NULL
52    
# Line 74  typedef struct article_block_pool_t ARTI Line 75  typedef struct article_block_pool_t ARTI
75    
76  static ARTICLE_BLOCK_POOL *p_article_block_pool = NULL;  static ARTICLE_BLOCK_POOL *p_article_block_pool = NULL;
77    
78  struct section_list_pool_t  SECTION_LIST_POOL *p_section_list_pool = NULL;
 {  
         int shmid;  
         SECTION_LIST sections[BBS_max_section];  
         int section_count;  
         int semid;  
         TRIE_NODE *p_trie_dict_section_by_name;  
         TRIE_NODE *p_trie_dict_section_by_sid;  
 };  
 typedef struct section_list_pool_t SECTION_LIST_POOL;  
   
 static SECTION_LIST_POOL *p_section_list_pool = NULL;  
79    
80  int article_block_init(const char *filename, int block_count)  int article_block_init(const char *filename, int block_count)
81  {  {
# Line 105  int article_block_init(const char *filen Line 95  int article_block_init(const char *filen
95                  return -1;                  return -1;
96          }          }
97    
98          if (block_count > ARTICLE_BLOCK_PER_POOL)          if (block_count <= 0 || block_count > ARTICLE_BLOCK_PER_POOL)
99          {          {
100                  log_error("article_block_count exceed limit %d\n", ARTICLE_BLOCK_PER_POOL);                  log_error("article_block_count exceed limit %d\n", ARTICLE_BLOCK_PER_POOL);
101                  return -2;                  return -2;
# Line 233  void article_block_cleanup(void) Line 223  void article_block_cleanup(void)
223          p_article_block_pool = NULL;          p_article_block_pool = NULL;
224  }  }
225    
226    int set_article_block_shm_readonly(void)
227    {
228            int shmid;
229            void *p_shm;
230            int i;
231    
232            if (p_article_block_pool == NULL)
233            {
234                    log_error("article_block_pool not initialized\n");
235                    return -1;
236            }
237    
238            for (i = 0; i < p_article_block_pool->shm_count; i++)
239            {
240                    shmid = (p_article_block_pool->shm_pool + i)->shmid;
241    
242                    // Remap shared memory in read-only mode
243                    p_shm = shmat(shmid, (p_article_block_pool->shm_pool + i)->p_shm, SHM_RDONLY | SHM_REMAP);
244                    if (p_shm == (void *)-1)
245                    {
246                            log_error("shmat(article_block_pool shmid = %d) error (%d)\n", shmid, errno);
247                            return -2;
248                    }
249            }
250    
251            return 0;
252    }
253    
254    int detach_article_block_shm(void)
255    {
256            int shmid;
257    
258            if (p_article_block_pool == NULL)
259            {
260                    return -1;
261            }
262    
263            for (int i = 0; i < p_article_block_pool->shm_count; i++)
264            {
265                    if ((p_article_block_pool->shm_pool + i)->p_shm != NULL && shmdt((p_article_block_pool->shm_pool + i)->p_shm) == -1)
266                    {
267                            log_error("shmdt(shmid = %d) error (%d)\n", (p_article_block_pool->shm_pool + i)->shmid, errno);
268                            return -2;
269                    }
270            }
271    
272            shmid = p_article_block_pool->shmid;
273    
274            if (shmdt(p_article_block_pool) == -1)
275            {
276                    log_error("shmdt(shmid = %d) error (%d)\n", shmid, errno);
277                    return -3;
278            }
279    
280            p_article_block_pool = NULL;
281    
282            return 0;
283    }
284    
285  inline static ARTICLE_BLOCK *pop_free_article_block(void)  inline static ARTICLE_BLOCK *pop_free_article_block(void)
286  {  {
287          ARTICLE_BLOCK *p_block = NULL;          ARTICLE_BLOCK *p_block = NULL;
# Line 337  ARTICLE *article_block_find_by_aid(int32 Line 386  ARTICLE *article_block_find_by_aid(int32
386                  }                  }
387          }          }
388    
389          return (p_block->articles + left);          if (aid != p_block->articles[left].aid) // not found
390            {
391                    return NULL;
392            }
393    
394            return (p_block->articles + left); // found
395  }  }
396    
397  ARTICLE *article_block_find_by_index(int index)  ARTICLE *article_block_find_by_index(int index)
# Line 449  extern int section_list_init(const char Line 503  extern int section_list_init(const char
503          return 0;          return 0;
504  }  }
505    
506    void section_list_cleanup(void)
507    {
508            int shmid;
509    
510            if (p_section_list_pool == NULL)
511            {
512                    return;
513            }
514    
515            if (p_section_list_pool->p_trie_dict_section_by_name != NULL)
516            {
517                    trie_dict_destroy(p_section_list_pool->p_trie_dict_section_by_name);
518                    p_section_list_pool->p_trie_dict_section_by_name = NULL;
519            }
520    
521            if (p_section_list_pool->p_trie_dict_section_by_sid != NULL)
522            {
523                    trie_dict_destroy(p_section_list_pool->p_trie_dict_section_by_sid);
524                    p_section_list_pool->p_trie_dict_section_by_sid = NULL;
525            }
526    
527            shmid = p_section_list_pool->shmid;
528    
529            if (semctl(p_section_list_pool->semid, 0, IPC_RMID) == -1)
530            {
531                    log_error("semctl(semid = %d, IPC_RMID) error (%d)\n", p_section_list_pool->semid, errno);
532            }
533    
534            if (shmdt(p_section_list_pool) == -1)
535            {
536                    log_error("shmdt(shmid = %d) error (%d)\n", shmid, errno);
537            }
538    
539            if (shmctl(shmid, IPC_RMID, NULL) == -1)
540            {
541                    log_error("shmctl(shmid = %d, IPC_RMID) error (%d)\n", shmid, errno);
542            }
543    
544            p_section_list_pool = NULL;
545    }
546    
547    int set_section_list_shm_readonly(void)
548    {
549            int shmid;
550            void *p_shm;
551    
552            if (p_section_list_pool == NULL)
553            {
554                    log_error("p_section_list_pool not initialized\n");
555                    return -1;
556            }
557    
558            shmid = p_section_list_pool->shmid;
559    
560            // Remap shared memory in read-only mode
561            p_shm = shmat(shmid, p_section_list_pool, SHM_RDONLY | SHM_REMAP);
562            if (p_shm == (void *)-1)
563            {
564                    log_error("shmat(section_list_pool shmid = %d) error (%d)\n", shmid, errno);
565                    return -3;
566            }
567    
568            p_section_list_pool = p_shm;
569    
570            return 0;
571    }
572    
573    int detach_section_list_shm(void)
574    {
575            if (p_section_list_pool != NULL && shmdt(p_section_list_pool) == -1)
576            {
577                    log_error("shmdt(section_list_pool) error (%d)\n", errno);
578                    return -1;
579            }
580    
581            p_section_list_pool = NULL;
582    
583            return 0;
584    }
585    
586  inline static void sid_to_str(int32_t sid, char *p_sid_str)  inline static void sid_to_str(int32_t sid, char *p_sid_str)
587  {  {
588          uint32_t u_sid;          uint32_t u_sid;
# Line 463  inline static void sid_to_str(int32_t si Line 597  inline static void sid_to_str(int32_t si
597          p_sid_str[i] = '\0';          p_sid_str[i] = '\0';
598  }  }
599    
600  SECTION_LIST *section_list_create(int32_t sid, const char *sname, const char *stitle, const char *master_name)  SECTION_LIST *section_list_create(int32_t sid, const char *sname, const char *stitle, const char *master_list)
601  {  {
602          SECTION_LIST *p_section;          SECTION_LIST *p_section;
603          char sid_str[SID_STR_LEN];          char sid_str[SID_STR_LEN];
# Line 485  SECTION_LIST *section_list_create(int32_ Line 619  SECTION_LIST *section_list_create(int32_
619          p_section = p_section_list_pool->sections + p_section_list_pool->section_count;          p_section = p_section_list_pool->sections + p_section_list_pool->section_count;
620    
621          p_section->sid = sid;          p_section->sid = sid;
622            p_section->ex_menu_tm = 0;
623    
624          strncpy(p_section->sname, sname, sizeof(p_section->sname - 1));          strncpy(p_section->sname, sname, sizeof(p_section->sname) - 1);
625          p_section->sname[sizeof(p_section->sname - 1)] = '\0';          p_section->sname[sizeof(p_section->sname) - 1] = '\0';
626    
627          strncpy(p_section->stitle, stitle, sizeof(p_section->stitle - 1));          strncpy(p_section->stitle, stitle, sizeof(p_section->stitle) - 1);
628          p_section->stitle[sizeof(p_section->stitle - 1)] = '\0';          p_section->stitle[sizeof(p_section->stitle) - 1] = '\0';
629    
630          strncpy(p_section->master_name, master_name, sizeof(p_section->master_name - 1));          strncpy(p_section->master_list, master_list, sizeof(p_section->master_list) - 1);
631          p_section->master_name[sizeof(p_section->master_name - 1)] = '\0';          p_section->master_list[sizeof(p_section->master_list) - 1] = '\0';
632    
633          if (trie_dict_set(p_section_list_pool->p_trie_dict_section_by_name, sname, p_section_list_pool->section_count) != 1)          if (trie_dict_set(p_section_list_pool->p_trie_dict_section_by_name, sname, p_section_list_pool->section_count) != 1)
634          {          {
# Line 525  void section_list_reset_articles(SECTION Line 660  void section_list_reset_articles(SECTION
660    
661          p_section->page_count = 0;          p_section->page_count = 0;
662          p_section->last_page_visible_article_count = 0;          p_section->last_page_visible_article_count = 0;
 }  
   
 void section_list_cleanup(void)  
 {  
         int shmid;  
663    
664          if (p_section_list_pool == NULL)          p_section->ontop_article_count = 0;
         {  
                 return;  
         }  
   
         if (p_section_list_pool->p_trie_dict_section_by_name != NULL)  
         {  
                 trie_dict_destroy(p_section_list_pool->p_trie_dict_section_by_name);  
                 p_section_list_pool->p_trie_dict_section_by_name = NULL;  
         }  
   
         if (p_section_list_pool->p_trie_dict_section_by_sid != NULL)  
         {  
                 trie_dict_destroy(p_section_list_pool->p_trie_dict_section_by_sid);  
                 p_section_list_pool->p_trie_dict_section_by_sid = NULL;  
         }  
   
         if (p_section_list_pool != NULL)  
         {  
                 if (semctl(p_section_list_pool->semid, 0, IPC_RMID) == -1)  
                 {  
                         log_error("semctl(semid = %d, IPC_RMID) error (%d)\n", p_section_list_pool->semid, errno);  
                 }  
   
                 shmid = p_section_list_pool->shmid;  
   
                 if (shmdt(p_section_list_pool) == -1)  
                 {  
                         log_error("shmdt(shmid = %d) error (%d)\n", shmid, errno);  
                 }  
   
                 if (shmctl(shmid, IPC_RMID, NULL) == -1)  
                 {  
                         log_error("shmctl(shmid = %d, IPC_RMID) error (%d)\n", shmid, errno);  
                 }  
   
                 p_section_list_pool = NULL;  
         }  
665  }  }
666    
667  SECTION_LIST *section_list_find_by_name(const char *sname)  SECTION_LIST *section_list_find_by_name(const char *sname)
668  {  {
669          int64_t index;          int64_t index;
670            int ret;
671    
672          if (p_section_list_pool == NULL)          if (p_section_list_pool == NULL)
673          {          {
# Line 581  SECTION_LIST *section_list_find_by_name( Line 675  SECTION_LIST *section_list_find_by_name(
675                  return NULL;                  return NULL;
676          }          }
677    
678          if (trie_dict_get(p_section_list_pool->p_trie_dict_section_by_name, sname, &index) != 1)          ret = trie_dict_get(p_section_list_pool->p_trie_dict_section_by_name, sname, &index);
679            if (ret < 0)
680          {          {
681                  log_error("trie_dict_get(section, %s) error\n", sname);                  log_error("trie_dict_get(section, %s) error\n", sname);
682                  return NULL;                  return NULL;
683          }          }
684            else if (ret == 0)
685            {
686                    return NULL;
687            }
688    
689          return (p_section_list_pool->sections + index);          return (p_section_list_pool->sections + index);
690  }  }
# Line 593  SECTION_LIST *section_list_find_by_name( Line 692  SECTION_LIST *section_list_find_by_name(
692  SECTION_LIST *section_list_find_by_sid(int32_t sid)  SECTION_LIST *section_list_find_by_sid(int32_t sid)
693  {  {
694          int64_t index;          int64_t index;
695            int ret;
696          char sid_str[SID_STR_LEN];          char sid_str[SID_STR_LEN];
697    
698          if (p_section_list_pool == NULL)          if (p_section_list_pool == NULL)
# Line 603  SECTION_LIST *section_list_find_by_sid(i Line 703  SECTION_LIST *section_list_find_by_sid(i
703    
704          sid_to_str(sid, sid_str);          sid_to_str(sid, sid_str);
705    
706          if (trie_dict_get(p_section_list_pool->p_trie_dict_section_by_sid, sid_str, &index) != 1)          ret = trie_dict_get(p_section_list_pool->p_trie_dict_section_by_sid, sid_str, &index);
707            if (ret < 0)
708          {          {
709                  log_error("trie_dict_get(section, %d) error\n", sid);                  log_error("trie_dict_get(section, %d) error\n", sid);
710                  return NULL;                  return NULL;
711          }          }
712            else if (ret == 0)
713            {
714                    return NULL;
715            }
716    
717          return (p_section_list_pool->sections + index);          return (p_section_list_pool->sections + index);
718  }  }
# Line 622  int section_list_append_article(SECTION_ Line 727  int section_list_append_article(SECTION_
727    
728          if (p_section == NULL || p_article_src == NULL)          if (p_section == NULL || p_article_src == NULL)
729          {          {
730                  log_error("section_list_append_article() NULL pointer error\n");                  log_error("NULL pointer error\n");
731                  return -1;                  return -1;
732          }          }
733    
# Line 747  int section_list_append_article(SECTION_ Line 852  int section_list_append_article(SECTION_
852                  p_section->last_page_visible_article_count++;                  p_section->last_page_visible_article_count++;
853          }          }
854    
855            if (p_article->ontop && section_list_update_article_ontop(p_section, p_article) < 0)
856            {
857                    log_error("section_list_update_article_ontop(sid=%d, aid=%d) error\n",
858                                      p_section->sid, p_article->aid);
859                    return -5;
860            }
861    
862          return 0;          return 0;
863  }  }
864    
# Line 758  int section_list_set_article_visible(SEC Line 870  int section_list_set_article_visible(SEC
870    
871          if (p_section == NULL)          if (p_section == NULL)
872          {          {
873                  log_error("section_list_set_article_visible() NULL pointer error\n");                  log_error("NULL pointer error\n");
874                  return -2;                  return -1;
875          }          }
876    
877          p_article = article_block_find_by_aid(aid);          p_article = article_block_find_by_aid(aid);
# Line 770  int section_list_set_article_visible(SEC Line 882  int section_list_set_article_visible(SEC
882    
883          if (p_section->sid != p_article->sid)          if (p_section->sid != p_article->sid)
884          {          {
885                  log_error("section_list_set_article_visible() error: section sid %d != article sid %d\n", p_section->sid, p_article->sid);                  log_error("Inconsistent section sid %d != article sid %d\n", p_section->sid, p_article->sid);
886                  return -2;                  return -2;
887          }          }
888    
# Line 821  int section_list_set_article_visible(SEC Line 933  int section_list_set_article_visible(SEC
933          return affected_count;          return affected_count;
934  }  }
935    
936    int section_list_update_article_ontop(SECTION_LIST *p_section, ARTICLE *p_article)
937    {
938            int i;
939    
940            if (p_section == NULL || p_article == NULL)
941            {
942                    log_error("NULL pointer error\n");
943                    return -1;
944            }
945    
946            if (p_section->sid != p_article->sid)
947            {
948                    log_error("Inconsistent section sid %d != article sid %d\n", p_section->sid, p_article->sid);
949                    return -2;
950            }
951    
952            if (p_article->ontop)
953            {
954                    for (i = 0; i < p_section->ontop_article_count; i++)
955                    {
956                            if (p_section->p_ontop_articles[i]->aid == p_article->aid)
957                            {
958                                    log_error("Inconsistent state found: article %d already ontop in section %d\n", p_article->aid, p_section->sid);
959                                    return 0;
960                            }
961                            else if (p_section->p_ontop_articles[i]->aid > p_article->aid)
962                            {
963                                    break;
964                            }
965                    }
966    
967                    // Remove the oldest one if the array of ontop articles is full
968                    if (p_section->ontop_article_count >= BBS_ontop_article_limit_per_section)
969                    {
970                            if (i == 0) // p_article is the oldest one
971                            {
972                                    return 0;
973                            }
974                            memmove((void *)(p_section->p_ontop_articles),
975                                            (void *)(p_section->p_ontop_articles + 1),
976                                            sizeof(ARTICLE *) * (size_t)(i - 1));
977                            p_section->ontop_article_count--;
978                            i--;
979                    }
980                    else
981                    {
982                            memmove((void *)(p_section->p_ontop_articles + i + 1),
983                                            (void *)(p_section->p_ontop_articles + i),
984                                            sizeof(ARTICLE *) * (size_t)(p_section->ontop_article_count - i));
985                    }
986    
987                    p_section->p_ontop_articles[i] = p_article;
988                    p_section->ontop_article_count++;
989    
990                    // TODO: debug
991            }
992            else // ontop == 0
993            {
994                    for (i = 0; i < p_section->ontop_article_count; i++)
995                    {
996                            if (p_section->p_ontop_articles[i]->aid == p_article->aid)
997                            {
998                                    break;
999                            }
1000                    }
1001                    if (i == p_section->ontop_article_count) // not found
1002                    {
1003                            log_error("Inconsistent state found: article %d not ontop in section %d\n", p_article->aid, p_section->sid);
1004                            return 0;
1005                    }
1006    
1007                    memmove((void *)(p_section->p_ontop_articles + i),
1008                                    (void *)(p_section->p_ontop_articles + i + 1),
1009                                    sizeof(ARTICLE *) * (size_t)(p_section->ontop_article_count - i - 1));
1010                    p_section->ontop_article_count--;
1011            }
1012    
1013            return 0;
1014    }
1015    
1016    int section_list_page_count_with_ontop(SECTION_LIST *p_section)
1017    {
1018            int page_count;
1019    
1020            if (p_section == NULL)
1021            {
1022                    log_error("NULL pointer error\n");
1023                    return -1;
1024            }
1025    
1026            page_count = p_section->page_count - (p_section->last_page_visible_article_count > 0 ? 1 : 0) +
1027                                     (p_section->last_page_visible_article_count + p_section->ontop_article_count) / BBS_article_limit_per_page +
1028                                     ((p_section->last_page_visible_article_count + p_section->ontop_article_count) % BBS_article_limit_per_page == 0 ? 0 : 1);
1029    
1030            return page_count;
1031    }
1032    
1033    int section_list_page_article_count_with_ontop(SECTION_LIST *p_section, int32_t page_id)
1034    {
1035            if (p_section == NULL)
1036            {
1037                    log_error("NULL pointer error\n");
1038                    return -1;
1039            }
1040    
1041            if (page_id < p_section->page_count - 1)
1042            {
1043                    return BBS_article_limit_per_page;
1044            }
1045            else // if (page_id >= p_section->page_count - 1)
1046            {
1047                    return MAX(0, (p_section->last_page_visible_article_count + p_section->ontop_article_count -
1048                                               BBS_article_limit_per_page * (page_id - p_section->page_count + 1)));
1049            }
1050    }
1051    
1052  ARTICLE *section_list_find_article_with_offset(SECTION_LIST *p_section, int32_t aid, int32_t *p_page, int32_t *p_offset, ARTICLE **pp_next)  ARTICLE *section_list_find_article_with_offset(SECTION_LIST *p_section, int32_t aid, int32_t *p_page, int32_t *p_offset, ARTICLE **pp_next)
1053  {  {
1054          ARTICLE *p_article;          ARTICLE *p_article;
# Line 834  ARTICLE *section_list_find_article_with_ Line 1062  ARTICLE *section_list_find_article_with_
1062    
1063          if (p_section == NULL)          if (p_section == NULL)
1064          {          {
1065                  log_error("section_list_find_article_with_offset() NULL pointer error\n");                  log_error("NULL pointer error\n");
1066                  return NULL;                  return NULL;
1067          }          }
1068    
# Line 926  int section_list_calculate_page(SECTION_ Line 1154  int section_list_calculate_page(SECTION_
1154    
1155          if (p_section == NULL)          if (p_section == NULL)
1156          {          {
1157                  log_error("section_list_calculate_page() NULL pointer error\n");                  log_error("NULL pointer error\n");
1158                  return -1;                  return -1;
1159          }          }
1160    
# Line 1020  int section_list_calculate_page(SECTION_ Line 1248  int section_list_calculate_page(SECTION_
1248          return 0;          return 0;
1249  }  }
1250    
1251    int32_t article_block_last_aid(void)
1252    {
1253            ARTICLE_BLOCK *p_block = p_article_block_pool->p_block[p_article_block_pool->block_count - 1];
1254            int32_t last_aid = p_block->articles[p_block->article_count - 1].aid;
1255    
1256            return last_aid;
1257    }
1258    
1259    int article_block_article_count(void)
1260    {
1261            int ret;
1262    
1263            if (p_article_block_pool == NULL || p_article_block_pool->block_count <= 0)
1264            {
1265                    return -1;
1266            }
1267    
1268            ret = (p_article_block_pool->block_count - 1) * ARTICLE_PER_BLOCK +
1269                      p_article_block_pool->p_block[p_article_block_pool->block_count - 1]->article_count;
1270    
1271            return ret;
1272    }
1273    
1274  int article_count_of_topic(int32_t aid)  int article_count_of_topic(int32_t aid)
1275  {  {
1276          ARTICLE *p_article;          ARTICLE *p_article;
# Line 1060  int section_list_move_topic(SECTION_LIST Line 1311  int section_list_move_topic(SECTION_LIST
1311          int32_t first_inserted_aid_dest;          int32_t first_inserted_aid_dest;
1312          int move_counter;          int move_counter;
1313    
1314          if (p_section_dest == NULL)          if (p_section_src == NULL || p_section_dest == NULL)
1315            {
1316                    log_error("NULL pointer error\n");
1317                    return -1;
1318            }
1319    
1320            if (p_section_src->sid == p_section_dest->sid)
1321          {          {
1322                  log_error("section_list_move_topic() NULL pointer error\n");                  log_error("section_list_move_topic() src and dest section are the same\n");
1323                  return -1;                  return -1;
1324          }          }
1325    
1326          if ((p_article = article_block_find_by_aid(aid)) == NULL)          if ((p_article = article_block_find_by_aid(aid)) == NULL)
1327          {          {
1328                  log_error("section_list_move_topic() error: article %d not found in block\n", aid);                  log_error("article_block_find_by_aid(aid = %d) error: article not found\n", aid);
1329                  return -2;                  return -2;
1330          }          }
1331    
# Line 1109  int section_list_move_topic(SECTION_LIST Line 1366  int section_list_move_topic(SECTION_LIST
1366          {          {
1367                  if (p_section_src->sid != p_article->sid)                  if (p_section_src->sid != p_article->sid)
1368                  {                  {
1369                          log_error("section_list_move_topic() error: src section sid %d != article %d sid %d\n",                          log_error("section_list_move_topic() warning: src section sid %d != article %d sid %d\n",
1370                                            p_section_src->sid, p_article->aid, p_article->sid);                                            p_section_src->sid, p_article->aid, p_article->sid);
1371                          return -2;                          p_article = p_article->p_topic_next;
1372                            continue;
1373                  }                  }
1374    
1375                  // Remove from bi-directional article list of src section                  // Remove from bi-directional article list of src section
# Line 1137  int section_list_move_topic(SECTION_LIST Line 1395  int section_list_move_topic(SECTION_LIST
1395    
1396                  if (section_list_find_article_with_offset(p_section_dest, p_article->aid, &page, &offset, &p_next) != NULL)                  if (section_list_find_article_with_offset(p_section_dest, p_article->aid, &page, &offset, &p_next) != NULL)
1397                  {                  {
1398                          log_error("section_list_move_topic() error: article %d already in section %d\n", p_article->aid, p_section_dest->sid);                          log_error("section_list_move_topic() warning: article %d already in section %d\n", p_article->aid, p_section_dest->sid);
1399                          return -4;                          p_article = p_article->p_topic_next;
1400                            continue;
1401                  }                  }
1402    
1403                  // Insert into bi-directional article list of dest section                  // Insert into bi-directional article list of dest section
# Line 1216  int section_list_move_topic(SECTION_LIST Line 1475  int section_list_move_topic(SECTION_LIST
1475    
1476          if (p_section_dest->article_count - dest_article_count_old != move_article_count)          if (p_section_dest->article_count - dest_article_count_old != move_article_count)
1477          {          {
1478                  log_error("section_list_move_topic() error: count of moved articles %d != %d\n",                  log_error("section_list_move_topic() warning: count of moved articles %d != %d\n",
1479                                    p_section_dest->article_count - dest_article_count_old, move_article_count);                                    p_section_dest->article_count - dest_article_count_old, move_article_count);
1480          }          }
1481    
# Line 1406  int section_list_rw_unlock(SECTION_LIST Line 1665  int section_list_rw_unlock(SECTION_LIST
1665          }          }
1666    
1667          return ret;          return ret;
1668    }
1669    
1670    int section_list_rd_lock(SECTION_LIST *p_section)
1671    {
1672            int timer = 0;
1673            int sid = (p_section == NULL ? 0 : p_section->sid);
1674            int ret = -1;
1675    
1676            while (!SYS_server_exit)
1677            {
1678                    ret = section_list_try_rd_lock(p_section, SECTION_TRY_LOCK_WAIT_TIME);
1679                    if (ret == 0) // success
1680                    {
1681                            break;
1682                    }
1683                    else if (errno == EAGAIN || errno == EINTR) // retry
1684                    {
1685                            timer++;
1686                            if (timer % SECTION_TRY_LOCK_TIMES == 0)
1687                            {
1688                                    log_error("section_list_try_rd_lock() tried %d times on section %d\n", sid, timer);
1689                            }
1690                    }
1691                    else // failed
1692                    {
1693                            log_error("section_list_try_rd_lock() failed on section %d\n", sid);
1694                            break;
1695                    }
1696            }
1697    
1698            return ret;
1699    }
1700    
1701    int section_list_rw_lock(SECTION_LIST *p_section)
1702    {
1703            int timer = 0;
1704            int sid = (p_section == NULL ? 0 : p_section->sid);
1705            int ret = -1;
1706    
1707            while (!SYS_server_exit)
1708            {
1709                    ret = section_list_try_rw_lock(p_section, SECTION_TRY_LOCK_WAIT_TIME);
1710                    if (ret == 0) // success
1711                    {
1712                            break;
1713                    }
1714                    else if (errno == EAGAIN || errno == EINTR) // retry
1715                    {
1716                            timer++;
1717                            if (timer % SECTION_TRY_LOCK_TIMES == 0)
1718                            {
1719                                    log_error("section_list_try_rw_lock() tried %d times on section %d\n", sid, timer);
1720                            }
1721                    }
1722                    else // failed
1723                    {
1724                            log_error("section_list_try_rw_lock() failed on section %d\n", sid);
1725                            break;
1726                    }
1727            }
1728    
1729            return ret;
1730  }  }

Legend:
Removed lines/characters  
Changed lines/characters
  Added lines/characters
webmaster@leafok.com
 ViewVC Help
Powered by ViewVC 1.3.0-beta1